Merge pull request #107 from Burgestrand/base-respond_to

Add CouchRest::Model::Base.respond_to_missing? and respond_to?
This commit is contained in:
Marcos Tapajós 2011-08-20 21:58:11 -07:00
commit 29de79290f
2 changed files with 31 additions and 1 deletions

View file

@ -82,6 +82,21 @@ module CouchRest
super
end
# compatbility for 1.8, it does not use respond_to_missing?
# thing is, when using it like this only, doing method(:find_by_view)
# will throw an error
def self.respond_to?(m, include_private = false)
super || respond_to_missing?(m, include_private)
end
# ruby 1.9 feature
# this allows ruby to know that the method is defined using
# method_missing, and as such, method(:find_by_view) will actually
# give a Method back, and not throw an error like in 1.8!
def self.respond_to_missing?(m, include_private = false)
has_view?(m) || has_view?(m.to_s[/^find_(by_.+)/, 1])
end
def to_key
new? ? nil : [id]
end

View file

@ -173,7 +173,22 @@ describe CouchRest::Model::Views do
end
end
describe "#method_missing for find_by methods" do
before(:all) { reset_test_db! }
specify { Course.should respond_to :find_by_title_and_active }
specify { Course.should respond_to :by_title }
specify "#method should work in ruby 1.9, but not 1.8" do
if RUBY_VERSION >= "1.9"
Course.method(:find_by_title_and_active).should be_a Method
else
expect { Course.method(:find_by_title_and_active) }.to raise_error(NameError)
end
end
end
describe "a ducktype view" do
before(:all) do
reset_test_db!