diff options
| author | Renato Mascarenhas <mascarenhas.renato@gmail.com> | 2012-12-01 15:29:49 -0200 |
|---|---|---|
| committer | Renato Mascarenhas <mascarenhas.renato@gmail.com> | 2012-12-01 16:58:09 -0200 |
| commit | cf7ab6056adfd1d2feb98444d82f89adcb6e5533 (patch) | |
| tree | 7b5b1a3ed17327dd23779f2bd24d5b32e2710914 /activemodel/test | |
| parent | 0181c2da977fc3de4e4c4eac602b26ff180cda2c (diff) | |
| download | rails-cf7ab6056adfd1d2feb98444d82f89adcb6e5533.tar.gz rails-cf7ab6056adfd1d2feb98444d82f89adcb6e5533.tar.bz2 rails-cf7ab6056adfd1d2feb98444d82f89adcb6e5533.zip | |
Reset attributes should not report changes.
When resetting an attribute, you expect it to return to the state it was
before any changes. Namely, this fixes this unexpected behavior:
~~~ruby
model.name = "Bob"
model.reset_name!
model.name_changed? #=> true
~~~
Diffstat (limited to 'activemodel/test')
| -rw-r--r-- | activemodel/test/cases/dirty_test.rb | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/activemodel/test/cases/dirty_test.rb b/activemodel/test/cases/dirty_test.rb index eaaf910bac..0b9f9537e2 100644 --- a/activemodel/test/cases/dirty_test.rb +++ b/activemodel/test/cases/dirty_test.rb @@ -78,7 +78,7 @@ class DirtyTest < ActiveModel::TestCase @model.name = "Bob" @model.reset_name! assert_nil @model.name - #assert !@model.name_changed #Doesn't work yet + assert !@model.name_changed? end test "setting color to same value should not result in change being recorded" do @@ -114,5 +114,4 @@ class DirtyTest < ActiveModel::TestCase assert_equal ["Otto", "Mr. Manfredgensonton"], @model.name_change assert_equal @model.name_was, "Otto" end - end |