2 files changed, 71 insertions, 35 deletions
diff --git a/activemodel/test/cases/validations/i18n_validation_test.rb b/activemodel/test/cases/validations/i18n_validation_test.rb
index 70ee7afecc..be17575402 100644
--- a/activemodel/test/cases/validations/i18n_validation_test.rb
+++ b/activemodel/test/cases/validations/i18n_validation_test.rb
@@ -11,6 +11,7 @@ class I18nValidationTest < ActiveModel::TestCase
     I18n.load_path.clear
     I18n.backend = I18n::Backend::Simple.new
     I18n.backend.store_translations('en', errors: { messages: { custom: nil } })
+    @mock_generator = MiniTest::Mock.new
   end
 
   def teardown
@@ -18,6 +19,7 @@ class I18nValidationTest < ActiveModel::TestCase
     I18n.load_path.replace @old_load_path
     I18n.backend = @old_backend
     I18n.backend.reload!
+    @mock_generator.verify
   end
 
   def test_full_message_encoding
@@ -30,8 +32,10 @@ class I18nValidationTest < ActiveModel::TestCase
 
   def test_errors_full_messages_translates_human_attribute_name_for_model_attributes
     @person.errors.add(:name, 'not found')
-    Person.expects(:human_attribute_name).with(:name, default: 'Name').returns("Person's name")
-    assert_equal ["Person's name not found"], @person.errors.full_messages
+    @mock_generator.expect(:call, "Person's name", [:name, default: 'Name'])
+    Person.stub(:human_attribute_name, @mock_generator) do
+      assert_equal ["Person's name not found"], @person.errors.full_messages
+    end
   end
 
   def test_errors_full_messages_uses_format
@@ -60,8 +64,10 @@ class I18nValidationTest < ActiveModel::TestCase
     test "validates_confirmation_of on generated message #{name}" do
       Person.validates_confirmation_of :title, validation_options
       @person.title_confirmation = 'foo'
-      @person.errors.expects(:generate_message).with(:title_confirmation, :confirmation, generate_message_options.merge(attribute: 'Title'))
-      @person.valid?
+      @mock_generator.expect(:call, nil, [:title_confirmation, :confirmation, generate_message_options.merge(attribute: 'Title')])
+      @person.errors.stub(:generate_message, @mock_generator) do
+        @person.valid?
+      end
     end
   end
 
@@ -70,8 +76,10 @@ class I18nValidationTest < ActiveModel::TestCase
   COMMON_CASES.each do |name, validation_options, generate_message_options|
     test "validates_acceptance_of on generated message #{name}" do
       Person.validates_acceptance_of :title, validation_options.merge(allow_nil: false)
-      @person.errors.expects(:generate_message).with(:title, :accepted, generate_message_options)
-      @person.valid?
+      @mock_generator.expect(:call, nil, [:title, :accepted, generate_message_options])
+      @person.errors.stub(:generate_message, @mock_generator) do
+        @person.valid?
+      end
     end
   end
 
@@ -80,8 +88,10 @@ class I18nValidationTest < ActiveModel::TestCase
   COMMON_CASES.each do |name, validation_options, generate_message_options|
     test "validates_presence_of on generated message #{name}" do
       Person.validates_presence_of :title, validation_options
-      @person.errors.expects(:generate_message).with(:title, :blank, generate_message_options)
-      @person.valid?
+      @mock_generator.expect(:call, nil, [:title, :blank, generate_message_options])
+      @person.errors.stub(:generate_message, @mock_generator) do
+        @person.valid?
+      end
     end
   end
 
@@ -90,8 +100,10 @@ class I18nValidationTest < ActiveModel::TestCase
   COMMON_CASES.each do |name, validation_options, generate_message_options|
     test "validates_length_of for :withing on generated message when too short #{name}" do
       Person.validates_length_of :title, validation_options.merge(within: 3..5)
-      @person.errors.expects(:generate_message).with(:title, :too_short, generate_message_options.merge(count: 3))
-      @person.valid?
+      @mock_generator.expect(:call, nil, [:title, :too_short, generate_message_options.merge(count: 3)])
+      @person.errors.stub(:generate_message, @mock_generator) do
+        @person.valid?
+      end
     end
   end
 
@@ -101,8 +113,10 @@ class I18nValidationTest < ActiveModel::TestCase
     test "validates_length_of for :too_long generated message #{name}" do
       Person.validates_length_of :title, validation_options.merge(within: 3..5)
       @person.title = 'this title is too long'
-      @person.errors.expects(:generate_message).with(:title, :too_long, generate_message_options.merge(count: 5))
-      @person.valid?
+      @mock_generator.expect(:call, nil, [:title, :too_long, generate_message_options.merge(count: 5)])
+      @person.errors.stub(:generate_message, @mock_generator) do
+        @person.valid?
+      end
     end
   end
 
@@ -111,8 +125,10 @@ class I18nValidationTest < ActiveModel::TestCase
   COMMON_CASES.each do |name, validation_options, generate_message_options|
     test "validates_length_of for :is on generated message #{name}" do
       Person.validates_length_of :title, validation_options.merge(is: 5)
-      @person.errors.expects(:generate_message).with(:title, :wrong_length, generate_message_options.merge(count: 5))
-      @person.valid?
+      @mock_generator.expect(:call, nil, [:title, :wrong_length, generate_message_options.merge(count: 5)])
+      @person.errors.stub(:generate_message, @mock_generator) do
+        @person.valid?
+      end
     end
   end
 
@@ -122,8 +138,10 @@ class I18nValidationTest < ActiveModel::TestCase
     test "validates_format_of on generated message #{name}" do
       Person.validates_format_of :title, validation_options.merge(with: /\A[1-9][0-9]*\z/)
       @person.title = '72x'
-      @person.errors.expects(:generate_message).with(:title, :invalid, generate_message_options.merge(value: '72x'))
-      @person.valid?
+      @mock_generator.expect(:call, nil, [:title, :invalid, generate_message_options.merge(value: '72x')])
+      @person.errors.stub(:generate_message, @mock_generator) do
+        @person.valid?
+      end
     end
   end
 
@@ -133,8 +151,10 @@ class I18nValidationTest < ActiveModel::TestCase
     test "validates_inclusion_of on generated message #{name}" do
       Person.validates_inclusion_of :title, validation_options.merge(in: %w(a b c))
       @person.title = 'z'
-      @person.errors.expects(:generate_message).with(:title, :inclusion, generate_message_options.merge(value: 'z'))
-      @person.valid?
+      @mock_generator.expect(:call, nil, [:title, :inclusion, generate_message_options.merge(value: 'z')])
+      @person.errors.stub(:generate_message, @mock_generator) do
+        @person.valid?
+      end
     end
   end
 
@@ -144,8 +164,10 @@ class I18nValidationTest < ActiveModel::TestCase
     test "validates_inclusion_of using :within on generated message #{name}" do
       Person.validates_inclusion_of :title, validation_options.merge(within: %w(a b c))
       @person.title = 'z'
-      @person.errors.expects(:generate_message).with(:title, :inclusion, generate_message_options.merge(value: 'z'))
-      @person.valid?
+      @mock_generator.expect(:call, nil, [:title, :inclusion, generate_message_options.merge(value: 'z')])
+      @person.errors.stub(:generate_message, @mock_generator) do
+        @person.valid?
+      end
     end
   end
 
@@ -155,8 +177,10 @@ class I18nValidationTest < ActiveModel::TestCase
     test "validates_exclusion_of generated message #{name}" do
       Person.validates_exclusion_of :title, validation_options.merge(in: %w(a b c))
       @person.title = 'a'
-      @person.errors.expects(:generate_message).with(:title, :exclusion, generate_message_options.merge(value: 'a'))
-      @person.valid?
+      @mock_generator.expect(:call, nil, [:title, :exclusion, generate_message_options.merge(value: 'a')])
+      @person.errors.stub(:generate_message, @mock_generator) do
+        @person.valid?
+      end
     end
   end
 
@@ -166,8 +190,10 @@ class I18nValidationTest < ActiveModel::TestCase
     test "validates_exclusion_of using :within generated message #{name}" do
       Person.validates_exclusion_of :title, validation_options.merge(within: %w(a b c))
       @person.title = 'a'
-      @person.errors.expects(:generate_message).with(:title, :exclusion, generate_message_options.merge(value: 'a'))
-      @person.valid?
+      @mock_generator.expect(:call, nil, [:title, :exclusion, generate_message_options.merge(value: 'a')])
+      @person.errors.stub(:generate_message, @mock_generator) do
+        @person.valid?
+      end
     end
   end
 
@@ -177,8 +203,10 @@ class I18nValidationTest < ActiveModel::TestCase
     test "validates_numericality_of generated message #{name}" do
       Person.validates_numericality_of :title, validation_options
       @person.title = 'a'
-      @person.errors.expects(:generate_message).with(:title, :not_a_number, generate_message_options.merge(value: 'a'))
-      @person.valid?
+      @mock_generator.expect(:call, nil, [:title, :not_a_number, generate_message_options.merge(value: 'a')])
+      @person.errors.stub(:generate_message, @mock_generator) do
+        @person.valid?
+      end
     end
   end
 
@@ -188,8 +216,10 @@ class I18nValidationTest < ActiveModel::TestCase
     test "validates_numericality_of for :only_integer on generated message #{name}" do
       Person.validates_numericality_of :title, validation_options.merge(only_integer: true)
       @person.title = '0.0'
-      @person.errors.expects(:generate_message).with(:title, :not_an_integer, generate_message_options.merge(value: '0.0'))
-      @person.valid?
+      @mock_generator.expect(:call, nil, [:title, :not_an_integer, generate_message_options.merge(value: '0.0')])
+      @person.errors.stub(:generate_message, @mock_generator) do
+        @person.valid?
+      end
     end
   end
 
@@ -199,8 +229,10 @@ class I18nValidationTest < ActiveModel::TestCase
     test "validates_numericality_of for :odd on generated message #{name}" do
       Person.validates_numericality_of :title, validation_options.merge(only_integer: true, odd: true)
       @person.title = 0
-      @person.errors.expects(:generate_message).with(:title, :odd, generate_message_options.merge(value: 0))
-      @person.valid?
+      @mock_generator.expect(:call, nil, [:title, :odd, generate_message_options.merge(value: 0)])
+      @person.errors.stub(:generate_message, @mock_generator) do
+        @person.valid?
+      end
     end
   end
 
@@ -210,8 +242,10 @@ class I18nValidationTest < ActiveModel::TestCase
     test "validates_numericality_of for :less_than on generated message #{name}" do
       Person.validates_numericality_of :title, validation_options.merge(only_integer: true, less_than: 0)
       @person.title = 1
-      @person.errors.expects(:generate_message).with(:title, :less_than, generate_message_options.merge(value: 1, count: 0))
-      @person.valid?
+      @mock_generator.expect(:call, nil, [:title, :less_than, generate_message_options.merge(value: 1, count: 0)])
+      @person.errors.stub(:generate_message, @mock_generator) do
+        @person.valid?
+      end
     end
   end
 
diff --git a/activemodel/test/cases/validations/with_validation_test.rb b/activemodel/test/cases/validations/with_validation_test.rb
index 01804032f0..9ee8b79da9 100644
--- a/activemodel/test/cases/validations/with_validation_test.rb
+++ b/activemodel/test/cases/validations/with_validation_test.rb
@@ -97,12 +97,14 @@ class ValidatesWithTest < ActiveModel::TestCase
 
   test "passes all configuration options to the validator class" do
     topic = Topic.new
-    validator = mock()
-    validator.expects(:new).with(foo: :bar, if: "1 == 1", class: Topic).returns(validator)
-    validator.expects(:validate).with(topic)
+    validator = MiniTest::Mock.new
+    validator.expect(:new, validator, [{foo: :bar, if: "1 == 1", class: Topic}])
+    validator.expect(:validate, nil, [topic])
+    validator.expect(:is_a?, false, [Symbol])
 
     Topic.validates_with(validator, if: "1 == 1", foo: :bar)
     assert topic.valid?
+    validator.verify
   end
 
   test "validates_with with options" do