sevencop 0.9.3 → 0.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3d3ab099568a85aea228ce2fcf282f4e0bc7a2e2c6f3315191c2e9e14367a173
-  data.tar.gz: aeba9572d3eeb74d5d4da5473b30b62d291b0e01d8f5644f041ca41ecd43994f
+  metadata.gz: ee9785d463d0e83d3f55dc1554e250e826a9cbd5aa6aacb510f11a4bdfb1947a
+  data.tar.gz: 9ca8456b525226ba8a599c4ef87f08ea208000268fa73ccf9f1501531682d294
 SHA512:
-  metadata.gz: 643b778e5c44256e590b1a7344825f46c1057a18d3e424076583b522a21f957ca1acf37e2ba6f59d58b72a66e929e2aada551c627cf039d065dbea2646d4e485
-  data.tar.gz: 723e43f5d54dab22c1be3d02bb1074d6899783aec74c64c1fa6ead9294b01ffde02d5aae1b1792e483082d5d297f784de393a20285d34a303e892febc2522ae6
+  metadata.gz: edc7f9620eaa81224e47c8e8785df67bc29c51a0c6969f30bd24a3d6869a64be26686ccce7c3b43c89bc89f06b9283b67030587c86eb0b7ef5598d42ee1dc90b
+  data.tar.gz: 9a3bf2ecc2eed54ae8dd6207d0f0cd93c13f577bbcab69cdc9992e42f87687e416d9612136f1483952cfb51bdf22890727b14d6d66653e9294732dcb1da8d389

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    sevencop (0.9.3)
+    sevencop (0.10.0)
       rubocop
 
 GEM

data/README.md

@@ -67,6 +67,25 @@ belongs_to :group, options
 
 This is useful for migration of `config.active_record.belongs_to_required_by_default`.
 
+### Sevencop/FactoryBotCreateList
+
+Finds possible substitutions for `FactoryBot.create_list`.
+
+```ruby
+# bad
+Array.new(2) do
+  create(:user)
+end
+
+# good
+create_list(:user, 2)
+
+# good
+Array.new(2) do |i|
+  create(:user, order: i)
+end
+```
+
 ### Sevencop/HashLiteralOrder
 
 Sort Hash literal entries by key.

data/config/default.yml

@@ -5,6 +5,12 @@ Sevencop/BelongsToOptional:
   Safe: false
   VersionAdded: '0.5'
 
+Sevencop/FactoryBotCreateList:
+  Description: |
+    Finds possible substitutions for `FactoryBot.create_list`.
+  Enabled: false
+  VersionAdded: '0.10'
+
 Sevencop/HashLiteralOrder:
   Description: |
     Sort Hash literal entries by key.

data/lib/rubocop/cop/sevencop/factory_bot_create_list.rb

@@ -0,0 +1,94 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Sevencop
+      # Finds possible substitutions for `FactoryBot.create_list`.
+      #
+      # @example
+      #
+      #   # bad
+      #   Array.new(2) do
+      #     create(:user)
+      #   end
+      #
+      #   # good
+      #   create_list(:user, 2)
+      #
+      #   # good
+      #   Array.new(2) do |i|
+      #     create(:user, order: i)
+      #   end
+      #
+      # @note
+      #   This cop does not support `Integer#times` intentionally because it
+      #   should be treated by `Performance/TimesMap` cop.
+      #
+      class FactoryBotCreateList < Base
+        extend AutoCorrector
+
+        MSG = 'Use `create_list` instead.'
+
+        # @param node [RuboCop::AST::BlockNode]
+        # @return [void]
+        def on_block(node)
+          count_node, factory_name_node, extra_argument_nodes = extract(node)
+          return unless count_node
+
+          add_offense(node) do |corrector|
+            corrector.replace(
+              node,
+              format(
+                'create_list(%<arguments>s)',
+                arguments: [
+                  factory_name_node,
+                  count_node,
+                  *extra_argument_nodes
+                ].map(&:source).join(', ')
+              )
+            )
+          end
+        end
+
+        private
+
+        # @!method extract(node)
+        #   @param node [RuboCop::AST::BlockNode]
+        #   @return [Array(Integer, RuboCop::AST::SendNode)]
+        def_node_matcher :extract, <<~PATTERN
+          (block
+            (send
+              (const {nil? | cbase} :Array)
+              :new
+              $(int _)
+            )
+            (args)
+            (send
+              nil?
+              :create
+              $(sym _) $(...)?
+            )
+          )
+        PATTERN
+
+        # @param corrector [RuboCop::Cop::Corrector]
+        # @param node [RuboCop::AST::BlockNode]
+        # @return [void]
+        def autocorrect(corrector, node)
+          count_node, factory_name_node, extra_argument_nodes = extract(node)
+          corrector.replace(
+            node,
+            format(
+              'create_list(%<arguments>s)',
+              arguments: [
+                factory_name_node,
+                count_node,
+                *extra_argument_nodes
+              ].map(&:source).join(', ')
+            )
+          )
+        end
+      end
+    end
+  end
+end

data/lib/sevencop/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Sevencop
-  VERSION = '0.9.3'
+  VERSION = '0.10.0'
 end

data/lib/sevencop.rb

@@ -1,12 +1,10 @@
 # frozen_string_literal: true
 
-require 'pathname'
-require 'yaml'
-
 require_relative 'sevencop/rubocop_extension'
 require_relative 'sevencop/version'
 
 require_relative 'rubocop/cop/sevencop/belongs_to_optional'
+require_relative 'rubocop/cop/sevencop/factory_bot_create_list'
 require_relative 'rubocop/cop/sevencop/hash_literal_order'
 require_relative 'rubocop/cop/sevencop/inferred_spec_type'
 require_relative 'rubocop/cop/sevencop/order_field'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sevencop
 version: !ruby/object:Gem::Version
-  version: 0.9.3
+  version: 0.10.0
 platform: ruby
 authors:
 - Ryo Nakamura
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-08-27 00:00:00.000000000 Z
+date: 2022-08-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -42,6 +42,7 @@ files:
 - Rakefile
 - config/default.yml
 - lib/rubocop/cop/sevencop/belongs_to_optional.rb
+- lib/rubocop/cop/sevencop/factory_bot_create_list.rb
 - lib/rubocop/cop/sevencop/hash_literal_order.rb
 - lib/rubocop/cop/sevencop/inferred_spec_type.rb
 - lib/rubocop/cop/sevencop/order_field.rb