[Fix rubocop#147] Minitest/GlobalExpectations: add PreferredMethod co…

…nfig value Add a new configuration value for Minitest/GlobalExpectations: PreferredMethod. This value sets the preferred method with which to correct errors. Note: This does not alter the set of correct methods, only the replacement for offenses. Fixes rubocop#147
gi · Oct 19, 2021 · cefaaa3 · cefaaa3
1 parent 62fef12
commit cefaaa3
Show file tree

Hide file tree

Showing 5 changed files with 131 additions and 33 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,9 @@
 ### Bug fixes
 
 * [#142](https://github.com/rubocop/rubocop-minitest/issues/142): Fix `Minitest/GlobalExpectations` autocorrect when receiver is lambda. ([@gi][])
+### New features
+
+* [#148](https://github.com/rubocop/rubocop-minitest/pull/148): `Minitest/GlobalExpectations`: add `PreferredMethod` config value. ([@gi][])
 
 ## 0.15.2 (2021-10-11)
 

diff --git a/config/default.yml b/config/default.yml
@@ -1,7 +1,9 @@
 Minitest:
   Enabled: true
   Include:
+    - '**/spec/**/*'
     - '**/test/**/*'
+    - '**/*_spec.rb'
     - '**/*_test.rb'
 
 Minitest/AssertEmpty:
@@ -103,6 +105,7 @@ Minitest/GlobalExpectations:
   Description: 'This cop checks for deprecated global expectations.'
   StyleGuide: 'https://minitest.rubystyle.guide#global-expectations'
   Enabled: true
+  PreferredMethod: _
   VersionAdded: '0.7'
 
 Minitest/LiteralAsActualArgument:

diff --git a/docs/modules/ROOT/pages/cops_minitest.adoc b/docs/modules/ROOT/pages/cops_minitest.adoc
@@ -527,6 +527,8 @@ and autocorrects them to use expect format.
 
 === Examples
 
+==== PreferredMethod: _ (default)
+
 [source,ruby]
 ----
 # bad
@@ -540,6 +542,46 @@ _(wonts).wont_match expected_wonts
 _ { musts }.must_raise TypeError
 ----
 
+==== PreferredMethod: expect
+
+[source,ruby]
+----
+# bad
+musts.must_equal expected_musts
+wonts.wont_match expected_wonts
+musts.must_raise TypeError
+
+# good
+expect(musts).must_equal expected_musts
+expect(wonts).wont_match expected_wonts
+expect { musts }.must_raise TypeError
+----
+
+==== PreferredMethod: value
+
+[source,ruby]
+----
+# bad
+musts.must_equal expected_musts
+wonts.wont_match expected_wonts
+musts.must_raise TypeError
+
+# good
+value(musts).must_equal expected_musts
+value(wonts).wont_match expected_wonts
+value { musts }.must_raise TypeError
+----
+
+=== Configurable attributes
+
+|===
+| Name | Default value | Configurable values
+
+| PreferredMethod
+| `_`
+| String
+|===
+
 === References
 
 * https://minitest.rubystyle.guide#global-expectations

diff --git a/lib/rubocop/cop/minitest/global_expectations.rb b/lib/rubocop/cop/minitest/global_expectations.rb
@@ -6,7 +6,7 @@ module Minitest
       # This cop checks for deprecated global expectations
       # and autocorrects them to use expect format.
       #
-      # @example
+      # @example PreferredMethod: _ (default)
       #   # bad
       #   musts.must_equal expected_musts
       #   wonts.wont_match expected_wonts
@@ -16,6 +16,28 @@ module Minitest
       #   _(musts).must_equal expected_musts
       #   _(wonts).wont_match expected_wonts
       #   _ { musts }.must_raise TypeError
+      #
+      # @example PreferredMethod: expect
+      #   # bad
+      #   musts.must_equal expected_musts
+      #   wonts.wont_match expected_wonts
+      #   musts.must_raise TypeError
+      #
+      #   # good
+      #   expect(musts).must_equal expected_musts
+      #   expect(wonts).wont_match expected_wonts
+      #   expect { musts }.must_raise TypeError
+      #
+      # @example PreferredMethod: value
+      #   # bad
+      #   musts.must_equal expected_musts
+      #   wonts.wont_match expected_wonts
+      #   musts.must_raise TypeError
+      #
+      #   # good
+      #   value(musts).must_equal expected_musts
+      #   value(wonts).wont_match expected_wonts
+      #   value { musts }.must_raise TypeError
       class GlobalExpectations < Base
         extend AutoCorrector