rubocop-minitest 0.10.2 → 0.12.1

data/docs/antora.yml

@@ -2,6 +2,6 @@ name: rubocop-minitest
 title: RuboCop Minitest
 # We always provide version without patch here (e.g. 1.1),
 # as patch versions should not appear in the docs.
-version: '0.10'
+version: '0.12'
 nav:
 - modules/ROOT/nav.adoc

data/docs/modules/ROOT/pages/cops.adoc

@@ -27,10 +27,12 @@ based on the https://minitest.rubystyle.guide/[Minitest Style Guide].
 * xref:cops_minitest.adoc#minitestassertrespondto[Minitest/AssertRespondTo]
 * xref:cops_minitest.adoc#minitestassertsilent[Minitest/AssertSilent]
 * xref:cops_minitest.adoc#minitestasserttruthy[Minitest/AssertTruthy]
+* xref:cops_minitest.adoc#minitestassertwithexpectedargument[Minitest/AssertWithExpectedArgument]
 * xref:cops_minitest.adoc#minitestassertioninlifecyclehook[Minitest/AssertionInLifecycleHook]
 * xref:cops_minitest.adoc#minitestglobalexpectations[Minitest/GlobalExpectations]
 * xref:cops_minitest.adoc#minitestliteralasactualargument[Minitest/LiteralAsActualArgument]
 * xref:cops_minitest.adoc#minitestmultipleassertions[Minitest/MultipleAssertions]
+* xref:cops_minitest.adoc#minitestnoassertions[Minitest/NoAssertions]
 * xref:cops_minitest.adoc#minitestrefuteempty[Minitest/RefuteEmpty]
 * xref:cops_minitest.adoc#minitestrefuteequal[Minitest/RefuteEqual]
 * xref:cops_minitest.adoc#minitestrefutefalse[Minitest/RefuteFalse]

data/docs/modules/ROOT/pages/cops_minitest.adoc

@@ -41,19 +41,19 @@ assert_empty(object, 'message')
 | Yes
 | Yes (Unsafe)
 | 0.5
-| 0.10
+| 0.11
 |===
 
 This cop enforces the test to use `assert_empty`
-instead of using `assert([], object)`.
+instead of using `assert_equal([], object)`.
 
 === Examples
 
 [source,ruby]
 ----
 # bad
-assert([], object)
-assert({}, object)
+assert_equal([], object)
+assert_equal({}, object)
 
 # good
 assert_empty(object)
@@ -215,7 +215,7 @@ assert_kind_of(Class, object, 'message')
 
 === References
 
-* https://github.com/rubocop-hq/minitest-style-guide#assert-kind-of
+* https://github.com/rubocop/minitest-style-guide#assert-kind-of
 
 == Minitest/AssertMatch
 
@@ -407,7 +407,7 @@ assert_silent { puts object.do_something }
 
 === References
 
-* https://github.com/rubocop-hq/minitest-style-guide#assert-silent
+* https://github.com/rubocop/minitest-style-guide#assert-silent
 
 == Minitest/AssertTruthy
 
@@ -441,6 +441,38 @@ assert(actual, 'message')
 
 * https://minitest.rubystyle.guide#assert-truthy
 
+== Minitest/AssertWithExpectedArgument
+
+|===
+| Enabled by default | Safe | Supports autocorrection | VersionAdded | VersionChanged
+
+| Pending
+| No
+| No
+| 0.11
+| -
+|===
+
+This cop tries to detect when a user accidentally used
+`assert` when they meant to use `assert_equal`.
+
+It is marked as unsafe because it is not possible to determine
+whether the second argument of `assert` is a message or not.
+
+=== Examples
+
+[source,ruby]
+----
+# bad
+assert(3, my_list.length)
+assert(expected, actual)
+
+# good
+assert_equal(3, my_list.length)
+assert_equal(expected, actual)
+assert(foo, 'message')
+----
+
 == Minitest/AssertionInLifecycleHook
 
 |===
@@ -593,6 +625,38 @@ end
 | Integer
 |===
 
+== Minitest/NoAssertions
+
+|===
+| Enabled by default | Safe | Supports autocorrection | VersionAdded | VersionChanged
+
+| Disabled
+| Yes
+| No
+| 0.12
+| -
+|===
+
+This cop checks if test cases contain any assertion calls.
+
+=== Examples
+
+[source,ruby]
+----
+# bad
+class FooTest < Minitest::Test
+  def test_the_truth
+  end
+end
+
+# good
+class FooTest < Minitest::Test
+  def test_the_truth
+    assert true
+  end
+end
+----
+
 == Minitest/RefuteEmpty
 
 |===
@@ -817,7 +881,7 @@ refute_kind_of(Class, object, 'message')
 
 === References
 
-* https://github.com/rubocop-hq/minitest-style-guide#refute-kind-of
+* https://github.com/rubocop/minitest-style-guide#refute-kind-of
 
 == Minitest/RefuteMatch
 
@@ -962,6 +1026,7 @@ refute_respond_to(self, :do_something)
 |===
 
 This cop enforces that test method names start with `test_` prefix.
+It aims to prevent tests that aren't executed by forgetting to start test method name with `test_`.
 
 === Examples
 
@@ -980,6 +1045,12 @@ class FooTest < Minitest::Test
     assert_equal 42, do_something
   end
 end
+
+# good
+class FooTest < Minitest::Test
+  def helper_method(argument)
+  end
+end
 ----
 
 == Minitest/UnspecifiedException

data/docs/modules/ROOT/pages/index.adoc

@@ -1,5 +1,5 @@
 = RuboCop Minitest
 
-A https://github.com/rubocop-hq/rubocop[RuboCop] extension focused on enforcing Minitest best practices and coding conventions.
+A https://github.com/rubocop/rubocop[RuboCop] extension focused on enforcing Minitest best practices and coding conventions.
 
 It's based on the community-driven https://minitest.rubystyle.guide[Minitest style guide].

data/legacy-docs/cops_minitest.md

@@ -23,7 +23,7 @@ assert_empty(object, 'message')
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#assert-empty](https://github.com/rubocop-hq/minitest-style-guide#assert-empty)
+* [https://github.com/rubocop/minitest-style-guide#assert-empty](https://github.com/rubocop/minitest-style-guide#assert-empty)
 
 ## Minitest/AssertEmptyLiteral
 
@@ -66,7 +66,7 @@ assert_equal("rubocop-minitest", actual)
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#assert-equal-arguments-order](https://github.com/rubocop-hq/minitest-style-guide#assert-equal-arguments-order)
+* [https://github.com/rubocop/minitest-style-guide#assert-equal-arguments-order](https://github.com/rubocop/minitest-style-guide#assert-equal-arguments-order)
 
 ## Minitest/AssertIncludes
 
@@ -91,7 +91,7 @@ assert_includes(collection, object, 'message')
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#assert-includes](https://github.com/rubocop-hq/minitest-style-guide#assert-includes)
+* [https://github.com/rubocop/minitest-style-guide#assert-includes](https://github.com/rubocop/minitest-style-guide#assert-includes)
 
 ## Minitest/AssertInstanceOf
 
@@ -116,7 +116,7 @@ assert_instance_of(Class, object, 'message')
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#assert-instance-of](https://github.com/rubocop-hq/minitest-style-guide#assert-instance-of)
+* [https://github.com/rubocop/minitest-style-guide#assert-instance-of](https://github.com/rubocop/minitest-style-guide#assert-instance-of)
 
 ## Minitest/AssertMatch
 
@@ -141,7 +141,7 @@ assert_match(matcher, string, 'message')
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#assert-match](https://github.com/rubocop-hq/minitest-style-guide#assert-match)
+* [https://github.com/rubocop/minitest-style-guide#assert-match](https://github.com/rubocop/minitest-style-guide#assert-match)
 
 ## Minitest/AssertNil
 
@@ -166,7 +166,7 @@ assert_nil(actual, 'message')
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#assert-nil](https://github.com/rubocop-hq/minitest-style-guide#assert-nil)
+* [https://github.com/rubocop/minitest-style-guide#assert-nil](https://github.com/rubocop/minitest-style-guide#assert-nil)
 
 ## Minitest/AssertRespondTo
 
@@ -193,7 +193,7 @@ assert_respond_to(self, :do_something)
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#assert-responds-to-method](https://github.com/rubocop-hq/minitest-style-guide#assert-responds-to-method)
+* [https://github.com/rubocop/minitest-style-guide#assert-responds-to-method](https://github.com/rubocop/minitest-style-guide#assert-responds-to-method)
 
 ## Minitest/AssertTruthy
 
@@ -218,7 +218,7 @@ assert(actual, 'message')
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#assert-truthy](https://github.com/rubocop-hq/minitest-style-guide#assert-truthy)
+* [https://github.com/rubocop/minitest-style-guide#assert-truthy](https://github.com/rubocop/minitest-style-guide#assert-truthy)
 
 ## Minitest/GlobalExpectations
 
@@ -266,7 +266,7 @@ refute_empty(object, 'message')
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#refute-empty](https://github.com/rubocop-hq/minitest-style-guide#refute-empty)
+* [https://github.com/rubocop/minitest-style-guide#refute-empty](https://github.com/rubocop/minitest-style-guide#refute-empty)
 
 ## Minitest/RefuteEqual
 
@@ -290,7 +290,7 @@ refute_equal("rubocop-minitest", actual)
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#refute-equal](https://github.com/rubocop-hq/minitest-style-guide#refute-equal)
+* [https://github.com/rubocop/minitest-style-guide#refute-equal](https://github.com/rubocop/minitest-style-guide#refute-equal)
 
 ## Minitest/RefuteFalse
 
@@ -318,7 +318,7 @@ refute(actual, 'message')
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#refute-false](https://github.com/rubocop-hq/minitest-style-guide#refute-false)
+* [https://github.com/rubocop/minitest-style-guide#refute-false](https://github.com/rubocop/minitest-style-guide#refute-false)
 
 ## Minitest/RefuteIncludes
 
@@ -343,7 +343,7 @@ refute_includes(collection, object, 'message')
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#refute-includes](https://github.com/rubocop-hq/minitest-style-guide#refute-includes)
+* [https://github.com/rubocop/minitest-style-guide#refute-includes](https://github.com/rubocop/minitest-style-guide#refute-includes)
 
 ## Minitest/RefuteInstanceOf
 
@@ -368,7 +368,7 @@ refute_instance_of(Class, object, 'message')
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#refute-instance-of](https://github.com/rubocop-hq/minitest-style-guide#refute-instance-of)
+* [https://github.com/rubocop/minitest-style-guide#refute-instance-of](https://github.com/rubocop/minitest-style-guide#refute-instance-of)
 
 ## Minitest/RefuteMatch
 
@@ -393,7 +393,7 @@ refute_match(matcher, string, 'message')
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#refute-match](https://github.com/rubocop-hq/minitest-style-guide#refute-match)
+* [https://github.com/rubocop/minitest-style-guide#refute-match](https://github.com/rubocop/minitest-style-guide#refute-match)
 
 ## Minitest/RefuteNil
 
@@ -418,7 +418,7 @@ refute_nil(actual, 'message')
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#refute-nil](https://github.com/rubocop-hq/minitest-style-guide#refute-nil)
+* [https://github.com/rubocop/minitest-style-guide#refute-nil](https://github.com/rubocop/minitest-style-guide#refute-nil)
 
 ## Minitest/RefuteRespondTo
 
@@ -445,4 +445,4 @@ refute_respond_to(self, :do_something)
 
 ### References
 
-* [https://github.com/rubocop-hq/minitest-style-guide#refute-respond-to](https://github.com/rubocop-hq/minitest-style-guide#refute-respond-to)
+* [https://github.com/rubocop/minitest-style-guide#refute-respond-to](https://github.com/rubocop/minitest-style-guide#refute-respond-to)

data/legacy-docs/index.md

@@ -1 +1 @@
-A [RuboCop](https://github.com/rubocop-hq/rubocop) extension focused on enforcing Minitest best practices and coding conventions.
+A [RuboCop](https://github.com/rubocop/rubocop) extension focused on enforcing Minitest best practices and coding conventions.

data/lib/rubocop/cop/minitest/assert_empty.rb

@@ -15,7 +15,7 @@ module RuboCop
       #   assert_empty(object)
       #   assert_empty(object, 'message')
       #
-      class AssertEmpty < Cop
+      class AssertEmpty < Base
         extend MinitestCopRule
 
         define_rule :assert, target_method: :empty?

data/lib/rubocop/cop/minitest/assert_empty_literal.rb

@@ -4,42 +4,38 @@ module RuboCop
   module Cop
     module Minitest
       # This cop enforces the test to use `assert_empty`
-      # instead of using `assert([], object)`.
+      # instead of using `assert_equal([], object)`.
       #
       # @example
       #   # bad
-      #   assert([], object)
-      #   assert({}, object)
+      #   assert_equal([], object)
+      #   assert_equal({}, object)
       #
       #   # good
       #   assert_empty(object)
       #
-      class AssertEmptyLiteral < Cop
+      class AssertEmptyLiteral < Base
         include ArgumentRangeHelper
+        extend AutoCorrector
 
         MSG = 'Prefer using `assert_empty(%<arguments>s)` over ' \
-              '`assert(%<literal>s, %<arguments>s)`.'
+              '`assert_equal(%<literal>s, %<arguments>s)`.'
+        RESTRICT_ON_SEND = %i[assert_equal].freeze
 
-        def_node_matcher :assert_with_empty_literal, <<~PATTERN
-          (send nil? :assert ${hash array} $...)
+        def_node_matcher :assert_equal_with_empty_literal, <<~PATTERN
+          (send nil? :assert_equal ${hash array} $...)
         PATTERN
 
         def on_send(node)
-          assert_with_empty_literal(node) do |literal, matchers|
+          assert_equal_with_empty_literal(node) do |literal, matchers|
             return unless literal.values.empty?
 
             args = matchers.map(&:source).join(', ')
 
             message = format(MSG, literal: literal.source, arguments: args)
-            add_offense(node, message: message)
-          end
-        end
-
-        def autocorrect(node)
-          assert_with_empty_literal(node) do |_literal, matchers|
-            object = matchers.first
+            add_offense(node, message: message) do |corrector|
+              object = matchers.first
 
-            lambda do |corrector|
               corrector.replace(node.loc.selector, 'assert_empty')
               corrector.replace(first_and_second_arguments_range(node), object.source)
             end

data/lib/rubocop/cop/minitest/assert_equal.rb

@@ -13,7 +13,7 @@ module RuboCop
       #   # good
       #   assert_equal("rubocop-minitest", actual)
       #
-      class AssertEqual < Cop
+      class AssertEqual < Base
         extend MinitestCopRule
 
         define_rule :assert, target_method: :==, preferred_method: :assert_equal

data/lib/rubocop/cop/minitest/assert_in_delta.rb

@@ -15,8 +15,11 @@ module RuboCop
       #   assert_in_delta(0.2, actual)
       #   assert_in_delta(0.2, actual, 0.001, 'message')
       #
-      class AssertInDelta < Cop
+      class AssertInDelta < Base
         include InDeltaMixin
+        extend AutoCorrector
+
+        RESTRICT_ON_SEND = %i[assert_equal].freeze
 
         def_node_matcher :equal_floats_call, <<~PATTERN
           (send nil? :assert_equal $_ $_ $...)

data/lib/rubocop/cop/minitest/assert_includes.rb

@@ -15,7 +15,7 @@ module RuboCop
       #   assert_includes(collection, object)
       #   assert_includes(collection, object, 'message')
       #
-      class AssertIncludes < Cop
+      class AssertIncludes < Base
         extend MinitestCopRule
 
         define_rule :assert, target_method: :include?, preferred_method: :assert_includes

data/lib/rubocop/cop/minitest/assert_instance_of.rb

@@ -15,7 +15,7 @@ module RuboCop
       #   assert_instance_of(Class, object)
       #   assert_instance_of(Class, object, 'message')
       #
-      class AssertInstanceOf < Cop
+      class AssertInstanceOf < Base
         extend MinitestCopRule
 
         define_rule :assert, target_method: :instance_of?, inverse: true

data/lib/rubocop/cop/minitest/assert_kind_of.rb

@@ -15,7 +15,7 @@ module RuboCop
       #   assert_kind_of(Class, object)
       #   assert_kind_of(Class, object, 'message')
       #
-      class AssertKindOf < Cop
+      class AssertKindOf < Base
         extend MinitestCopRule
 
         define_rule :assert, target_method: :kind_of?, inverse: true

data/lib/rubocop/cop/minitest/assert_match.rb

@@ -15,7 +15,7 @@ module RuboCop
       #   assert_match(regex, string)
       #   assert_match(matcher, string, 'message')
       #
-      class AssertMatch < Cop
+      class AssertMatch < Base
         extend MinitestCopRule
 
         define_rule :assert, target_method: :match

data/lib/rubocop/cop/minitest/assert_nil.rb

@@ -15,11 +15,13 @@ module RuboCop
       #   assert_nil(actual)
       #   assert_nil(actual, 'message')
       #
-      class AssertNil < Cop
+      class AssertNil < Base
         include ArgumentRangeHelper
+        extend AutoCorrector
 
         MSG = 'Prefer using `assert_nil(%<arguments>s)` over ' \
               '`assert_equal(nil, %<arguments>s)`.'
+        RESTRICT_ON_SEND = %i[assert_equal].freeze
 
         def_node_matcher :assert_equal_with_nil, <<~PATTERN
           (send nil? :assert_equal nil $_ $...)
@@ -31,17 +33,9 @@ module RuboCop
 
             arguments = [actual.source, message&.source].compact.join(', ')
 
-            add_offense(node, message: format(MSG, arguments: arguments))
-          end
-        end
-
-        def autocorrect(node)
-          lambda do |corrector|
-            assert_equal_with_nil(node) do |actual|
+            add_offense(node, message: format(MSG, arguments: arguments)) do |corrector|
               corrector.replace(node.loc.selector, 'assert_nil')
-              corrector.replace(
-                first_and_second_arguments_range(node), actual.source
-              )
+              corrector.replace(first_and_second_arguments_range(node), actual.source)
             end
           end
         end