Permalink
Browse files

Merge pull request #4369 from lest/remove-warnings

use stub instead of redefining method to suppress warnings
  • Loading branch information...
2 parents f082004 + f1b05de commit ba168e8f067d72f7a21691f4cfdcd40432135246 @drogus drogus committed Jan 7, 2012
Showing with 2 additions and 2 deletions.
  1. +2 −2 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

0 comments on commit ba168e8

Please sign in to comment.