aboutsummaryrefslogtreecommitdiffstats
path: root/activemodel/test
diff options
context:
space:
mode:
authorJahfer Husain <jahfer.husain@shopify.com>2017-07-07 13:16:06 -0400
committerJahfer Husain <jahfer.husain@shopify.com>2017-07-07 14:32:59 -0400
commit3650ca983c5c2ffd1a2993fa091bf504594325a7 (patch)
treefa37bcac571b39046e4affd9d81fac7edda68d36 /activemodel/test
parentc9b514cd3e8fe7d74ae91ecd30db839132e7899c (diff)
downloadrails-3650ca983c5c2ffd1a2993fa091bf504594325a7.tar.gz
rails-3650ca983c5c2ffd1a2993fa091bf504594325a7.tar.bz2
rails-3650ca983c5c2ffd1a2993fa091bf504594325a7.zip
Add ActiveModel::Errors#merge!
ActiveModel::Errors#merge! allows ActiveModel::Errors to append errors from a separate ActiveModel::Errors instance onto their own. Example: person = Person.new person.errors.add(:name, :blank) errors = ActiveModel::Errors.new(Person.new) errors.add(:name, :invalid) person.errors.merge!(errors) puts person.errors.messages # => { name: ["can't be blank", "is invalid"] }
Diffstat (limited to 'activemodel/test')
-rw-r--r--activemodel/test/cases/errors_test.rb12
1 files changed, 12 insertions, 0 deletions
diff --git a/activemodel/test/cases/errors_test.rb b/activemodel/test/cases/errors_test.rb
index 43aee5a814..7383b3e9b0 100644
--- a/activemodel/test/cases/errors_test.rb
+++ b/activemodel/test/cases/errors_test.rb
@@ -375,6 +375,18 @@ class ErrorsTest < ActiveModel::TestCase
assert_equal [:name], person.errors.details.keys
end
+ test "merge errors" do
+ errors = ActiveModel::Errors.new(Person.new)
+ errors.add(:name, :invalid)
+
+ person = Person.new
+ person.errors.add(:name, :blank)
+ person.errors.merge!(errors)
+
+ assert_equal({ name: ["can't be blank", "is invalid"] }, person.errors.messages)
+ assert_equal({ name: [{ error: :blank }, { error: :invalid }] }, person.errors.details)
+ end
+
test "errors are marshalable" do
errors = ActiveModel::Errors.new(Person.new)
errors.add(:name, :invalid)