rubocop-minitest 0.9.0 → 0.11.0

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

@@ -0,0 +1,5 @@
+= RuboCop Minitest
+
+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/docs/modules/ROOT/pages/installation.adoc

@@ -0,0 +1,15 @@
+= Installation
+
+Just install the `rubocop-minitest` gem
+
+[source,sh]
+----
+gem install rubocop-minitest
+----
+
+or if you use bundler put this in your `Gemfile`
+
+[source,ruby]
+----
+gem 'rubocop-minitest'
+----

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

@@ -0,0 +1,32 @@
+= Usage
+
+You need to tell RuboCop to load the Minitest extension. There are three
+ways to do this:
+
+== RuboCop configuration file
+
+Put this into your `.rubocop.yml`.
+
+[source,yaml]
+----
+require: rubocop-minitest
+----
+
+Now you can run `rubocop` and it will automatically load the RuboCop Minitest
+cops together with the standard cops.
+
+== Command line
+
+[source,sh]
+----
+rubocop --require rubocop-minitest
+----
+
+== Rake task
+
+[source,ruby]
+----
+RuboCop::RakeTask.new do |task|
+  task.requires << 'rubocop-minitest'
+end
+----

data/{manual → 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

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

data/lib/rubocop/cop/generator.rb

@@ -0,0 +1,56 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    # Source and test generator for new cops
+    #
+    # This generator will take a cop name and generate a source file
+    # and test file when given a valid qualified cop name.
+    class Generator
+      TEST_TEMPLATE = <<~TEST
+        # frozen_string_literal: true
+
+        require 'test_helper'
+
+        class %<cop_name>sTest < Minitest::Test
+          def test_registers_offense_when_using_bad_method
+            assert_offense(<<~RUBY)
+              bad_method
+              ^^^^^^^^^^ Use `#good_method` instead of `#bad_method`.
+            RUBY
+
+            assert_correction(<<~RUBY)
+              good_method
+            RUBY
+          end
+
+          def test_does_not_register_offense_when_using_good_method
+            assert_no_offenses(<<~RUBY)
+              good_method
+            RUBY
+          end
+        end
+      TEST
+
+      def write_test
+        write_unless_file_exists(test_path, generated_test)
+      end
+
+      private
+
+      def test_path
+        File.join(
+          'test',
+          'rubocop',
+          'cop',
+          'minitest',
+          "#{snake_case(badge.cop_name.to_s)}_test.rb"
+        )
+      end
+
+      def generated_test
+        generate(TEST_TEMPLATE)
+      end
+    end
+  end
+end

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

@@ -4,32 +4,48 @@ 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
+        include ArgumentRangeHelper
+
         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_equal_with_empty_literal(node) do |_literal, matchers|
+            object = matchers.first
+
+            lambda do |corrector|
+              corrector.replace(node.loc.selector, 'assert_empty')
+              corrector.replace(first_and_second_arguments_range(node), object.source)
+            end
+          end
+        end
       end
     end
   end

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

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Minitest
+      # This cop enforces the test to use `assert_in_delta`
+      # instead of using `assert_equal` to compare floats.
+      #
+      # @example
+      #   # bad
+      #   assert_equal(0.2, actual)
+      #   assert_equal(0.2, actual, 'message')
+      #
+      #   # good
+      #   assert_in_delta(0.2, actual)
+      #   assert_in_delta(0.2, actual, 0.001, 'message')
+      #
+      class AssertInDelta < Cop
+        include InDeltaMixin
+
+        RESTRICT_ON_SEND = %i[assert_equal].freeze
+
+        def_node_matcher :equal_floats_call, <<~PATTERN
+          (send nil? :assert_equal $_ $_ $...)
+        PATTERN
+      end
+    end
+  end
+end

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

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Minitest
+      # This cop enforces the test to use `assert_kind_of(Class, object)`
+      # over `assert(object.kind_of?(Class))`.
+      #
+      # @example
+      #   # bad
+      #   assert(object.kind_of?(Class))
+      #   assert(object.kind_of?(Class), 'message')
+      #
+      #   # good
+      #   assert_kind_of(Class, object)
+      #   assert_kind_of(Class, object, 'message')
+      #
+      class AssertKindOf < Cop
+        extend MinitestCopRule
+
+        define_rule :assert, target_method: :kind_of?, inverse: true
+      end
+    end
+  end
+end

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

@@ -20,6 +20,7 @@ module RuboCop
 
         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 $_ $...)

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

@@ -0,0 +1,49 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Minitest
+      # This cop checks for opportunities to use `assert_output`.
+      #
+      # @example
+      #   # bad
+      #   $stdout = StringIO.new
+      #   puts object.method
+      #   $stdout.rewind
+      #   assert_match expected, $stdout.read
+      #
+      #   # good
+      #   assert_output(expected) { puts object.method }
+      #
+      class AssertOutput < Cop
+        include MinitestExplorationHelpers
+
+        MSG = 'Use `assert_output` instead of mutating %<name>s.'
+        OUTPUT_GLOBAL_VARIABLES = %i[$stdout $stderr].freeze
+
+        def on_gvasgn(node)
+          test_case_node = find_test_case(node)
+          return unless test_case_node
+
+          gvar_name = node.children.first
+          return unless OUTPUT_GLOBAL_VARIABLES.include?(gvar_name)
+
+          assertions(test_case_node).each do |assertion|
+            add_offense(assertion, message: format(MSG, name: gvar_name)) if references_gvar?(assertion, gvar_name)
+          end
+        end
+
+        private
+
+        def find_test_case(node)
+          def_ancestor = node.each_ancestor(:def).first
+          def_ancestor if test_case?(def_ancestor)
+        end
+
+        def references_gvar?(assertion, gvar_name)
+          assertion.each_descendant(:gvar).any? { |d| d.children.first == gvar_name }
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,59 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Minitest
+      # This cop enforces the test to use `assert_path_exists`
+      # instead of using `assert(File.exist?(path))`.
+      #
+      # @example
+      #   # bad
+      #   assert(File.exist?(path))
+      #   assert(File.exist?(path), 'message')
+      #
+      #   # good
+      #   assert_path_exists(path)
+      #   assert_path_exists(path, 'message')
+      #
+      class AssertPathExists < Cop
+        MSG = 'Prefer using `%<good_method>s` over `%<bad_method>s`.'
+        RESTRICT_ON_SEND = %i[assert].freeze
+
+        def_node_matcher :assert_file_exists, <<~PATTERN
+          (send nil? :assert
+            (send
+              (const _ :File) {:exist? :exists?} $_)
+              $...)
+        PATTERN
+
+        def on_send(node)
+          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)
+
+            add_offense(node, message: message)
+          end
+        end
+
+        def autocorrect(node)
+          assert_file_exists(node) do |path, failure_message|
+            failure_message = failure_message.first
+
+            lambda do |corrector|
+              replacement = build_good_method(path, failure_message)
+              corrector.replace(node, replacement)
+            end
+          end
+        end
+
+        private
+
+        def build_good_method(path, message)
+          args = [path.source, message&.source].compact.join(', ')
+          "assert_path_exists(#{args})"
+        end
+      end
+    end
+  end
+end