rubocop-minitest 0.2.0 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c5d715a17205c012bd5da344d7a8cd6438334554ee22cb509731756c0be17cb3
-  data.tar.gz: f833ffb2d8c3b23e8c7e49b9fc756193e9db5f83668423975201e8a8d52df433
+  metadata.gz: 4fb2a1153e8f5e4eb37a349fcdfff211a421369606facb6d866fbc4701a60d6c
+  data.tar.gz: 2e258868d43eddc169e142e408e01d52e7d6e63ac5e9f1f53643feca8ff8968e
 SHA512:
-  metadata.gz: 94c2bca0cb05ac331ed6fd20fb2f0d12ef8ebe8866d2b9201ec5a0ef1194a0b4550d3fe4940e976d303dfef643e6258ac8c3f3b9e44b24086b1466f314385fdc
-  data.tar.gz: 377e96496bf87ba7ab7d49bffd9abf7b355ff43a2259af03be88bbb214fe389b85c89e1d0f1851f2eee88530bd5bad8cc4b2c252d3c4269f2ccbd686e4b32c05
+  metadata.gz: 466706a3aa3020369c7b855c20ad0619cfd8d2e4c94a6f260287908ce2d5ecefdc8190bc88a60d7cb59d0cdd39a276dc87ebdea91c240a5d820e167d984424c9
+  data.tar.gz: 6aede577f029efa7fb8a2cf89151261c16dfe57f980100b568c8e11252b7652e45cac051f404212411e2cac2a0232d0c4c752322f6781fa3c37297e6a10eae11

data/.github/FUNDING.yml

@@ -0,0 +1,6 @@
+# These are supported funding model platforms
+
+github: bbatsov
+patreon: bbatsov
+open_collective: rubocop
+custom: https://www.paypal.me/bbatsov

data/CHANGELOG.md

@@ -1,22 +1,70 @@
+# Change log
+
 ## master (unreleased)
 
+## 0.5.0 (2019-11-24)
+
+### New features
+
+* [#32](https://github.com/rubocop-hq/rubocop-minitest/issues/32): Add new `Minitest/AssertEmptyLiteral` cop. ([@tejasbubane][])
+
+## 0.4.1 (2019-11-10)
+
+### Bug fixes
+
+* [#39](https://github.com/rubocop-hq/rubocop-minitest/issues/39): Fix an incorrect autocorrect for `Minitest/AssertRespondTo` and `Minitest/RefuteRespondTo` when using assertion method calling `respond_to` with receiver omitted. ([@koic][])
+
+## 0.4.0 (2019-11-07)
+
+### New features
+
+* [#29](https://github.com/rubocop-hq/rubocop-minitest/pull/29): Add new `Minitest/RefuteRespondTo` cop.  ([@herwinw][])
+* [#31](https://github.com/rubocop-hq/rubocop-minitest/pull/31): Add new `Minitest/AssertEqual` cop. ([@herwinw][])
+* [#34](https://github.com/rubocop-hq/rubocop-minitest/pull/34): Add new `Minitest/AssertInstanceOf` cop. ([@abhaynikam][])
+* [#35](https://github.com/rubocop-hq/rubocop-minitest/pull/35): Add new `Minitest/RefuteInstanceOf` cop. ([@abhaynikam][])
+
+### Bug fixes
+
+* [#25](https://github.com/rubocop-hq/rubocop-minitest/issues/25): Add `Enabled: true` to `Minitest` department config to suppress `Warning: Minitest does not support Enabled parameter`. ([@koic][])
+
+## 0.3.0 (2019-10-13)
+
+### New features
+
+* [#15](https://github.com/rubocop-hq/rubocop-minitest/pull/15): Add new `Minitest/RefuteIncludes` cop. ([@abhaynikam][])
+* [#18](https://github.com/rubocop-hq/rubocop-minitest/pull/18): Add new `Minitest/RefuteFalse` cop. ([@duduribeiro][])
+* [#20](https://github.com/rubocop-hq/rubocop-minitest/pull/20): Add new `Minitest/RefuteEmpty` cop. ([@abhaynikam][])
+* [#21](https://github.com/rubocop-hq/rubocop-minitest/pull/21): Add new `Minitest/RefuteEqual` cop. ([@duduribeiro][])
+* [#27](https://github.com/rubocop-hq/rubocop-minitest/pull/27): Add new `Minitest/AssertRespondTo` cop. ([@duduribeiro][])
+
+### Bug fixes
+
+* [#19](https://github.com/rubocop-hq/rubocop-minitest/pull/19): Fix a false negative for `Minitest/AssertIncludes` when using `include` method in arguments of `assert` method. ([@abhaynikam][])
+
+## 0.2.1 (2019-09-24)
+
+### Bug fixes
+
+* [#13](https://github.com/rubocop-hq/rubocop-minitest/issues/13): Fix the execution target specified in `Include` parameter. ([@koic][])
+
 ## 0.2.0 (2019-09-21)
 
 ### New features
 
-* [#11](https://github.com/rubocop-hq/rubocop-minitest/pull/11): Add new `Minitest/RefuteNil` cop. ([@tejasbubane ][])
-* [#8](https://github.com/rubocop-hq/rubocop-minitest/pull/8): Add new `Minitest/AssertTruthy` cop. ([@abhaynikam ][])
-* [#9](https://github.com/rubocop-hq/rubocop-minitest/pull/9): Add new `Minitest/AssertIncludes` cop. ([@abhaynikam ][])
-* [#10](https://github.com/rubocop-hq/rubocop-minitest/pull/10): Add new `Minitest/AssertEmpty` cop. ([@abhaynikam ][])
+* [#11](https://github.com/rubocop-hq/rubocop-minitest/pull/11): Add new `Minitest/RefuteNil` cop. ([@tejasbubane][])
+* [#8](https://github.com/rubocop-hq/rubocop-minitest/pull/8): Add new `Minitest/AssertTruthy` cop. ([@abhaynikam][])
+* [#9](https://github.com/rubocop-hq/rubocop-minitest/pull/9): Add new `Minitest/AssertIncludes` cop. ([@abhaynikam][])
+* [#10](https://github.com/rubocop-hq/rubocop-minitest/pull/10): Add new `Minitest/AssertEmpty` cop. ([@abhaynikam][])
 
 ## 0.1.0 (2019-09-01)
 
 ### New features
 
 * Create RuboCop Minitest gem. ([@koic][])
-* [#6](https://github.com/rubocop-hq/rubocop-minitest/pull/6): Add new `Minitest/AssertNil` cop. ([@duduribeiro ][])
+* [#6](https://github.com/rubocop-hq/rubocop-minitest/pull/6): Add new `Minitest/AssertNil` cop. ([@duduribeiro][])
 
 [@koic]: https://github.com/koic
 [@duduribeiro]: https://github.com/duduribeiro
 [@tejasbubane]: https://github.com/tejasbubane
 [@abhaynikam]: https://github.com/abhaynikam
+[@herwinw]: https://github.com/herwinw

data/Gemfile

@@ -6,7 +6,8 @@ git_source(:github) { |repo| "https://github.com/#{repo}.git" }
 
 gemspec
 
+gem 'bump', require: false
 gem 'rake'
 gem 'rubocop', github: 'rubocop-hq/rubocop'
-gem 'rubocop-performance', '~> 1.4.0'
+gem 'rubocop-performance', '~> 1.5.0'
 gem 'yard', '~> 0.9'

data/README.md

@@ -4,6 +4,7 @@
 [![CircleCI](https://circleci.com/gh/rubocop-hq/rubocop-minitest.svg?style=svg)](https://circleci.com/gh/rubocop-hq/rubocop-minitest)
 
 A [RuboCop](https://github.com/rubocop-hq/rubocop) extension focused on enforcing Minitest best practices and coding conventions.
+The library is based on the guidelines outlined in the community [Minitest Style Guide](https://minitest.rubystyle.guide).
 
 ## Installation
 
@@ -61,7 +62,7 @@ end
 
 All cops are located under
 [`lib/rubocop/cop/minitest`](lib/rubocop/cop/minitest), and contain
-examples/documentation.
+examples/documentation. The documentation is published [here](https://docs.rubocop.org/projects/minitest).
 
 In your `.rubocop.yml`, you may treat the Minitest cops just like any other
 cop. For example:

data/config/default.yml

@@ -1,33 +1,93 @@
-AllCops:
+Minitest:
+  Enabled: true
   Include:
     - '**/test/**/*'
 
-Minitest/AssertNil:
-  Description: 'Check if your test uses `assert_nil` instead of `assert_equal(nil, something)`.'
-  StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#assert-nil'
-  Enabled: true
-  VersionAdded: '0.1'
-
 Minitest/AssertEmpty:
-  Description: 'Check if your test uses `assert_empty` instead of `assert(actual.empty?)`.'
+  Description: 'This cop enforces the test to use `assert_empty` instead of using `assert(object.empty?)`.'
   StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#assert-empty'
   Enabled: true
   VersionAdded: '0.2'
 
+Minitest/AssertEmptyLiteral:
+  Description: 'This cop enforces the test to use `assert_empty` instead of using `assert([], object)` or `assert({}, object)`.'
+  Enabled: true
+  VersionAdded: '0.5'
+
+Minitest/AssertEqual:
+  Description: 'This cop enforces the test to use `assert_equal` instead of using `assert(expected == actual)`.'
+  StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#assert-equal-arguments-order'
+  Enabled: true
+  VersionAdded: '0.4'
+
 Minitest/AssertIncludes:
-  Description: 'Check if your test uses `assert_includes` instead of `assert(collection.includes?(actual))`.'
+  Description: 'This cop enforces the test to use `assert_includes` instead of using `assert(collection.include?(object))`.'
   StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#assert-includes'
   Enabled: true
   VersionAdded: '0.2'
 
+Minitest/AssertInstanceOf:
+  Description: 'This cop enforces the test to use `assert_instance_of(Class, object)` over `assert(object.instance_of?(Class))`'
+  StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#assert-instance-of'
+  Enabled: true
+  VersionAdded: '0.4'
+
+Minitest/AssertNil:
+  Description: 'This cop enforces the test to use `assert_nil` instead of using `assert_equal(nil, something)`.'
+  StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#assert-nil'
+  Enabled: true
+  VersionAdded: '0.1'
+
+Minitest/AssertRespondTo:
+  Description: 'This cop enforces the test to use `assert_respond_to(object, :some_method)` over `assert(object.respond_to?(:some_method))`.'
+  StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#assert-responds-to-method'
+  Enabled: true
+  VersionAdded: '0.3'
+
 Minitest/AssertTruthy:
-  Description: 'Check if your test uses `assert(actual)` instead of `assert_equal(true, actual)`.'
+  Description: 'This cop enforces the test to use `assert(actual)` instead of using `assert_equal(true, actual)`.'
   StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#assert-truthy'
   Enabled: true
   VersionAdded: '0.2'
 
+Minitest/RefuteEmpty:
+  Description: 'This cop enforces to use `refute_empty` instead of using `refute(object.empty?)`.'
+  StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#refute-empty'
+  Enabled: true
+  VersionAdded: '0.3'
+
+Minitest/RefuteEqual:
+  Description: 'Check if your test uses `refute_equal` instead of `assert(expected != object)` or `assert(! expected == object))`.'
+  StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#refute-equal'
+  Enabled: true
+  VersionAdded: '0.3'
+
+Minitest/RefuteFalse:
+  Description: 'Check if your test uses `refute(actual)` instead of `assert_equal(false, actual)`.'
+  StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#refute-false'
+  Enabled: true
+  VersionAdded: '0.3'
+
+Minitest/RefuteIncludes:
+  Description: 'This cop enforces the test to use `refute_includes` instead of using `refute(collection.include?(object))`.'
+  StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#refute-includes'
+  Enabled: true
+  VersionAdded: '0.3'
+
+Minitest/RefuteInstanceOf:
+  Description: 'This cop enforces the test to use `refute_instance_of(Class, object)` over `refute(object.instance_of?(Class))`.'
+  StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#refute-instance-of'
+  Enabled: true
+  VersionAdded: '0.4'
+
 Minitest/RefuteNil:
-  Description: 'Check if your test uses `refute_nil` instead of `refute_equal(nil, something)`.'
+  Description: 'This cop enforces the test to use `refute_nil` instead of using `refute_equal(nil, something)`.'
   StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#refute-nil'
   Enabled: true
   VersionAdded: '0.2'
+
+Minitest/RefuteRespondTo:
+  Description: 'This cop enforces the test to use `refute_respond_to(object, :some_method)` over `refute(object.respond_to?(:some_method))`.'
+  StyleGuide: 'https://github.com/rubocop-hq/minitest-style-guide#refute-respond-to'
+  Enabled: true
+  VersionAdded: '0.4'

data/lib/rubocop-minitest.rb

@@ -1,8 +1,5 @@
 # frozen_string_literal: true
 
-require 'pathname'
-require 'yaml'
-
 require 'rubocop'
 
 require_relative 'rubocop/minitest'

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

@@ -3,16 +3,17 @@
 module RuboCop
   module Cop
     module Minitest
-      # Check if your test uses `assert_empty` instead of `assert(actual.empty?)`.
+      # This cop enforces the test to use `assert_empty`
+      # instead of using `assert(object.empty?)`.
       #
       # @example
       #   # bad
-      #   assert(actual.empty?)
-      #   assert(actual.empty?, 'the message')
+      #   assert(object.empty?)
+      #   assert(object.empty?, 'the message')
       #
       #   # good
-      #   assert_empty(actual)
-      #   assert_empty(actual, 'the message')
+      #   assert_empty(object)
+      #   assert_empty(object, 'the message')
       #
       class AssertEmpty < Cop
         MSG = 'Prefer using `assert_empty(%<arguments>s)` over ' \

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

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Minitest
+      # This cop enforces the test to use `assert_empty`
+      # instead of using `assert([], object)`.
+      #
+      # @example
+      #   # bad
+      #   assert([], object)
+      #   assert({}, object)
+      #
+      #   # good
+      #   assert_empty(object)
+      #
+      class AssertEmptyLiteral < Cop
+        MSG = 'Prefer using `assert_empty(%<arguments>s)` over ' \
+              '`assert(%<literal>s, %<arguments>s)`.'
+
+        def_node_matcher :assert_with_empty_literal, <<~PATTERN
+          (send nil? :assert ${hash array} $...)
+        PATTERN
+
+        def on_send(node)
+          assert_with_empty_literal(node) do |literal, matchers|
+            args = matchers.map(&:source).join(', ')
+
+            message = format(MSG, literal: literal.source, arguments: args)
+            add_offense(node, message: message)
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,57 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Minitest
+      # This cop enforces the use of `assert_equal(expected, actual)`
+      # over `assert(expected == actual)`.
+      #
+      # @example
+      #   # bad
+      #   assert("rubocop-minitest" == actual)
+      #
+      #   # good
+      #   assert_equal("rubocop-minitest", actual)
+      #
+      class AssertEqual < Cop
+        MSG = 'Prefer using `assert_equal(%<preferred>s)` over ' \
+              '`assert(%<over>s)`.'
+
+        def_node_matcher :assert_equal, <<~PATTERN
+          (send nil? :assert $(send $_ :== $_) $...)
+        PATTERN
+
+        def on_send(node)
+          assert_equal(node) do
+            |first_receiver_arg, expected, actual, rest_receiver_arg|
+
+            message = rest_receiver_arg.first
+            preferred = [expected.source, actual.source, message&.source]
+                        .compact.join(', ')
+            over = [first_receiver_arg.source, message&.source].compact.join(', ')
+
+            offense_message = format(MSG, preferred: preferred, over: over)
+
+            add_offense(node, message: offense_message)
+          end
+        end
+
+        def autocorrect(node)
+          lambda do |corrector|
+            assert_equal(node) do |_receiver, expected, actual, rest_receiver_arg|
+              message = rest_receiver_arg.first
+              replacement = node_arguments(expected, actual, message)
+              corrector.replace(node.loc.expression, "assert_equal(#{replacement})")
+            end
+          end
+        end
+
+        private
+
+        def node_arguments(expected, actual, message)
+          [expected.source, actual.source, message&.source].compact.join(', ')
+        end
+      end
+    end
+  end
+end

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

@@ -3,24 +3,24 @@
 module RuboCop
   module Cop
     module Minitest
-      # Check if your test uses `assert_includes`
-      # instead of `assert(collection.includes?(actual))`.
+      # This cop enforces the test to use `assert_includes`
+      # instead of using `assert(collection.include?(object))`.
       #
       # @example
       #   # bad
-      #   assert(collection.includes?(actual))
-      #   assert(collection.includes?(actual), 'the message')
+      #   assert(collection.include?(object))
+      #   assert(collection.include?(object), 'the message')
       #
       #   # good
-      #   assert_includes(collection, actual)
-      #   assert_includes(collection, actual, 'the message')
+      #   assert_includes(collection, object)
+      #   assert_includes(collection, object, 'the message')
       #
       class AssertIncludes < Cop
         MSG = 'Prefer using `assert_includes(%<arguments>s)` over ' \
               '`assert(%<receiver>s)`.'
 
         def_node_matcher :assert_with_includes, <<~PATTERN
-          (send nil? :assert $(send $_ :includes? $_) $...)
+          (send nil? :assert $(send $_ :include? $_) $...)
         PATTERN
 
         def on_send(node)

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

@@ -0,0 +1,60 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Minitest
+      # This cop enforces the test to use `assert_instance_of(Class, object)`
+      # over `assert(object.instance_of?(Class))`.
+      #
+      # @example
+      #   # bad
+      #   assert(object.instance_of?(Class))
+      #   assert(object.instance_of?(Class), 'the message')
+      #
+      #   # good
+      #   assert_instance_of(Class, object)
+      #   assert_instance_of(Class, object, 'the message')
+      #
+      class AssertInstanceOf < Cop
+        MSG = 'Prefer using `assert_instance_of(%<arguments>s)` over ' \
+              '`assert(%<receiver>s)`.'
+
+        def_node_matcher :assert_with_instance_of, <<~PATTERN
+          (send nil? :assert $(send $_ :instance_of? $_) $...)
+        PATTERN
+
+        def on_send(node)
+          assert_with_instance_of(node) do
+            |first_receiver_arg, object, method, rest_args|
+
+            message = rest_args.first
+            arguments = node_arguments(object, method, message)
+            receiver = [first_receiver_arg.source, message&.source].compact.join(', ')
+
+            offense_message = format(MSG, arguments: arguments, receiver: receiver)
+
+            add_offense(node, message: offense_message)
+          end
+        end
+
+        def autocorrect(node)
+          lambda do |corrector|
+            assert_with_instance_of(node) do |_, object, method, rest_args|
+              message = rest_args.first
+              arguments = node_arguments(object, method, message)
+
+              replacement = "assert_instance_of(#{arguments})"
+              corrector.replace(node.loc.expression, replacement)
+            end
+          end
+        end
+
+        private
+
+        def node_arguments(object, method, message)
+          [method, object, message].compact.map(&:source).join(', ')
+        end
+      end
+    end
+  end
+end