mirror of https://github.com/rails/rails
Cleaning up ActiveModel::Dirty tests
* Clarifying what the #changed method returns * Adding tests to describe what the #changed_attributes returns * Updating test name based on pull request comment * Moving the test lower in the file per pull request comment
This commit is contained in:
parent
bf095770e5
commit
63333e600f
|
@ -46,7 +46,7 @@ class DirtyTest < ActiveModel::TestCase
|
|||
assert @model.name_changed?
|
||||
end
|
||||
|
||||
test "list of changed attributes" do
|
||||
test "list of changed attribute keys" do
|
||||
assert_equal [], @model.changed
|
||||
@model.name = "Paul"
|
||||
assert_equal ['name'], @model.changed
|
||||
|
@ -106,6 +106,17 @@ class DirtyTest < ActiveModel::TestCase
|
|||
assert_equal [nil, "Jericho Cane"], @model.previous_changes['name']
|
||||
end
|
||||
|
||||
test "previous value is preserved when changed after save" do
|
||||
assert_equal({}, @model.changed_attributes)
|
||||
@model.name = "Paul"
|
||||
assert_equal({ "name" => nil }, @model.changed_attributes)
|
||||
|
||||
@model.save
|
||||
|
||||
@model.name = "John"
|
||||
assert_equal({ "name" => "Paul" }, @model.changed_attributes)
|
||||
end
|
||||
|
||||
test "changing the same attribute multiple times retains the correct original value" do
|
||||
@model.name = "Otto"
|
||||
@model.save
|
||||
|
|
Loading…
Reference in New Issue