diff options
| author | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2014-03-11 15:16:42 -0300 |
|---|---|---|
| committer | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2014-03-11 15:16:42 -0300 |
| commit | 7e21be3c29fc8b4a8edbf93b9cc06b9e9186bb4a (patch) | |
| tree | 0ddd44c150600fe48e0db3b986619d34f57c9910 | |
| parent | ccaf0ed791d69567f91a659c9efc7d3bb5bc8b3b (diff) | |
| parent | 3c23f768440659c7c4eb20cea82032f71ef5556d (diff) | |
| download | rails-7e21be3c29fc8b4a8edbf93b9cc06b9e9186bb4a.tar.gz rails-7e21be3c29fc8b4a8edbf93b9cc06b9e9186bb4a.tar.bz2 rails-7e21be3c29fc8b4a8edbf93b9cc06b9e9186bb4a.zip | |
Merge pull request #14353 from arthurnn/remove_small_mock
Remove mocking on save, when not necessary
| -rw-r--r-- | activerecord/test/cases/autosave_association_test.rb | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/activerecord/test/cases/autosave_association_test.rb b/activerecord/test/cases/autosave_association_test.rb index 8e3825999d..09892d50ba 100644 --- a/activerecord/test/cases/autosave_association_test.rb +++ b/activerecord/test/cases/autosave_association_test.rb @@ -691,14 +691,12 @@ class TestDestroyAsPartOfAutosaveAssociation < ActiveRecord::TestCase def test_should_save_changed_has_one_changed_object_if_child_is_saved @pirate.ship.name = "NewName" - @pirate.ship.expects(:save).once.returns(true) - assert @pirate.save + assert_equal "NewName", @pirate.ship.reload.name end def test_should_not_save_changed_has_one_unchanged_object_if_child_is_saved @pirate.ship.expects(:save).never - assert @pirate.save end |