rubocop-minitest 0.10.1 → 0.12.0

data/bin/console

@@ -2,6 +2,8 @@
 # frozen_string_literal: true
 
 require 'bundler/setup'
+require 'pathname'
+require 'yaml'
 require 'rubocop/minitest'
 
 # You can add fixtures and/or initialization code here to make experimenting

data/config/default.yml

@@ -15,7 +15,7 @@ Minitest/AssertEmptyLiteral:
   Enabled: true
   SafeAutoCorrect: false
   VersionAdded: '0.5'
-  VersionChanged: '0.10'
+  VersionChanged: '0.11'
 
 Minitest/AssertEqual:
   Description: 'This cop enforces the test to use `assert_equal` instead of using `assert(expected == actual)`.'
@@ -54,7 +54,7 @@ Minitest/AssertInstanceOf:
 
 Minitest/AssertKindOf:
   Description: 'This cop enforces the test to use `assert_kind_of(Class, object)` over `assert(object.kind_of?(Class))`'
-  StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#assert-kind-of'
+  StyleGuide: 'https://github.com/rubocop/minitest-style-guide#assert-kind-of'
   Enabled: 'pending'
   VersionAdded: '0.10'
 
@@ -84,7 +84,7 @@ Minitest/AssertRespondTo:
 
 Minitest/AssertSilent:
   Description: "This cop enforces the test to use `assert_silent { ... }` instead of using `assert_output('', '') { ... }`."
-  StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#assert-silent'
+  StyleGuide: 'https://github.com/rubocop/minitest-style-guide#assert-silent'
   Enabled: 'pending'
   VersionAdded: '0.10'
 
@@ -94,6 +94,12 @@ Minitest/AssertTruthy:
   Enabled: true
   VersionAdded: '0.2'
 
+Minitest/AssertWithExpectedArgument:
+  Description: 'This cop tries to detect when a user accidentally used `assert` when they meant to use `assert_equal`.'
+  Enabled: pending
+  Safe: false
+  VersionAdded: '0.11'
+
 Minitest/GlobalExpectations:
   Description: 'This cop checks for deprecated global expectations.'
   StyleGuide: 'https://minitest.rubystyle.guide#global-expectations'
@@ -112,6 +118,11 @@ Minitest/MultipleAssertions:
   VersionAdded: '0.10'
   Max: 3
 
+Minitest/NoAssertions:
+  Description: 'This cop checks for at least one assertion (or flunk) in tests.'
+  Enabled: false
+  VersionAdded: '0.12'
+
 Minitest/RefuteEmpty:
   Description: 'This cop enforces to use `refute_empty` instead of using `refute(object.empty?)`.'
   StyleGuide: 'https://minitest.rubystyle.guide#refute-empty'
@@ -156,7 +167,7 @@ Minitest/RefuteInstanceOf:
 
 Minitest/RefuteKindOf:
   Description: 'This cop enforces the test to use `refute_kind_of(Class, object)` over `refute(object.kind_of?(Class))`.'
-  StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#refute-kind-of'
+  StyleGuide: 'https://github.com/rubocop/minitest-style-guide#refute-kind-of'
   Enabled: 'pending'
   VersionAdded: '0.10'
 

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

@@ -1,6 +1,17 @@
+= Cops
+
+In RuboCop lingo the various checks performed on the code are called cops.
+Each cop is responsible for detecting one particular offense.
+RuboCop Minitest has only one Minitest department.
+
+== Minitest
+
+Minitest cops check for Minitest best practices and coding conventions. Many of the them are
+based on the https://minitest.rubystyle.guide/[Minitest Style Guide].
+
 // START_COP_LIST
 
-= Department xref:cops_minitest.adoc[Minitest]
+=== Department xref:cops_minitest.adoc[Minitest]
 
 * xref:cops_minitest.adoc#minitestassertempty[Minitest/AssertEmpty]
 * xref:cops_minitest.adoc#minitestassertemptyliteral[Minitest/AssertEmptyLiteral]
@@ -16,10 +27,12 @@
 * 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
 
 |===
@@ -638,7 +702,7 @@ refute_empty(object, 'message')
 |===
 
 This cop enforces the use of `refute_equal(expected, object)`
-over `assert_equal(expected != actual)` or `assert(! expected == actual)`.
+over `assert(expected != actual)` or `assert(! expected == actual)`.
 
 === Examples
 
@@ -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