diff options
| author | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2014-07-15 16:12:23 -0300 |
|---|---|---|
| committer | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2014-07-15 18:09:38 -0300 |
| commit | 41fb06fa47b0c11d6b943163ec1bc8ce9fd4d229 (patch) | |
| tree | 8273991bd00e5b3b47d0994630b9d6875f5abdc6 /activerecord/test | |
| parent | 66d0a0153578ce760d822580c5b8c0b726042ac2 (diff) | |
| download | rails-41fb06fa47b0c11d6b943163ec1bc8ce9fd4d229.tar.gz rails-41fb06fa47b0c11d6b943163ec1bc8ce9fd4d229.tar.bz2 rails-41fb06fa47b0c11d6b943163ec1bc8ce9fd4d229.zip | |
Deprecate `reset_#{attribute}` in favor of `restore_#{attribute}`.
These methods may cause confusion with the `reset_changes` that
behaves differently
of them.
Also rename undo_changes to restore_changes to match this new set of
methods.
Diffstat (limited to 'activerecord/test')
| -rw-r--r-- | activerecord/test/cases/dirty_test.rb | 16 |
1 files changed, 14 insertions, 2 deletions
diff --git a/activerecord/test/cases/dirty_test.rb b/activerecord/test/cases/dirty_test.rb index ea73c561e9..69a7f25213 100644 --- a/activerecord/test/cases/dirty_test.rb +++ b/activerecord/test/cases/dirty_test.rb @@ -169,7 +169,19 @@ class DirtyTest < ActiveRecord::TestCase pirate = Pirate.create!(:catchphrase => 'Yar!') pirate.catchphrase = 'Ahoy!' - pirate.reset_catchphrase! + assert_deprecated do + pirate.reset_catchphrase! + end + assert_equal "Yar!", pirate.catchphrase + assert_equal Hash.new, pirate.changes + assert !pirate.catchphrase_changed? + end + + def test_restore_attribute! + pirate = Pirate.create!(:catchphrase => 'Yar!') + pirate.catchphrase = 'Ahoy!' + + pirate.restore_catchphrase! assert_equal "Yar!", pirate.catchphrase assert_equal Hash.new, pirate.changes assert !pirate.catchphrase_changed? @@ -398,7 +410,7 @@ class DirtyTest < ActiveRecord::TestCase def test_dup_objects_should_not_copy_dirty_flag_from_creator pirate = Pirate.create!(:catchphrase => "shiver me timbers") pirate_dup = pirate.dup - pirate_dup.reset_catchphrase! + pirate_dup.restore_catchphrase! pirate.catchphrase = "I love Rum" assert pirate.catchphrase_changed? assert !pirate_dup.catchphrase_changed? |