Merge pull request #10785 from koic/fix_a_false_negative_for_style_re…

…dundant_parentheses

Fix a false negative for `Style/RedundantParentheses`

changelog/fix_a_false_negative_for_style_redundant_parentheses.md

@@ -0,0 +1 @@
+* [#10785](https://github.com/rubocop/rubocop/pull/10785): Fix a false negative for `Style/RedundantParentheses` when parens around a receiver of a method call with an argument. ([@koic][])

lib/rubocop/cop/style/redundant_parentheses.rb

@@ -81,7 +81,8 @@ def allowed_multiple_expression?(node)
         end
 
         def like_method_argument_parentheses?(node)
-          node.send_type? && node.arguments.size == 1 && !node.arithmetic_operation?
+          node.send_type? && node.arguments.one? &&
+            !node.arithmetic_operation? && node.first_argument.begin_type?
         end
 
         def empty_parentheses?(node)

spec/rubocop/cop/style/redundant_parentheses_spec.rb

@@ -125,6 +125,17 @@
   it_behaves_like 'plausible', '+(1.foo.bar)'
   it_behaves_like 'plausible', '()'
 
+  it 'registers an offense for parens around a receiver of a method call with an argument' do
+    expect_offense(<<~RUBY)
+      (x).y(z)
+      ^^^ Don't use parentheses around a method call.
+    RUBY
+
+    expect_correction(<<~RUBY)
+      x.y(z)
+    RUBY
+  end
+
   it 'registers an offense for parens around an interpolated expression' do
     expect_offense(<<~RUBY)
       "\#{(foo)}"