Merge pull request #9324 from acapilleri/add_on_empty

added more accurate test for add_on_empty

1 files changed, 7 insertions, 0 deletions

diff --git a/activemodel/test/cases/errors_test.rb b/activemodel/test/cases/errors_test.rb
index cc0c3f16d2..51dcfc37d8 100644
--- a/activemodel/test/cases/errors_test.rb
+++ b/activemodel/test/cases/errors_test.rb
@@ -274,6 +274,13 @@ class ErrorsTest < ActiveModel::TestCase
     person.errors.add_on_empty :name, :message => 'custom'
   end
 
+  test "add_on_empty generates message with empty string value" do
+    person = Person.new
+    person.name = ''
+    person.errors.expects(:generate_message).with(:name, :empty, {})
+    person.errors.add_on_empty :name
+  end
+
   test "add_on_blank generates message" do
     person = Person.new
     person.errors.expects(:generate_message).with(:name, :blank, {})