Merge branch 'master' of git@github.com:adamwiggins/rush

bmizerany · Mar 13, 2008 · 47f5ec3 · 47f5ec3
2 parents e4e8026 + e8c9424
commit 47f5ec3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/lib/rush/find_by.rb b/lib/rush/find_by.rb
@@ -12,6 +12,8 @@ def method_missing(meth, *args)
 			find_by(m[1], args.first)
 		elsif m = meth.to_s.match(/^find_all_by_([a-z_]+)$/)
 			find_all_by(m[1], args.first)
+		else
+			super
 		end
 	end
 

diff --git a/spec/find_by_spec.rb b/spec/find_by_spec.rb
@@ -7,8 +7,7 @@ class Foo
 
 			def initialize(bar)
 				@bar = bar
-			end
-		end
+			end end
 
 		@one = Foo.new('one')
 		@two = Foo.new('two')
@@ -52,4 +51,8 @@ def initialize(bar)
 	it "find_by_ with field not recognized by objects raises no errors" do
 		@list.find_by_nothing('x')
 	end
+
+	it "raises NoMethodError for things other than find_by" do
+		lambda { @list.does_not_exist }.should raise_error(NoMethodError)
+	end
 end