assert 2.18.4 → 2.19.4

data/test/unit/context_info_tests.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "assert"
 require "assert/context_info"
 
@@ -6,20 +8,20 @@ require "assert/context"
 class Assert::ContextInfo
   class UnitTests < Assert::Context
     desc "Assert::ContextInfo"
-    subject { unit_class }
+    subject{ unit_class }
 
-    let(:unit_class) { Assert::ContextInfo }
+    let(:unit_class){ Assert::ContextInfo }
   end
 
   class InitTests < UnitTests
     desc "when init"
-    subject { unit_class.new(context1, nil, @caller.first) }
+    subject{ unit_class.new(context1, nil, @caller.first) }
 
     setup do
       @caller = caller
     end
 
-    let(:context1) { Assert::Context }
+    let(:context1){ Assert::Context }
 
     should have_readers :called_from, :klass, :file
     should have_imeths :test_name

data/test/unit/context_tests.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "assert"
 require "assert/context"
 
@@ -8,9 +10,9 @@ require "assert/utils"
 class Assert::Context
   class UnitTests < Assert::Context
     desc "Assert::Context"
-    subject { unit_class }
+    subject{ unit_class }
 
-    let(:unit_class) { Assert::Context }
+    let(:unit_class){ Assert::Context }
 
     # DSL methods
     should have_imeths :description, :desc, :describe, :subject, :suite, :let
@@ -25,20 +27,23 @@ class Assert::Context
 
   class InitTests < UnitTests
     desc "when init"
-    subject { context_class1.new(test1, test1.config, result_callback1) }
+    subject{ context_class1.new(test1, test1.config, result_callback1) }
 
     setup do
       @callback_result = nil
     end
 
-    let(:test1)            { Factory.test }
-    let(:context_class1)   { test1.context_class }
-    let(:test_results1)    { [] }
-    let(:result_callback1) {
-      proc { |result| @callback_result = result; test_results1 << result }
-    }
-    let(:halt_config1) { Assert::Config.new(:halt_on_fail => true) }
-    let(:msg1) { Factory.string }
+    let(:test1){ Factory.test }
+    let(:context_class1){ test1.context_class }
+    let(:test_results1){ [] }
+    let(:result_callback1) do
+      proc do |result|
+        @callback_result = result
+        test_results1 << result
+      end
+    end
+    let(:halt_config1){ Assert::Config.new(halt_on_fail: true) }
+    let(:msg1){ Factory.string }
 
     should have_imeths :assert, :assert_not, :refute, :assert_that
     should have_imeths :pass, :ignore, :fail, :flunk, :skip
@@ -46,7 +51,8 @@ class Assert::Context
 
     should "collect context info" do
       test = @__assert_running_test__
-      assert_that(test.context_info.file).matches(/test\/unit\/context_tests.rb$/)
+      assert_that(test.context_info.file)
+        .matches(%r{test/unit/context_tests.rb$})
       assert_that(test.context_info.klass).equals(self.class)
     end
 
@@ -58,7 +64,9 @@ class Assert::Context
       with_backtrace(caller) do
         assert_that(result.with_bt_set?).is_true
 
-        exp = Assert::Result::Backtrace.to_s(exp_with_bt+[(result.backtrace.filtered.first)])
+        exp =
+          Assert::Result::Backtrace.to_s(exp_with_bt +
+          [(result.backtrace.filtered.first)])
         assert_that(result.trace).equals(exp)
         assert_that(result.src_line).equals(exp_with_bt.first)
       end
@@ -69,7 +77,8 @@ class Assert::Context
         assert_that(result.with_bt_set?).is_false
 
         assert_that(result.trace).equals(result.src_line)
-        assert_that(result.src_line).equals(result.backtrace.filtered.first.to_s)
+        assert_that(result.src_line)
+          .equals(result.backtrace.filtered.first.to_s)
       end
     end
   end
@@ -78,7 +87,12 @@ class Assert::Context
     desc "skip method"
 
     setup do
-      begin; subject.skip(msg1); rescue StandardError => @exception; end
+      @exception =
+        begin
+          subject.skip(msg1)
+        rescue => ex
+          ex
+        end
       @result = Factory.skip_result(@exception)
     end
 
@@ -96,7 +110,12 @@ class Assert::Context
       assert_that(@exception.backtrace.size).does_not_equal(1)
 
       called_from = Factory.string
-      begin; subject.skip(msg1, called_from); rescue StandardError => exception; end
+      exception =
+        begin
+          subject.skip(msg1, called_from)
+        rescue => ex
+          ex
+        end
       assert_that(exception.backtrace.size).equals(1)
       assert_that(exception.backtrace.first).equals(called_from)
     end
@@ -178,14 +197,20 @@ class Assert::Context
 
   class HaltOnFailTests < InitTests
     desc "failing when halting on fails"
-    subject { context_class1.new(test1, halt_config1, result_callback1) }
+    subject{ context_class1.new(test1, halt_config1, result_callback1) }
 
     should "raise an exception with the failure's message" do
-      begin; subject.fail(msg1); rescue StandardError => err; end
-      assert_that(err).is_kind_of(Assert::Result::TestFailure)
-      assert_that(err.message).equals(msg1)
-
-      result = Assert::Result::Fail.for_test(Factory.test("something"), err)
+      exception =
+        begin
+          subject.fail(msg1)
+        rescue => ex
+          ex
+        end
+      assert_that(exception).is_kind_of(Assert::Result::TestFailure)
+      assert_that(exception.message).equals(msg1)
+
+      result =
+        Assert::Result::Fail.for_test(Factory.test("something"), exception)
       assert_that(result.message).equals(msg1)
     end
 
@@ -197,7 +222,7 @@ class Assert::Context
   class AssertTests < InitTests
     desc "assert method"
 
-    let(:what_failed) { Factory.string }
+    let(:what_failed){ Factory.string }
 
     should "return a pass result given a `true` assertion" do
       result = subject.assert(true, msg1){ what_failed }
@@ -211,7 +236,8 @@ class Assert::Context
     end
 
     should "pp the assertion value in the fail message by default" do
-      exp_def_what = "Failed assert: assertion was `#{Assert::U.show(false, test1.config)}`."
+      exp_def_what =
+        "Failed assert: assertion was `#{Assert::U.show(false, test1.config)}`."
       result = subject.assert(false, msg1)
 
       assert_that(result.message).equals([msg1, exp_def_what].join("\n"))
@@ -246,7 +272,9 @@ class Assert::Context
     end
 
     should "pp the assertion value in the fail message by default" do
-      exp_def_what = "Failed assert_not: assertion was `#{Assert::U.show(true, test1.config)}`."
+      exp_def_what =
+        "Failed assert_not: "\
+        "assertion was `#{Assert::U.show(true, test1.config)}`."
       result = subject.assert_not(true, msg1)
 
       assert_that(result.message).equals([msg1, exp_def_what].join("\n"))
@@ -265,32 +293,32 @@ class Assert::Context
     desc "`assert_that` method"
 
     setup do
-      Assert.stub_tap_on_call(Assert::ActualValue, :new) { |_, call|
+      Assert.stub_tap_on_call(Assert::ActualValue, :new) do |_, call|
         @actual_value_new_call = call
-      }
+      end
     end
 
-    let(:actual_value) { Factory.string }
+    let(:actual_value){ Factory.string }
 
     should "build an Assert::ActualValue" do
       assert_instance_of Assert::ActualValue, subject.assert_that(actual_value)
       assert_equal [actual_value], @actual_value_new_call.pargs
-      assert_equal({ context: subject },  @actual_value_new_call.kargs)
+      assert_equal({ context: subject }, @actual_value_new_call.kargs)
     end
   end
 
   class SubjectTests < InitTests
     desc "subject method"
-    subject { context_class1.new(test1, test1.config, proc{ |result| }) }
+    subject{ context_class1.new(test1, test1.config, proc{ |result| }) }
 
     setup do
       expected = expected1
-      context_class1.subject { @something = expected }
+      context_class1.subject{ @something = expected }
       @subject = subject.subject
     end
 
-    let(:context_class1) { Factory.modes_off_context_class }
-    let(:expected1) { Factory.string }
+    let(:context_class1){ Factory.modes_off_context_class }
+    let(:expected1){ Factory.string }
 
     should "instance evaluate the block set with the class setup method" do
       assert_that(@subject).equals(expected1)
@@ -300,8 +328,17 @@ class Assert::Context
   class PendingTests < InitTests
     desc "`pending` method"
 
-    let(:block2) { proc { fail; pass; } }
-    let(:block1) { block = block2; proc { pending(&block) } } # test nesting
+    let(:block2) do
+      proc do
+        fail # rubocop:disable Style/SignalException
+        pass # rubocop:disable Lint/UnreachableCode
+      end
+    end
+    # test nesting
+    let(:block1) do
+      block = block2
+      proc{ pending(&block) }
+    end
 
     should "make fails skips and make passes fails" do
       subject.fail "not affected"
@@ -324,29 +361,42 @@ class Assert::Context
 
   class PendingWithHaltOnFailTests < PendingTests
     desc "when halting on fails"
-    subject { context_class1.new(test1, halt_config1, result_callback1) }
+    subject{ context_class1.new(test1, halt_config1, result_callback1) }
 
     should "make fails skips and stop the test" do
-      begin; subject.pending(&block1); rescue StandardError => err; end
-      assert_that(err).is_kind_of(Assert::Result::TestSkipped)
-      assert_that(err.message).includes("Pending fail")
-
-      assert_that(test_results1.size).equals(0) # it halted before the pending pass
+      exception =
+        begin
+          subject.pending(&block1)
+        rescue => ex
+          ex
+        end
+      assert_that(exception).is_kind_of(Assert::Result::TestSkipped)
+      assert_that(exception.message).includes("Pending fail")
+
+      # it halted before the pending pass
+      assert_that(test_results1.size).equals(0)
     end
   end
 
   class WithBacktraceTests < InitTests
     desc "`with_backtrace` method"
 
-    let(:from_bt1)    { ["called_from_here", Factory.string] }
-    let(:from_block1) { proc { ignore; fail; pass; skip "todo"; } }
+    let(:from_bt1){ ["called_from_here", Factory.string] }
+    let(:from_block1) do
+      proc do
+        ignore
+        fail # rubocop:disable Style/SignalException
+        pass # rubocop:disable Lint/UnreachableCode
+        skip "todo"
+      end
+    end
 
     should "alter non-error block results' bt with given bt's first line" do
       subject.fail "not affected"
       begin
         subject.with_backtrace(from_bt1, &from_block1)
-      rescue Assert::Result::TestSkipped => e
-        test_results1 << Assert::Result::Skip.for_test(test1, e)
+      rescue Assert::Result::TestSkipped => ex
+        test_results1 << Assert::Result::Skip.for_test(test1, ex)
       end
 
       assert_that(test_results1.size).equals(5)
@@ -365,21 +415,29 @@ class Assert::Context
   class WithNestedBacktraceTests < InitTests
     desc "`with_backtrace` method nested"
 
-    let(:from_bt1)    { ["called_from_here 1", Factory.string] }
-    let(:from_bt2)    { ["called_from_here 2", Factory.string] }
-    let(:from_block2) { proc { ignore; fail; pass; skip "todo"; } }
-    let(:from_block1) {
+    let(:from_bt1){ ["called_from_here 1", Factory.string] }
+    let(:from_bt2){ ["called_from_here 2", Factory.string] }
+    let(:from_block2) do
+      proc do
+        ignore
+        fail # rubocop:disable Style/SignalException
+        pass # rubocop:disable Lint/UnreachableCode
+        skip "todo"
+      end
+    end
+    let(:from_block1) do
       from_bt    = from_bt2
       from_block = from_block2
-      proc { with_backtrace(from_bt, &from_block) }
-    }
+      proc{ with_backtrace(from_bt, &from_block) }
+    end
 
-    should "alter non-error block results' bt with nested wbt accrued first lines" do
+    should "alter non-error block results' bt with nested wbt accrued "\
+           "first lines" do
       subject.fail "not affected"
       begin
         subject.with_backtrace(from_bt1, &from_block1)
-      rescue Assert::Result::TestSkipped => e
-        test_results1 << Assert::Result::Skip.for_test(test1, e)
+      rescue Assert::Result::TestSkipped => ex
+        test_results1 << Assert::Result::Skip.for_test(test1, ex)
       end
 
       assert_that(test_results1.size).equals(5)

data/test/unit/default_runner_tests.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "assert"
 require "assert/default_runner"
 

data/test/unit/default_suite_tests.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "assert"
 require "assert/default_suite"
 
@@ -6,18 +8,22 @@ require "assert/suite"
 class Assert::DefaultSuite
   class UnitTests < Assert::Context
     desc "Assert::DefaultSuite"
-    subject { unit_class }
+    subject{ unit_class }
 
-    let(:unit_class) { Assert::DefaultSuite }
+    let(:unit_class){ Assert::DefaultSuite }
   end
 
   class InitTests < UnitTests
     desc "when init"
-    subject { unit_class.new(config1) }
+    subject{ unit_class.new(config1) }
+
+    let(:ci1){ Factory.context_info(Factory.modes_off_context_class) }
+    let(:test1){ Factory.test(Factory.string, ci1){} }
+    let(:config1){ Factory.modes_off_config }
 
-    let(:ci1)     { Factory.context_info(Factory.modes_off_context_class) }
-    let(:test1)   { Factory.test(Factory.string, ci1) { } }
-    let(:config1) { Factory.modes_off_config }
+    should have_readers :test_count, :result_count, :pass_result_count
+    should have_readers :fail_result_count, :error_result_count
+    should have_readers :skip_result_count, :ignore_result_count
 
     should "be a Suite" do
       assert_that(subject).is_kind_of(Assert::Suite)

data/test/unit/factory_tests.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "assert"
 require "assert/factory"
 
@@ -6,9 +8,9 @@ require "much-factory"
 module Assert::Factory
   class UnitTests < Assert::Context
     desc "Assert::Factory"
-    subject { unit_class }
+    subject{ unit_class }
 
-    let(:unit_class) { Assert::Factory }
+    let(:unit_class){ Assert::Factory }
 
     should "include and extend MuchFactory" do
       assert_that(subject).includes(MuchFactory)

data/test/unit/file_line_tests.rb

@@ -1,22 +1,24 @@
+# frozen_string_literal: true
+
 require "assert"
 require "assert/file_line"
 
 class Assert::FileLine
   class UnitTests < Assert::Context
     desc "Assert::FileLine"
-    subject { unit_class }
+    subject{ unit_class }
 
-    let(:unit_class) { Assert::FileLine }
+    let(:unit_class){ Assert::FileLine }
 
-    let(:file1) { "#{Factory.path}_tests.rb" }
-    let(:line1) { Factory.integer.to_s }
+    let(:file1){ "#{Factory.path}_tests.rb" }
+    let(:line1){ Factory.integer.to_s }
 
     should have_imeths :parse
 
     should "know how to parse and init from a file line path string" do
       file_line_path = [
         "#{file1}:#{line1}",
-        "#{file1}:#{line1} #{Factory.string}"
+        "#{file1}:#{line1} #{Factory.string}",
       ].sample
       file_line = subject.parse(file_line_path)
 
@@ -45,7 +47,7 @@ class Assert::FileLine
 
   class InitTests < UnitTests
     desc "when init"
-    subject { unit_class.new(file1, line1) }
+    subject{ unit_class.new(file1, line1) }
 
     should have_readers :file, :line
 
@@ -71,7 +73,7 @@ class Assert::FileLine
       no = unit_class.new("#{Factory.path}_tests.rb", Factory.integer.to_s)
 
       assert_that(subject).equals(yes)
-      assert_not_equal no,  subject
+      assert_not_equal no, subject
     end
   end
 end