Merge pull request rails#4369 from lest/remove-warnings

use stub instead of redefining method to suppress warnings
drogus · Jan 7, 2012 · ba168e8 · ba168e8
2 parents f082004 + f1b05de
commit ba168e8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/actionpack/test/template/form_helper_test.rb b/actionpack/test/template/form_helper_test.rb
@@ -875,7 +875,7 @@ def test_form_for_with_remote_in_html
 
   def test_form_for_with_remote_without_html
     @post.persisted = false
-    def @post.to_key; nil; end
+    @post.stubs(:to_key).returns(nil)
     form_for(@post, :remote => true) do |f|
       concat f.text_field(:title)
       concat f.text_area(:body)
@@ -1025,7 +1025,7 @@ def test_submit_with_object_as_new_record_and_locale_strings
     old_locale, I18n.locale = I18n.locale, :submit
 
     @post.persisted = false
-    def @post.to_key; nil; end
+    @post.stubs(:to_key).returns(nil)
     form_for(@post) do |f|
       concat f.submit
     end