assert 2.19.0 → 2.19.5

data/test/unit/assert_tests.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "assert"
 
 require "assert/config"
@@ -7,9 +9,9 @@ require "much-stub"
 module Assert
   class UnitTests < Assert::Context
     desc "Assert"
-    subject { unit_class }
+    subject{ unit_class }
 
-    let(:unit_class) { Assert }
+    let(:unit_class){ Assert }
 
     should have_imeths :config, :configure, :view, :suite, :runner
     should have_imeths :stubs, :stub, :unstub, :unstub!, :stub_send
@@ -29,15 +31,20 @@ module Assert
   end
 
   class StubTests < UnitTests
-    let(:class1) {
+    let(:class1) do
       Class.new do
-        def initialize(value); @value = value; end
-        def mymeth; @value; end
+        def initialize(value)
+          @value = value
+        end
+
+        def mymeth
+          @value
+        end
       end
-    }
-    let(:object1) { class1.new(orig_value1) }
-    let(:orig_value1) { Factory.string }
-    let(:stub_value1) { Factory.string }
+    end
+    let(:object1){ class1.new(orig_value1) }
+    let(:orig_value1){ Factory.string }
+    let(:stub_value1){ Factory.string }
 
     should "build a stub" do
       stub1 = Assert.stub(object1, :mymeth)
@@ -47,9 +54,9 @@ module Assert
     should "build a stub with an on_call block" do
       my_meth_called_with = nil
       stub1 =
-        Assert.stub_on_call(object1, :mymeth) { |call|
+        Assert.stub_on_call(object1, :mymeth) do |call|
           my_meth_called_with = call
-        }
+        end
 
       object1.mymeth
       assert_kind_of MuchStub::Stub, stub1
@@ -64,7 +71,7 @@ module Assert
 
     should "set the stub's do block if given a block" do
       Assert.stub(object1, :mymeth)
-      assert_that { object1.mymeth }.raises(MuchStub::NotStubbedError)
+      assert_that{ object1.mymeth }.raises(MuchStub::NotStubbedError)
       Assert.stub(object1, :mymeth){ stub_value1 }
       assert_that(object1.mymeth).equals(stub_value1)
     end
@@ -96,7 +103,7 @@ module Assert
     should "auto-unstub any stubs on teardown" do
       context_class = ::Factory.modes_off_context_class do
         setup do
-          Assert.stub("1", :to_s){ "one" }
+          Assert.stub(+"1", :to_s){ "one" }
         end
       end
 
@@ -109,7 +116,7 @@ module Assert
 
     should "be able to call a stub's original method" do
       err =
-        assert_that {
+        assert_that{
           Assert.stub_send(object1, :mymeth)
         }.raises(MuchStub::NotStubbedError)
       assert_that(err.message).includes("not stubbed.")
@@ -123,9 +130,9 @@ module Assert
 
     should "be able to add a stub tap" do
       my_meth_called_with = nil
-      Assert.stub_tap(object1, :mymeth){ |value, *args, &block|
+      Assert.stub_tap(object1, :mymeth) do |_value, *args|
         my_meth_called_with = args
-      }
+      end
 
       assert_that(object1.mymeth).equals(orig_value1)
       assert_that(my_meth_called_with).equals([])
@@ -133,9 +140,9 @@ module Assert
 
     should "be able to add a stub tap with an on_call block" do
       my_meth_called_with = nil
-      Assert.stub_tap_on_call(object1, :mymeth){ |value, call|
+      Assert.stub_tap_on_call(object1, :mymeth) do |_value, call|
         my_meth_called_with = call
-      }
+      end
 
       assert_that(object1.mymeth).equals(orig_value1)
       assert_that(my_meth_called_with.args).equals([])
@@ -143,10 +150,21 @@ module Assert
 
     should "be able to add a stubbed spy" do
       myclass = Class.new do
-        def one; self; end
-        def two(val); self; end
-        def three; self; end
-        def ready?; false; end
+        def one
+          self
+        end
+
+        def two(_val)
+          self
+        end
+
+        def three
+          self
+        end
+
+        def ready?
+          false
+        end
       end
       myobj = myclass.new
 
@@ -156,7 +174,8 @@ module Assert
           :one,
           :two,
           :three,
-          ready?: true)
+          ready?: true,
+        )
 
       assert_that(myobj.one).equals(spy)
       assert_that(myobj.two("a")).equals(spy)

data/test/unit/assertions/assert_block_tests.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "assert"
 require "assert/assertions"
 
@@ -6,15 +8,15 @@ module Assert::Assertions
     include Assert::Test::TestHelpers
 
     desc "`assert_block`"
-    subject {
+    subject do
       desc = desc1
       Factory.test do
-        assert_block { true }        # pass
-        assert_block(desc) { false } # fail
+        assert_block{ true }        # pass
+        assert_block(desc){ false } # fail
       end
-    }
+    end
 
-    let(:desc1) { "assert block fail desc" }
+    let(:desc1){ "assert block fail desc" }
 
     should "produce results as expected" do
       subject.run(&test_run_callback)
@@ -32,15 +34,15 @@ module Assert::Assertions
     include Assert::Test::TestHelpers
 
     desc "`assert_not_block`"
-    subject {
+    subject do
       desc = desc1
       Factory.test do
-        assert_not_block(desc) { true } # fail
-        assert_not_block { false }      # pass
+        assert_not_block(desc){ true } # fail
+        assert_not_block{ false }      # pass
       end
-    }
+    end
 
-    let(:desc1) { "assert not block fail desc" }
+    let(:desc1){ "assert not block fail desc" }
 
     should "produce results as expected" do
       subject.run(&test_run_callback)

data/test/unit/assertions/assert_changes_tests.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "assert"
 require "assert/assertions"
 
@@ -6,30 +8,30 @@ module Assert::Assertions
     include Assert::Test::TestHelpers
 
     desc "`assert_changes`"
-    subject {
+    subject do
       desc = desc1
       Factory.test do
         @my_var = 1
-        assert_changes("@my_var", from: 1, to: 2) { @my_var = 2 } # pass
+        assert_changes("@my_var", from: 1, to: 2){ @my_var = 2 } # pass
         @my_var = 1
-        assert_changes("@my_var", from: 1) { @my_var = 2 }        # pass
+        assert_changes("@my_var", from: 1){ @my_var = 2 }        # pass
         @my_var = 1
-        assert_changes("@my_var", to: 2) { @my_var = 2 }          # pass
+        assert_changes("@my_var", to: 2){ @my_var = 2 }          # pass
         @my_var = 1
-        assert_changes("@my_var", desc: desc) { @my_var = 2 }     # pass
+        assert_changes("@my_var", desc: desc){ @my_var = 2 }     # pass
 
         @my_var = 1
-        assert_changes("@my_var", from: 2, to: 1) { @my_var = 2 } # fail
+        assert_changes("@my_var", from: 2, to: 1){ @my_var = 2 } # fail
         @my_var = 1
-        assert_changes("@my_var", from: 2) { @my_var = 2 }        # fail
+        assert_changes("@my_var", from: 2){ @my_var = 2 }        # fail
         @my_var = 1
-        assert_changes("@my_var", to: 1) { @my_var = 2 }          # fail
+        assert_changes("@my_var", to: 1){ @my_var = 2 }          # fail
         @my_var = 1
-        assert_changes("@my_var", desc: desc) { @my_var = 1 }     # fail
+        assert_changes("@my_var", desc: desc){ @my_var = 1 }     # fail
       end
-    }
+    end
 
-    let(:desc1) { "assert changes fail desc" }
+    let(:desc1){ "assert changes fail desc" }
 
     should "produce results as expected" do
       subject.run(&test_run_callback)
@@ -48,7 +50,9 @@ module Assert::Assertions
           "it was `1` and didn't change",
         ]
       messages = test_run_results(:fail).map(&:message)
-      messages.each_with_index{ |msg, n| assert_that(msg).matches(/^#{exp[n]}/) }
+      messages.each_with_index do |msg, n|
+        assert_that(msg).matches(/^#{exp[n]}/)
+      end
     end
   end
 
@@ -56,24 +60,24 @@ module Assert::Assertions
     include Assert::Test::TestHelpers
 
     desc "`assert_changes`"
-    subject {
+    subject do
       desc = desc1
       Factory.test do
         @my_var = 1
-        assert_not_changes("@my_var", from: 1) { @my_var = 1 }    # pass
+        assert_not_changes("@my_var", from: 1){ @my_var = 1 }    # pass
         @my_var = 1
-        assert_not_changes("@my_var", desc: desc) { @my_var = 1 } # pass
+        assert_not_changes("@my_var", desc: desc){ @my_var = 1 } # pass
 
         @my_var = 1
-        assert_not_changes("@my_var", from: 2) { @my_var = 1 }    # fail
+        assert_not_changes("@my_var", from: 2){ @my_var = 1 }    # fail
         @my_var = 1
-        assert_not_changes("@my_var", from: 1) { @my_var = 2 }    # fail
+        assert_not_changes("@my_var", from: 1){ @my_var = 2 }    # fail
         @my_var = 1
-        assert_not_changes("@my_var", desc: desc) { @my_var = 2 } # fail
+        assert_not_changes("@my_var", desc: desc){ @my_var = 2 } # fail
       end
-    }
+    end
 
-    let(:desc1) { "assert not changes fail desc" }
+    let(:desc1){ "assert not changes fail desc" }
 
     should "produce results as expected" do
       subject.run(&test_run_callback)
@@ -91,7 +95,9 @@ module Assert::Assertions
           "it was `1` and changed to `2`",
         ]
       messages = test_run_results(:fail).map(&:message)
-      messages.each_with_index{ |msg, n| assert_that(msg).matches(/^#{exp[n]}/) }
+      messages.each_with_index do |msg, n|
+        assert_that(msg).matches(/^#{exp[n]}/)
+      end
     end
   end
 end

data/test/unit/assertions/assert_empty_tests.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "assert"
 require "assert/assertions"
 
@@ -8,17 +10,17 @@ module Assert::Assertions
     include Assert::Test::TestHelpers
 
     desc "`assert_empty`"
-    subject {
+    subject do
       args = args1
       Factory.test do
         assert_empty([])    # pass
         assert_empty(*args) # fail
       end
-    }
+    end
 
-    let(:desc1) { "assert empty fail desc" }
-    let(:args1) { [[1], desc1] }
-    let(:config1) { subject.config }
+    let(:desc1){ "assert empty fail desc" }
+    let(:args1){ [[1], desc1] }
+    let(:config1){ subject.config }
 
     should "produce results as expected" do
       subject.run(&test_run_callback)
@@ -28,7 +30,8 @@ module Assert::Assertions
       assert_that(test_run_result_count(:fail)).equals(1)
 
       exp =
-        "#{args1[1]}\nExpected #{Assert::U.show(args1[0], config1)} to be empty."
+        "#{args1[1]}\nExpected #{Assert::U.show(args1[0], config1)} to be "\
+        "empty."
       assert_that(test_run_results(:fail).first.message).equals(exp)
     end
   end
@@ -37,17 +40,17 @@ module Assert::Assertions
     include Assert::Test::TestHelpers
 
     desc "`assert_not_empty`"
-    subject {
+    subject do
       args = args1
       Factory.test do
         assert_not_empty([1])   # pass
         assert_not_empty(*args) # fail
       end
-    }
+    end
 
-    let(:desc1) { "assert not empty fail desc" }
-    let(:args1) { [[], desc1] }
-    let(:config1) { subject.config }
+    let(:desc1){ "assert not empty fail desc" }
+    let(:args1){ [[], desc1] }
+    let(:config1){ subject.config }
 
     should "produce results as expected" do
       subject.run(&test_run_callback)
@@ -57,7 +60,8 @@ module Assert::Assertions
       assert_that(test_run_result_count(:fail)).equals(1)
 
       exp =
-        "#{args1[1]}\nExpected #{Assert::U.show(args1[0], config1)} to not be empty."
+        "#{args1[1]}\nExpected #{Assert::U.show(args1[0], config1)} to not "\
+        "be empty."
       assert_that(test_run_results(:fail).first.message).equals(exp)
     end
   end

data/test/unit/assertions/assert_equal_tests.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "assert"
 require "assert/assertions"
 
@@ -8,17 +10,17 @@ module Assert::Assertions
     include Assert::Test::TestHelpers
 
     desc "`assert_equal`"
-    subject {
+    subject do
       args = args1
       Factory.test do
         assert_equal(1, 1)  # pass
         assert_equal(*args) # fail
       end
-    }
+    end
 
-    let(:desc1) { "assert equal fail desc" }
-    let(:args1) { ["1", "2", desc1] }
-    let(:config1) { subject.config }
+    let(:desc1){ "assert equal fail desc" }
+    let(:args1){ ["1", "2", desc1] }
+    let(:config1){ subject.config }
 
     should "produce results as expected" do
       subject.run(&test_run_callback)
@@ -38,17 +40,17 @@ module Assert::Assertions
     include Assert::Test::TestHelpers
 
     desc "`assert_not_equal`"
-    subject {
+    subject do
       args = args1
       Factory.test do
         assert_not_equal(*args) # fail
         assert_not_equal(1, 2)  # pass
       end
-    }
+    end
 
-    let(:desc1) { "assert not equal fail desc" }
-    let(:args1) { ["1", "1", desc1] }
-    let(:config1) { subject.config }
+    let(:desc1){ "assert not equal fail desc" }
+    let(:args1){ ["1", "1", desc1] }
+    let(:config1){ subject.config }
 
     should "produce results as expected" do
       subject.run(&test_run_callback)
@@ -69,7 +71,7 @@ module Assert::Assertions
 
     desc "with objects that define custom equality operators"
 
-    let(:is_class)     {
+    let(:is_class) do
       Class.new do
         def ==(other)
           if other.is_a?(Assert::ActualValue.not_given.class)
@@ -79,8 +81,8 @@ module Assert::Assertions
           end
         end
       end
-    }
-    let(:is_not_class) {
+    end
+    let(:is_not_class) do
       Class.new do
         def ==(other)
           if other.is_a?(Assert::ActualValue.not_given.class)
@@ -90,10 +92,10 @@ module Assert::Assertions
           end
         end
       end
-    }
+    end
 
-    let(:is1) { is_class.new }
-    let(:is_not1) { is_not_class.new }
+    let(:is1){ is_class.new }
+    let(:is_not1){ is_not_class.new }
 
     should "use the equality operator of the exp value" do
       assert_that(is1).equals(is_not1)
@@ -106,27 +108,27 @@ module Assert::Assertions
 
     desc "with objects that should use diff when showing"
 
-    let(:config1) {
+    let(:config1) do
       Factory.modes_off_config.tap do |config|
         config.use_diff_proc(Assert::U.default_use_diff_proc)
         config.run_diff_proc(Assert::U.syscmd_diff_proc)
       end
-    }
+    end
 
-    let(:exp_obj1) { "I'm a\nstring" }
-    let(:act_obj1) { "I am a \nstring" }
-    let(:exp_obj_show1) { Assert::U.show_for_diff(exp_obj1, config1) }
-    let(:act_obj_show1) { Assert::U.show_for_diff(act_obj1, config1) }
+    let(:exp_obj1){ "I'm a\nstring" }
+    let(:act_obj1){ "I am a \nstring" }
+    let(:exp_obj_show1){ Assert::U.show_for_diff(exp_obj1, config1) }
+    let(:act_obj_show1){ Assert::U.show_for_diff(act_obj1, config1) }
   end
 
   class AssertEqualDiffTests < DiffTests
     desc "`assert_equal`"
-    subject {
+    subject do
       exp_obj, act_obj = exp_obj1, act_obj1
       Factory.test(config1) do
         assert_equal(exp_obj, act_obj)
       end
-    }
+    end
 
     should "include diff output in the fail messages" do
       subject.run(&test_run_callback)
@@ -140,12 +142,12 @@ module Assert::Assertions
 
   class AssertNotEqualDiffTests < DiffTests
     desc "`assert_not_equal`"
-    subject {
+    subject do
       exp_obj = exp_obj1
       Factory.test(config1) do
         assert_not_equal(exp_obj, exp_obj)
       end
-    }
+    end
 
     should "include diff output in the fail messages" do
       subject.run(&test_run_callback)