momocop 0.1.12 → 0.1.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: acef3c7f38f87034b619c4b5e9ca7ca77ac880a0a665f46abff3a19455ef2d5f
-  data.tar.gz: 4e06a90596825134bdad568b41b38ebdb93056d0fab4f58d381c8ffd6d16e6e5
+  metadata.gz: 220069b3119b5c48271dcd6881e4a1530074eabbc697d12dca9ea70cf5dbbbe8
+  data.tar.gz: 02fcf5cad166113116cb7876aa46f1ffa0de65391488901727286711645a4917
 SHA512:
-  metadata.gz: 7c18582cdb50926ee4bbc7074c076d90f06504cdec64d0dc4a1389dadac1ad000864cdcea4e9e4322a7fcc2bc6bbb8aadd381a29cef2e4bdee1dee3f751833a9
-  data.tar.gz: 6ca756d9793ca03ef4ca8ea88e37d9169bea071641464af88f36e96688a613fa5dc09c3e50b252c56e79e1cf9376f46db1717e0b4dfa7d46be8352fbb656582a
+  metadata.gz: 2ae69bdfff4429dc93e5e0a78fa4d87afb29bdf5d3bec1bffc3aec97b569066d47e2f0ba215593d0599cbc781591ea38b44dcf128cffbda503ba4b03ced79224
+  data.tar.gz: 22b9e10997ec1a2f8f32d33e503285a7f0ca6809d3c7081713b481c8ba245753d8ed944e57d2032f9f3a243894dbd16018f4d8cff7dcb24b55bf3144280d4813

data/lib/momocop/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Momocop
-  VERSION = '0.1.12'
+  VERSION = '0.1.13'
 end

data/lib/rubocop/cop/momocop/factory_bot_inline_association.rb

@@ -0,0 +1,61 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Momocop
+      # Enforces inline association definitions in FactoryBot factories.
+      #
+      # @example
+      #   # bad
+      #   factory :blog_post do
+      #     association(:user)
+      #   end
+      #
+      #   # good
+      #   factory :blog_post do
+      #     user { association :user }
+      #   end
+      class FactoryBotInlineAssociation < RuboCop::Cop::Base
+        extend AutoCorrector
+
+        MSG = 'Use inline association definition instead of separate `association` method call.'
+
+        RESTRICT_ON_SEND = %i[association].freeze
+
+        def_node_matcher :association_call?, <<-PATTERN
+          (send nil? :association (sym _))
+        PATTERN
+
+        def on_send(node)
+          return unless inside_factory_bot_define?(node)
+          return unless inside_factory_bot_factory?(node)
+
+          add_offense(node.loc.selector, message: MSG) do |corrector|
+            association_name = node.arguments.first.value
+            options = node.arguments.drop(1).map(&:source).join(', ')
+
+            replacement =
+              if options.empty?
+                "#{association_name} { association :#{association_name} }"
+              else
+                "#{association_name} { association :#{association_name}, #{options} }"
+              end
+            corrector.replace(node, replacement)
+          end
+        end
+
+        private def inside_factory_bot_factory?(node)
+          context = node.each_ancestor(:block).first
+          send_node = context.block_type? ? context.send_node : context
+
+          return send_node.method_name == :factory
+        end
+
+        private def inside_factory_bot_define?(node)
+          ancestors = node.each_ancestor(:block).to_a
+          ancestors.any? { |ancestor| ancestor.method_name == :define && ancestor.receiver&.const_name == 'FactoryBot' }
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/momocop/factory_bot_missing_associations.rb

@@ -117,7 +117,7 @@ module RuboCop
         end
 
         private def generate_association_definition(property)
-          "association(:#{property})"
+          "#{property} { association :#{property} }"
         end
 
         private def inside_factory_bot_define?(node)

data/lib/rubocop/cop/momocop/factory_bot_property_order.rb

@@ -116,8 +116,17 @@ module RuboCop
 
         private def definition_type(node)
           send_node = node.send_type? ? node : node.children.first
+          has_association_body =
+            send_node
+            .block_node
+            &.children
+            &.last
+            &.children
+            &.any? { _1.is_a?(Parser::AST::Node) && _1.send_type? && _1.method_name == :association }
           if %i[association sequence].include? send_node.method_name
             send_node.method_name
+          elsif has_association_body
+            :association
           else
             :property
           end

data/lib/rubocop/cop/momocop/factory_bot_singular_factory_name.rb

@@ -33,7 +33,6 @@ module RuboCop
           return if factory_name.singularize == factory_name
 
           add_offense(node.first_argument) do |corrector|
-            p node.first_argument
             if node.first_argument.type == :sym
               corrector.replace(node.first_argument.loc.expression, ":#{factory_name.singularize}")
             elsif node.first_argument.type == :str

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: momocop
 version: !ruby/object:Gem::Version
-  version: 0.1.12
+  version: 0.1.13
 platform: ruby
 authors:
 - supermomonga
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-02-22 00:00:00.000000000 Z
+date: 2024-02-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -93,6 +93,7 @@ files:
 - lib/momocop/version.rb
 - lib/rubocop/cop/momocop/factory_bot_class_existence.rb
 - lib/rubocop/cop/momocop/factory_bot_consistent_file_name.rb
+- lib/rubocop/cop/momocop/factory_bot_inline_association.rb
 - lib/rubocop/cop/momocop/factory_bot_missing_associations.rb
 - lib/rubocop/cop/momocop/factory_bot_missing_class_option.rb
 - lib/rubocop/cop/momocop/factory_bot_missing_properties.rb