diff options
author | Sean Griffin <sean@thoughtbot.com> | 2014-07-12 18:30:49 -0600 |
---|---|---|
committer | Godfrey Chan <godfreykfc@gmail.com> | 2014-08-16 23:08:41 -0700 |
commit | 877ea784e4cd0d539bdfbd15839ae3d28169b156 (patch) | |
tree | fe249c2d53d6cd7062b131b6157b3d00bbb7a683 /activerecord/test/cases | |
parent | 88d27ae9181151f448f39298adce9d1b7401a376 (diff) | |
download | rails-877ea784e4cd0d539bdfbd15839ae3d28169b156.tar.gz rails-877ea784e4cd0d539bdfbd15839ae3d28169b156.tar.bz2 rails-877ea784e4cd0d539bdfbd15839ae3d28169b156.zip |
Implement `_was` and `changes` for in-place mutations of AR attributes
Diffstat (limited to 'activerecord/test/cases')
-rw-r--r-- | activerecord/test/cases/dirty_test.rb | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/activerecord/test/cases/dirty_test.rb b/activerecord/test/cases/dirty_test.rb index 69a7f25213..0c77eedb52 100644 --- a/activerecord/test/cases/dirty_test.rb +++ b/activerecord/test/cases/dirty_test.rb @@ -661,6 +661,27 @@ class DirtyTest < ActiveRecord::TestCase assert_not model.foo_changed? end + test "in place mutation detection" do + pirate = Pirate.create!(catchphrase: "arrrr") + pirate.catchphrase << " matey!" + + assert pirate.catchphrase_changed? + expected_changes = { + "catchphrase" => ["arrrr", "arrrr matey!"] + } + assert_equal(expected_changes, pirate.changes) + assert_equal("arrrr", pirate.catchphrase_was) + assert pirate.catchphrase_changed?(from: "arrrr") + assert_not pirate.catchphrase_changed?(from: "anything else") + assert pirate.changed_attributes.include?(:catchphrase) + + pirate.save! + pirate.reload + + assert_equal "arrrr matey!", pirate.catchphrase + assert_not pirate.changed? + end + private def with_partial_writes(klass, on = true) old = klass.partial_writes? |