Merge pull request rubocop#11863 from koic/fix_an_incorrect_autocorre…

…ct_for_style_guard_clause

[Fix rubocop#11862] Fix an incorrect autocorrect for `Style/GuardClause`

changelog/fix_an_incorrect_autocorrect_for_style_guard_clause.md

@@ -0,0 +1 @@
+* [#11862](https://github.com/rubocop/rubocop/issues/11862): Fix an incorrect autocorrect for `Style/GuardClause` when using `raise` in `else` branch in a one-liner with `then`. ([@koic][])

lib/rubocop/cop/style/guard_clause.rb

@@ -187,6 +187,8 @@ def autocorrect(corrector, node, condition, replacement, guard)
           if_branch = node.if_branch
           else_branch = node.else_branch
 
+          corrector.replace(node.loc.begin, "\n") if node.loc.begin&.is?('then')
+
           if if_branch&.send_type? && heredoc?(if_branch.last_argument)
             autocorrect_heredoc_argument(corrector, node, if_branch, else_branch, guard)
           elsif else_branch&.send_type? && heredoc?(else_branch.last_argument)

spec/rubocop/cop/style/guard_clause_spec.rb

@@ -165,6 +165,18 @@ def func
     expect_no_corrections
   end
 
+  it 'registers an offense when using `raise` in `else` branch in a one-liner with `then`' do
+    expect_offense(<<~RUBY)
+      if something then work else raise('message') end
+      ^^ Use a guard clause (`raise('message') unless something`) instead of wrapping the code inside a conditional expression.
+    RUBY
+
+    expect_correction(<<~RUBY)
+      raise('message') unless something#{trailing_whitespace}
+       work#{trailing_whitespace * 3}
+    RUBY
+  end
+
   it 'registers an offense when using `and return` in `then` branch' do
     expect_offense(<<~RUBY)
       def func