Merge pull request rubocop#12604 from koic/fix_a_false_positive_for_s…

…tyle_multiline_ternary_operator

[Fix rubocop#12603] Fix an infinite loop error for `Style/MultilineTernaryOperator`

changelog/fix_a_false_positive_for_style_multiline_ternary_operator.md

@@ -0,0 +1 @@
+* [#12603](https://github.com/rubocop/rubocop/issues/12603): Fix an infinite loop error for `Style/MultilineTernaryOperator` when using a method call as a ternary operator condition with a line break between receiver and method. ([@koic][])

lib/rubocop/cop/style/multiline_ternary_operator.rb

@@ -54,12 +54,10 @@ def on_if(node)
         private
 
         def offense?(node)
-          node.ternary? && node.multiline?
+          node.ternary? && node.multiline? && node.source != replacement(node)
         end
 
         def autocorrect(corrector, node)
-          return unless offense?(node)
-
           corrector.replace(node, replacement(node))
           return unless (parent = node.parent)
           return unless (comments_in_condition = comments_in_condition(node))

spec/rubocop/cli/autocorrect_spec.rb

@@ -2971,6 +2971,21 @@ module Foo#{trailing_whitespace}
     RUBY
   end
 
+  it 'does not cause an infinite loop error for `Style/MultilineTernaryOperator`' do
+    source_file = Pathname('example.rb')
+    create_file(source_file, <<~RUBY)
+      do_something(arg
+                     .foo ? bar : baz)
+    RUBY
+
+    status = cli.run(%w[--autocorrect-all --only Style/MultilineTernaryOperator])
+    expect(status).to eq(0)
+    expect(source_file.read).to eq(<<~RUBY)
+      do_something(arg
+                     .foo ? bar : baz)
+    RUBY
+  end
+
   it 'respects `Lint/ConstantResolution` over `Style/RedundantConstantBase` when enabling`Lint/ConstantResolution`' do
     source_file = Pathname('example.rb')
     create_file(source_file, <<~RUBY)

spec/rubocop/cop/style/multiline_ternary_operator_spec.rb

@@ -124,6 +124,15 @@
     RUBY
   end
 
+  it 'does not register an offense when using a method call as a ternary operator condition with a line break ' \
+     'between receiver and method' do
+    # NOTE: Redundant line break is corrected by `Layout/RedundantLineBreak`.
+    expect_no_offenses(<<~RUBY)
+      do_something(arg
+                     .foo ? bar : baz)
+    RUBY
+  end
+
   it 'registers an offense and corrects when returning a multiline ternary operator expression with `return`' do
     expect_offense(<<~RUBY)
       return cond ?