Merge pull request #1279 from tldn0718/fix-skip-model-validation-safe-navigator

koic · web-flow · commit 171e6de001e3 · 2024-05-17T11:58:09.000+09:00
[Fix #1278] Fix a false positivie for Rails/SkipsModelValidations
diff --git a/changelog/fix_safe_navigator_for_skips_model_validation.md b/changelog/fix_safe_navigator_for_skips_model_validation.md
@@ -0,0 +1 @@
+* [#1278](https://github.com/rubocop/rubocop-rails/issues/1278): Fix a false positive for `Rails/SkipsModelValidations` when using `insert` or `insert!` with a safe navigator. ([@tldn0718][])
diff --git a/lib/rubocop/cop/rails/skips_model_validations.rb b/lib/rubocop/cop/rails/skips_model_validations.rb
@@ -63,7 +63,7 @@ class SkipsModelValidations < Base
         PATTERN
 
         def_node_matcher :good_insert?, <<~PATTERN
-          (send _ {:insert :insert!} _ {
+          (call _ {:insert :insert!} _ {
             !(hash ...)
             (hash <(pair (sym !{:returning :unique_by}) _) ...>)
           } ...)
diff --git a/spec/rubocop/cop/rails/skips_model_validations_spec.rb b/spec/rubocop/cop/rails/skips_model_validations_spec.rb
@@ -57,12 +57,14 @@
       it "does not register an offense for #{method} that looks like String#insert" do
         expect_no_offenses(<<~RUBY)
           string.#{method}(0, 'b')
+          string&.#{method}(0, 'b')
         RUBY
       end
 
       it "does not register an offense for #{method} that looks like Array#insert" do
         expect_no_offenses(<<~RUBY)
           array.#{method}(1, :a, :b)
+          array&.#{method}(1, :a, :b)
         RUBY
       end
 

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	+* [#1278](https://github.com/rubocop/rubocop-rails/issues/1278): Fix a false positive for `Rails/SkipsModelValidations` when using `insert` or `insert!` with a safe navigator. ([@tldn0718][])