Merge pull request #6807 from koic/fix_incorrect_autocorrect_for_rail…

…s_validation

[Fix #6800] Fix an incorrect auto-correct for `Rails/Validation`

CHANGELOG.md

@@ -12,6 +12,7 @@
 * [#6803](https://github.com/rubocop-hq/rubocop/pull/6803): Fix error for `Style/NumericLiterals` on a literal that contains spaces. ([@pocke][])
 * [#6801](https://github.com/rubocop-hq/rubocop/pull/6801): Fix auto-correction for `Style/Lambda` with no-space argument. ([@pocke][])
 * [#6804](https://github.com/rubocop-hq/rubocop/pull/6804): Fix auto-correction of `Style/NumericLiterals` on numeric literal with exponent. ([@pocke][])
+* [#6800](https://github.com/rubocop-hq/rubocop/issues/6800): Fix an incorrect auto-correct for `Rails/Validation` when method arguments are enclosed in parentheses. ([@koic][])
 
 ### Changes
 

lib/rubocop/cop/rails/validation.rb

@@ -87,8 +87,9 @@ def correct_validate_type(corrector, node)
               "#{validate_type}: #{braced_options(last_argument)}"
             )
           else
-            corrector.insert_after(node.loc.expression,
-                                   ", #{validate_type}: true")
+            range = last_argument.source_range
+
+            corrector.insert_after(range, ", #{validate_type}: true")
           end
         end
 

spec/rubocop/cop/rails/validation_spec.rb

@@ -45,6 +45,19 @@
 
         include_examples 'auto-corrects'
       end
+
+      context "with validates_#{type}_of " \
+              'when method arguments are enclosed in parentheses' do
+        let(:auto_corrected_source) do
+          "validates(:full_name, :birth_date, #{type}: true)"
+        end
+
+        let(:source) do
+          "validates_#{type}_of(:full_name, :birth_date)"
+        end
+
+        include_examples 'auto-corrects'
+      end
     end
 
     context 'with single attribute name' do