rubocop-minitest 0.17.2 → 0.18.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7b59e5fd9ae4f6ad15bcc96b2e615eb8aa132b653cfe99c62eb8874186b88d38
-  data.tar.gz: 710f664f0704ffd649d4741729d6c6c5cab86222c8b79540c553160ff30a0d68
+  metadata.gz: add241b07281db4494e25119722057177e5eb15056360b0907d4b58819b6f371
+  data.tar.gz: 0dbe3916ea796caa3c59e04df1b13ade8a11c25b7613187308b3ea90348380fa
 SHA512:
-  metadata.gz: 727cc013e57b69f43d5796e37d62521ed7880dfc961ef59f5df0cd1a48d2a2bb8a7f6be5698efa0b9021f01097065f4d09ada983c0e36e3455ed3f36af94954b
-  data.tar.gz: f3b1f390ff8a682c5866871ad83887c7cd891eb307261bfeb46aee8d44ff1ed081945d7daa279035cebb2730c0d797ce0514758907a81b08a74a712766ec293d
+  metadata.gz: 4ab3dd3b82e55154731f600b942f2ef3dd48e85bf55d8d5d09c94bcb5227e409ea201c45be221cfde00cdb434d4877508bfad26cc7d3d3427c531c0babf146c1
+  data.tar.gz: 261d27488e8ee453f9307bb4367a7072dddb544c0dd73c5ac4e70e033875be7916519cf346ede3c5956023d81e5cf0f22b6ab284c85650a8987b0fa95c77f18d

data/.github/workflows/linting.yml

@@ -0,0 +1,15 @@
+name: Linting
+on:
+  - pull_request
+jobs:
+  yamllint:
+    name: Yamllint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - name: Yamllint
+        uses: karancode/yamllint-github-action@master
+        with:
+          yamllint_comment: true
+        env:
+          GITHUB_ACCESS_TOKEN: ${{ secrets.GITHUB_TOKEN }}

data/.rubocop.yml

@@ -55,13 +55,13 @@ Layout/ClassStructure:
       - prepend
       - extend
   ExpectedOrder:
-      - module_inclusion
-      - constants
-      - public_class_methods
-      - initializer
-      - instance_methods
-      - protected_methods
-      - private_methods
+    - module_inclusion
+    - constants
+    - public_class_methods
+    - initializer
+    - instance_methods
+    - protected_methods
+    - private_methods
 
 # Trailing white space is meaningful in code examples
 Layout/TrailingWhitespace:

data/.yamllint.yml

@@ -0,0 +1,9 @@
+extends: default
+
+rules:
+  comments:
+    min-spaces-from-content: 1
+  document-start: disable
+  line-length: disable
+  truthy:
+    check-keys: false

data/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## master (unreleased)
 
+## 0.18.0 (2022-03-13)
+
+### New features
+
+* [#161](https://github.com/rubocop/rubocop-minitest/pull/161): Add new `Minitest/AssertPredicate` and `Minitest/RefutePredicate` cops. ([@koic][])
+
+### Changes
+
+* [#162](https://github.com/rubocop/rubocop-minitest/pull/162): Make `Minitest/AssertNil` (`Minitest/RefuteNil`) aware of `assert_predicate(obj, :nil?)` (`refute_predicate(obj, :nil?)`). ([@koic][])
+
 ## 0.17.2 (2022-02-12)
 
 ### Bug fixes

data/config/default.yml

@@ -81,6 +81,12 @@ Minitest/AssertRespondTo:
   Enabled: true
   VersionAdded: '0.3'
 
+Minitest/AssertPredicate:
+  Description: 'This cop enforces the test to use `assert_predicate` instead of using `assert(obj.a_predicate_method?)`.'
+  StyleGuide: 'https://minitest.rubystyle.guide/#assert-predicate'
+  Enabled: pending
+  VersionAdded: '0.18'
+
 Minitest/AssertSilent:
   Description: "This cop enforces the test to use `assert_silent { ... }` instead of using `assert_output('', '') { ... }`."
   StyleGuide: 'https://github.com/rubocop/minitest-style-guide#assert-silent'
@@ -194,6 +200,12 @@ Minitest/RefutePathExists:
   Enabled: 'pending'
   VersionAdded: '0.10'
 
+Minitest/RefutePredicate:
+  Description: 'This cop enforces the test to use `refute_predicate` instead of using `refute(obj.a_predicate_method?)`.'
+  StyleGuide: 'https://minitest.rubystyle.guide/#refute-predicate'
+  Enabled: pending
+  VersionAdded: '0.18'
+
 Minitest/RefuteRespondTo:
   Description: 'This cop enforces the test to use `refute_respond_to(object, :do_something)` over `refute(object.respond_to?(:do_something))`.'
   StyleGuide: 'https://minitest.rubystyle.guide#refute-respond-to'

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.17'
+version: '0.18'
 nav:
-- modules/ROOT/nav.adoc
+  - modules/ROOT/nav.adoc

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

@@ -24,6 +24,7 @@ based on the https://minitest.rubystyle.guide/[Minitest Style Guide].
 * xref:cops_minitest.adoc#minitestassertnil[Minitest/AssertNil]
 * xref:cops_minitest.adoc#minitestassertoutput[Minitest/AssertOutput]
 * xref:cops_minitest.adoc#minitestassertpathexists[Minitest/AssertPathExists]
+* xref:cops_minitest.adoc#minitestassertpredicate[Minitest/AssertPredicate]
 * xref:cops_minitest.adoc#minitestassertrespondto[Minitest/AssertRespondTo]
 * xref:cops_minitest.adoc#minitestassertsilent[Minitest/AssertSilent]
 * xref:cops_minitest.adoc#minitestasserttruthy[Minitest/AssertTruthy]
@@ -43,6 +44,7 @@ based on the https://minitest.rubystyle.guide/[Minitest Style Guide].
 * xref:cops_minitest.adoc#minitestrefutematch[Minitest/RefuteMatch]
 * xref:cops_minitest.adoc#minitestrefutenil[Minitest/RefuteNil]
 * xref:cops_minitest.adoc#minitestrefutepathexists[Minitest/RefutePathExists]
+* xref:cops_minitest.adoc#minitestrefutepredicate[Minitest/RefutePredicate]
 * xref:cops_minitest.adoc#minitestrefuterespondto[Minitest/RefuteRespondTo]
 * xref:cops_minitest.adoc#minitesttestmethodname[Minitest/TestMethodName]
 * xref:cops_minitest.adoc#minitestunreachableassertion[Minitest/UnreachableAssertion]

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

@@ -262,7 +262,7 @@ assert_match(matcher, string, 'message')
 |===
 
 This cop enforces the test to use `assert_nil` instead of using
-`assert_equal(nil, something)` or `assert(something.nil?)`.
+`assert_equal(nil, something)`, `assert(something.nil?)`, or `assert_predicate(something, :nil?)`.
 
 === Examples
 
@@ -273,6 +273,8 @@ assert_equal(nil, actual)
 assert_equal(nil, actual, 'message')
 assert(object.nil?)
 assert(object.nil?, 'message')
+assert_predicate(object, :nil?)
+assert_predicate(object, :nil?, 'message')
 
 # good
 assert_nil(actual)
@@ -347,6 +349,38 @@ assert_path_exists(path, 'message')
 
 * https://minitest.rubystyle.guide/#assert-path-exists
 
+== Minitest/AssertPredicate
+
+|===
+| Enabled by default | Safe | Supports autocorrection | Version Added | Version Changed
+
+| Pending
+| Yes
+| Yes
+| 0.18
+| -
+|===
+
+This cop enforces the test to use `assert_predicate`
+instead of using `assert(obj.a_predicate_method?)`.
+
+=== Examples
+
+[source,ruby]
+----
+# bad
+assert(obj.one?)
+assert(obj.one?, 'message')
+
+# good
+assert_predicate(obj, :one?)
+assert_predicate(obj, :one?, 'message')
+----
+
+=== References
+
+* https://minitest.rubystyle.guide/#assert-predicate
+
 == Minitest/AssertRespondTo
 
 |===
@@ -1025,7 +1059,7 @@ refute_match(matcher, string, 'message')
 |===
 
 This cop enforces the test to use `refute_nil` instead of using
-`refute_equal(nil, something)` or `refute(something.nil?)`.
+`refute_equal(nil, something)`, `refute(something.nil?)`, or `refute_predicate(something, :nil?)`.
 
 === Examples
 
@@ -1036,6 +1070,8 @@ refute_equal(nil, actual)
 refute_equal(nil, actual, 'message')
 refute(actual.nil?)
 refute(actual.nil?, 'message')
+refute_predicate(object, :nil?)
+refute_predicate(object, :nil?, 'message')
 
 # good
 refute_nil(actual)
@@ -1078,6 +1114,38 @@ refute_path_exists(path, 'message')
 
 * https://minitest.rubystyle.guide/#refute-path-exists
 
+== Minitest/RefutePredicate
+
+|===
+| Enabled by default | Safe | Supports autocorrection | Version Added | Version Changed
+
+| Pending
+| Yes
+| Yes
+| 0.18
+| -
+|===
+
+This cop enforces the test to use `refute_predicate`
+instead of using `refute(obj.a_predicate_method?)`.
+
+=== Examples
+
+[source,ruby]
+----
+# bad
+refute(obj.one?)
+refute(obj.one?, 'message')
+
+# good
+refute_predicate(obj, :one?)
+refute_predicate(obj, :one?, 'message')
+----
+
+=== References
+
+* https://minitest.rubystyle.guide/#refute-predicate
+
 == Minitest/RefuteRespondTo
 
 |===

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

@@ -18,8 +18,7 @@ module RuboCop
         include ArgumentRangeHelper
         extend AutoCorrector
 
-        MSG = 'Prefer using `assert_empty(%<arguments>s)` over ' \
-              '`assert_equal(%<literal>s, %<arguments>s)`.'
+        MSG = 'Prefer using `assert_empty(%<arguments>s)`.'
         RESTRICT_ON_SEND = %i[assert_equal].freeze
 
         def_node_matcher :assert_equal_with_empty_literal, <<~PATTERN

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

@@ -4,7 +4,7 @@ module RuboCop
   module Cop
     module Minitest
       # This cop enforces the test to use `assert_nil` instead of using
-      # `assert_equal(nil, something)` or `assert(something.nil?)`.
+      # `assert_equal(nil, something)`, `assert(something.nil?)`, or `assert_predicate(something, :nil?)`.
       #
       # @example
       #   # bad
@@ -12,6 +12,8 @@ module RuboCop
       #   assert_equal(nil, actual, 'message')
       #   assert(object.nil?)
       #   assert(object.nil?, 'message')
+      #   assert_predicate(object, :nil?)
+      #   assert_predicate(object, :nil?, 'message')
       #
       #   # good
       #   assert_nil(actual)
@@ -23,12 +25,13 @@ module RuboCop
         extend AutoCorrector
 
         ASSERTION_TYPE = 'assert'
-        RESTRICT_ON_SEND = %i[assert_equal assert].freeze
+        RESTRICT_ON_SEND = %i[assert assert_equal assert_predicate].freeze
 
         def_node_matcher :nil_assertion, <<~PATTERN
           {
             (send nil? :assert_equal nil $_ $...)
             (send nil? :assert (send $_ :nil?) $...)
+            (send nil? :assert_predicate $_ (sym :nil?) $...)
           }
         PATTERN
 

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

@@ -18,7 +18,7 @@ module RuboCop
       class AssertPathExists < Base
         extend AutoCorrector
 
-        MSG = 'Prefer using `%<good_method>s` over `%<bad_method>s`.'
+        MSG = 'Prefer using `%<good_method>s`.'
         RESTRICT_ON_SEND = %i[assert].freeze
 
         def_node_matcher :assert_file_exists, <<~PATTERN
@@ -32,7 +32,7 @@ module RuboCop
           assert_file_exists(node) do |path, failure_message|
             failure_message = failure_message.first
             good_method = build_good_method(path, failure_message)
-            message = format(MSG, good_method: good_method, bad_method: node.source)
+            message = format(MSG, good_method: good_method)
 
             add_offense(node, message: message) do |corrector|
               corrector.replace(node, good_method)

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

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Minitest
+      # This cop enforces the test to use `assert_predicate`
+      # instead of using `assert(obj.a_predicate_method?)`.
+      #
+      # @example
+      #   # bad
+      #   assert(obj.one?)
+      #   assert(obj.one?, 'message')
+      #
+      #   # good
+      #   assert_predicate(obj, :one?)
+      #   assert_predicate(obj, :one?, 'message')
+      #
+      class AssertPredicate < Base
+        include ArgumentRangeHelper
+        include PredicateAssertionHandleable
+        extend AutoCorrector
+
+        MSG = 'Prefer using `assert_predicate(%<new_arguments>s)`.'
+        RESTRICT_ON_SEND = %i[assert].freeze
+
+        private
+
+        def assertion_type
+          'assert'
+        end
+      end
+    end
+  end
+end

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

@@ -16,7 +16,7 @@ module RuboCop
       class AssertSilent < Base
         extend AutoCorrector
 
-        MSG = 'Prefer using `assert_silent` over `assert_output("", "")`.'
+        MSG = 'Prefer using `assert_silent`.'
 
         def_node_matcher :assert_silent_candidate?, <<~PATTERN
           (block

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

@@ -19,8 +19,7 @@ module RuboCop
         include ArgumentRangeHelper
         extend AutoCorrector
 
-        MSG = 'Prefer using `assert(%<arguments>s)` over ' \
-              '`assert_equal(true, %<arguments>s)`.'
+        MSG = 'Prefer using `assert(%<arguments>s)`.'
         RESTRICT_ON_SEND = %i[assert_equal].freeze
 
         def_node_matcher :assert_equal_with_truthy, <<~PATTERN

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

@@ -18,20 +18,19 @@ module RuboCop
         include ArgumentRangeHelper
         extend AutoCorrector
 
-        MSG = 'Prefer using `refute_equal(%<preferred>s)` over ' \
-              '`assert(%<over>s)`.'
+        MSG = 'Prefer using `refute_equal(%<preferred>s)`.'
         RESTRICT_ON_SEND = %i[assert].freeze
 
         def_node_matcher :assert_not_equal, <<~PATTERN
-          (send nil? :assert ${(send $_ :!= $_) (send (send $_ :! ) :== $_) } $... )
+          (send nil? :assert {(send $_ :!= $_) (send (send $_ :! ) :== $_) } $... )
         PATTERN
 
         def on_send(node)
-          preferred, over = process_not_equal(node)
-          return unless preferred && over
+          preferred = process_not_equal(node)
+          return unless preferred
 
-          assert_not_equal(node) do |_, expected, actual|
-            message = format(MSG, preferred: preferred, over: over)
+          assert_not_equal(node) do |expected, actual|
+            message = format(MSG, preferred: preferred)
 
             add_offense(node, message: message) do |corrector|
               corrector.replace(node.loc.selector, 'refute_equal')
@@ -54,11 +53,10 @@ module RuboCop
         end
 
         def process_not_equal(node)
-          assert_not_equal(node) do |over, first_arg, second_arg, rest_args|
+          assert_not_equal(node) do |first_arg, second_arg, rest_args|
             custom_message = rest_args.first
-            preferred = preferred_usage(first_arg, second_arg, custom_message)
-            over = original_usage(over.source, custom_message&.source)
-            return [preferred, over]
+
+            preferred_usage(first_arg, second_arg, custom_message)
           end
         end
       end

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

@@ -22,8 +22,8 @@ module RuboCop
         include ArgumentRangeHelper
         extend AutoCorrector
 
-        MSG_FOR_ASSERT_EQUAL = 'Prefer using `refute(%<arguments>s)` over `assert_equal(false, %<arguments>s)`.'
-        MSG_FOR_ASSERT = 'Prefer using `refute(%<arguments>s)` over `assert(!%<arguments>s)`.'
+        MSG_FOR_ASSERT_EQUAL = 'Prefer using `refute(%<arguments>s)`.'
+        MSG_FOR_ASSERT = 'Prefer using `refute(%<arguments>s)`.'
         RESTRICT_ON_SEND = %i[assert_equal assert].freeze
 
         def_node_matcher :assert_equal_with_false, <<~PATTERN

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

@@ -4,7 +4,7 @@ module RuboCop
   module Cop
     module Minitest
       # This cop enforces the test to use `refute_nil` instead of using
-      # `refute_equal(nil, something)` or `refute(something.nil?)`.
+      # `refute_equal(nil, something)`, `refute(something.nil?)`, or `refute_predicate(something, :nil?)`.
       #
       # @example
       #   # bad
@@ -12,6 +12,8 @@ module RuboCop
       #   refute_equal(nil, actual, 'message')
       #   refute(actual.nil?)
       #   refute(actual.nil?, 'message')
+      #   refute_predicate(object, :nil?)
+      #   refute_predicate(object, :nil?, 'message')
       #
       #   # good
       #   refute_nil(actual)
@@ -23,12 +25,13 @@ module RuboCop
         extend AutoCorrector
 
         ASSERTION_TYPE = 'refute'
-        RESTRICT_ON_SEND = %i[refute_equal refute].freeze
+        RESTRICT_ON_SEND = %i[refute refute_equal refute_predicate].freeze
 
         def_node_matcher :nil_refutation, <<~PATTERN
           {
             (send nil? :refute_equal nil $_ $...)
             (send nil? :refute (send $_ :nil?) $...)
+            (send nil? :refute_predicate $_ (sym :nil?) $...)
           }
         PATTERN
 

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

@@ -18,7 +18,7 @@ module RuboCop
       class RefutePathExists < Base
         extend AutoCorrector
 
-        MSG = 'Prefer using `%<good_method>s` over `%<bad_method>s`.'
+        MSG = 'Prefer using `%<good_method>s`.'
         RESTRICT_ON_SEND = %i[refute].freeze
 
         def_node_matcher :refute_file_exists, <<~PATTERN
@@ -32,7 +32,7 @@ module RuboCop
           refute_file_exists(node) do |path, failure_message|
             failure_message = failure_message.first
             good_method = build_good_method(path, failure_message)
-            message = format(MSG, good_method: good_method, bad_method: node.source)
+            message = format(MSG, good_method: good_method)
 
             add_offense(node, message: message) do |corrector|
               corrector.replace(node, good_method)

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

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Minitest
+      # This cop enforces the test to use `refute_predicate`
+      # instead of using `refute(obj.a_predicate_method?)`.
+      #
+      # @example
+      #   # bad
+      #   refute(obj.one?)
+      #   refute(obj.one?, 'message')
+      #
+      #   # good
+      #   refute_predicate(obj, :one?)
+      #   refute_predicate(obj, :one?, 'message')
+      #
+      class RefutePredicate < Base
+        include ArgumentRangeHelper
+        include PredicateAssertionHandleable
+        extend AutoCorrector
+
+        MSG = 'Prefer using `refute_predicate(%<new_arguments>s)`.'
+        RESTRICT_ON_SEND = %i[refute].freeze
+
+        private
+
+        def assertion_type
+          'refute'
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/minitest_cops.rb

@@ -5,10 +5,12 @@ require_relative 'mixin/in_delta_mixin'
 require_relative 'mixin/minitest_cop_rule'
 require_relative 'mixin/minitest_exploration_helpers'
 require_relative 'mixin/nil_assertion_handleable'
+require_relative 'mixin/predicate_assertion_handleable'
 require_relative 'minitest/assert_empty'
 require_relative 'minitest/assert_empty_literal'
 require_relative 'minitest/assert_equal'
 require_relative 'minitest/assert_in_delta'
+require_relative 'minitest/assert_predicate'
 require_relative 'minitest/assert_with_expected_argument'
 require_relative 'minitest/assertion_in_lifecycle_hook'
 require_relative 'minitest/assert_kind_of'
@@ -35,6 +37,7 @@ require_relative 'minitest/refute_includes'
 require_relative 'minitest/refute_match'
 require_relative 'minitest/refute_instance_of'
 require_relative 'minitest/refute_path_exists'
+require_relative 'minitest/refute_predicate'
 require_relative 'minitest/refute_respond_to'
 require_relative 'minitest/test_method_name'
 require_relative 'minitest/unreachable_assertion'

data/lib/rubocop/cop/mixin/in_delta_mixin.rb

@@ -4,7 +4,7 @@ module RuboCop
   module Cop
     # Common functionality for `AssertInDelta` and `RefuteInDelta` cops.
     module InDeltaMixin
-      MSG = 'Prefer using `%<good_method>s` over `%<bad_method>s`.'
+      MSG = 'Prefer using `%<good_method>s`.'
 
       def on_send(node)
         equal_floats_call(node) do |expected, actual, message|
@@ -12,7 +12,7 @@ module RuboCop
           good_method = build_good_method(expected, actual, message)
 
           if expected.float_type? || actual.float_type?
-            message = format(MSG, good_method: good_method, bad_method: node.source)
+            message = format(MSG, good_method: good_method)
 
             add_offense(node, message: message) do |corrector|
               corrector.replace(node, good_method)

data/lib/rubocop/cop/mixin/minitest_cop_rule.rb

@@ -28,8 +28,7 @@ module RuboCop
           include ArgumentRangeHelper
           extend AutoCorrector
 
-          MSG = 'Prefer using `#{preferred_method}(%<new_arguments>s)` over ' \
-                '`#{assertion_method}(%<original_arguments>s)`.'
+          MSG = 'Prefer using `#{preferred_method}(%<new_arguments>s)`.'
           RESTRICT_ON_SEND = %i[#{assertion_method}].freeze
 
           def on_send(node)
@@ -70,12 +69,9 @@ module RuboCop
               message_argument&.source
             ].flatten.compact.join(', ')
 
-            original_arguments = arguments.map(&:source).join(', ')
-
             format(
               MSG,
-              new_arguments: new_arguments,
-              original_arguments: original_arguments
+              new_arguments: new_arguments
             )
           end
 

data/lib/rubocop/cop/mixin/nil_assertion_handleable.rb

@@ -5,7 +5,7 @@ module RuboCop
     module Minitest
       # Common functionality for `AssertNil` and `RefuteNil` cops.
       module NilAssertionHandleable
-        MSG = 'Prefer using `%<assertion_type>s_nil(%<preferred_args>s)` over `%<method>s(%<current_args>s)`.'
+        MSG = 'Prefer using `%<assertion_type>s_nil(%<preferred_args>s)`.'
 
         private
 
@@ -22,23 +22,18 @@ module RuboCop
           message_source = message&.source
 
           preferred_args = [actual.source, message_source].compact
-          current_args = if comparison_assertion_method?(node)
-                           ['nil', preferred_args].join(', ')
-                         else
-                           ["#{actual.source}.nil?", message_source].compact.join(', ')
-                         end
 
           format(
             MSG,
             assertion_type: assertion_type,
             preferred_args: preferred_args.join(', '),
-            method: node.method_name, current_args: current_args
+            method: node.method_name
           )
         end
 
         def autocorrect(corrector, node, actual)
           corrector.replace(node.loc.selector, :"#{assertion_type}_nil")
-          if comparison_assertion_method?(node)
+          if comparison_or_predicate_assertion_method?(node)
             corrector.replace(first_and_second_arguments_range(node), actual.source)
           else
             corrector.remove(node.first_argument.loc.dot)
@@ -46,8 +41,8 @@ module RuboCop
           end
         end
 
-        def comparison_assertion_method?(node)
-          node.method?(:"#{assertion_type}_equal")
+        def comparison_or_predicate_assertion_method?(node)
+          node.method?(:"#{assertion_type}_equal") || node.method?(:"#{assertion_type}_predicate")
         end
       end
     end

data/lib/rubocop/cop/mixin/predicate_assertion_handleable.rb

@@ -0,0 +1,61 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Minitest
+      # Common functionality for `Minitest/AssertPredicate` and `Minitest/RefutePredicate` cops.
+      module PredicateAssertionHandleable
+        MSG = 'Prefer using `%<assertion_type>s_predicate(%<new_arguments>s)`.'
+        RESTRICT_ON_SEND = %i[assert].freeze
+
+        def on_send(node)
+          return unless (arguments = peel_redundant_parentheses_from(node.arguments))
+          return unless arguments.first.respond_to?(:predicate_method?) && arguments.first.predicate_method?
+          return unless arguments.first.arguments.count.zero?
+
+          add_offense(node, message: offense_message(arguments)) do |corrector|
+            autocorrect(corrector, node, arguments)
+          end
+        end
+
+        def autocorrect(corrector, node, arguments)
+          corrector.replace(node.loc.selector, "#{assertion_type}_predicate")
+
+          new_arguments = new_arguments(arguments).join(', ')
+
+          corrector.replace(first_argument_range(node), new_arguments)
+        end
+
+        private
+
+        def peel_redundant_parentheses_from(arguments)
+          return arguments unless arguments.first&.begin_type?
+
+          peel_redundant_parentheses_from(arguments.first.children)
+        end
+
+        def offense_message(arguments)
+          message_argument = arguments.last if arguments.first != arguments.last
+
+          new_arguments = [
+            new_arguments(arguments),
+            message_argument&.source
+          ].flatten.compact.join(', ')
+
+          format(MSG, assertion_type: assertion_type, new_arguments: new_arguments)
+        end
+
+        def new_arguments(arguments)
+          receiver = correct_receiver(arguments.first.receiver)
+          method_name = arguments.first.method_name
+
+          [receiver, ":#{method_name}"]
+        end
+
+        def correct_receiver(receiver)
+          receiver ? receiver.source : 'self'
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/minitest/assert_offense.rb

@@ -22,7 +22,7 @@ module RuboCop
     #     class FooTest < Minitest::Test
     #       def test_do_something
     #         assert_equal(nil, somestuff)
-    #         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Prefer using `assert_nil(somestuff)` over `assert_equal(nil, somestuff)`.
+    #         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Prefer using `assert_nil(somestuff)`.
     #       end
     #     end
     #   RUBY
@@ -36,7 +36,7 @@ module RuboCop
     #     class FooTest < Minitest::Test
     #       def test_do_something
     #         assert_equal(nil, somestuff)
-    #         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Prefer using `assert_nil(somestuff)` over `assert_equal(nil, somestuff)`.
+    #         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Prefer using `assert_nil(somestuff)`.
     #       end
     #     end
     #   RUBY
@@ -64,7 +64,7 @@ module RuboCop
     #     class FooTest < Minitest::Test
     #       def test_do_something
     #         assert_equal(nil, somestuff)
-    #         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Prefer using `assert_nil(somestuff)` over `assert_equal(nil, somestuff)`.
+    #         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Prefer using `assert_nil(somestuff)`.
     #       end
     #     end
     #   RUBY

data/lib/rubocop/minitest/version.rb

@@ -4,7 +4,7 @@ module RuboCop
   module Minitest
     # This module holds the RuboCop Minitest version information.
     module Version
-      STRING = '0.17.2'
+      STRING = '0.18.0'
 
       def self.document_version
         STRING.match('\d+\.\d+').to_s

data/mkdocs.yml

@@ -4,10 +4,10 @@ edit_uri: edit/master/legacy-docs/
 copyright: "Copyright © 2022 Bozhidar Batsov, Jonas Arvidsson, Koichi ITO, and RuboCop contributors"
 docs_dir: legacy-docs
 pages:
-- Home: index.md
-- Installation: installation.md
-- Usage: usage.md
-- Cops: cops.md
-- Cops Documentation:
-    - Minitest Cops: cops_minitest.md
+  - Home: index.md
+  - Installation: installation.md
+  - Usage: usage.md
+  - Cops: cops.md
+  - Cops Documentation:
+      - Minitest Cops: cops_minitest.md
 theme: readthedocs

data/relnotes/v0.18.0.md

@@ -0,0 +1,9 @@
+### New features
+
+* [#161](https://github.com/rubocop/rubocop-minitest/pull/161): Add new `Minitest/AssertPredicate` and `Minitest/RefutePredicate` cops. ([@koic][])
+
+### Changes
+
+* [#162](https://github.com/rubocop/rubocop-minitest/pull/162): Make `Minitest/AssertNil` (`Minitest/RefuteNil`) aware of `assert_predicate(obj, :nil?)` (`refute_predicate(obj, :nil?)`). ([@koic][])
+
+[@koic]: https://github.com/koic

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-minitest
 version: !ruby/object:Gem::Version
-  version: 0.17.2
+  version: 0.18.0
 platform: ruby
 authors:
 - Bozhidar Batsov
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-02-12 00:00:00.000000000 Z
+date: 2022-03-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -60,10 +60,12 @@ files:
 - ".github/ISSUE_TEMPLATE/bug_report.md"
 - ".github/ISSUE_TEMPLATE/feature_request.md"
 - ".github/PULL_REQUEST_TEMPLATE.md"
+- ".github/workflows/linting.yml"
 - ".github/workflows/spell_checking.yml"
 - ".gitignore"
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
+- ".yamllint.yml"
 - ".yardopts"
 - CHANGELOG.md
 - CONTRIBUTING.md
@@ -100,6 +102,7 @@ files:
 - lib/rubocop/cop/minitest/assert_nil.rb
 - lib/rubocop/cop/minitest/assert_output.rb
 - lib/rubocop/cop/minitest/assert_path_exists.rb
+- lib/rubocop/cop/minitest/assert_predicate.rb
 - lib/rubocop/cop/minitest/assert_respond_to.rb
 - lib/rubocop/cop/minitest/assert_silent.rb
 - lib/rubocop/cop/minitest/assert_truthy.rb
@@ -119,6 +122,7 @@ files:
 - lib/rubocop/cop/minitest/refute_match.rb
 - lib/rubocop/cop/minitest/refute_nil.rb
 - lib/rubocop/cop/minitest/refute_path_exists.rb
+- lib/rubocop/cop/minitest/refute_predicate.rb
 - lib/rubocop/cop/minitest/refute_respond_to.rb
 - lib/rubocop/cop/minitest/test_method_name.rb
 - lib/rubocop/cop/minitest/unreachable_assertion.rb
@@ -129,6 +133,7 @@ files:
 - lib/rubocop/cop/mixin/minitest_cop_rule.rb
 - lib/rubocop/cop/mixin/minitest_exploration_helpers.rb
 - lib/rubocop/cop/mixin/nil_assertion_handleable.rb
+- lib/rubocop/cop/mixin/predicate_assertion_handleable.rb
 - lib/rubocop/minitest.rb
 - lib/rubocop/minitest/assert_offense.rb
 - lib/rubocop/minitest/inject.rb
@@ -154,6 +159,7 @@ files:
 - relnotes/v0.17.0.md
 - relnotes/v0.17.1.md
 - relnotes/v0.17.2.md
+- relnotes/v0.18.0.md
 - relnotes/v0.2.0.md
 - relnotes/v0.2.1.md
 - relnotes/v0.3.0.md
@@ -180,7 +186,7 @@ metadata:
   homepage_uri: https://docs.rubocop.org/rubocop-minitest/
   changelog_uri: https://github.com/rubocop/rubocop-minitest/blob/master/CHANGELOG.md
   source_code_uri: https://github.com/rubocop/rubocop-minitest
-  documentation_uri: https://docs.rubocop.org/rubocop-minitest/0.17
+  documentation_uri: https://docs.rubocop.org/rubocop-minitest/0.18
   bug_tracker_uri: https://github.com/rubocop/rubocop-minitest/issues
   rubygems_mfa_required: 'true'
 post_install_message: