Merge pull request #8170 from koic/support_autocorrect_for_regexp_as_…

…condition

Support autocorrection for `Lint/RegexpAsCondition`

CHANGELOG.md

@@ -9,6 +9,7 @@
 * [#8148](https://github.com/rubocop-hq/rubocop/pull/8148): Support autocorrection for `Style/MultilineTernaryOperator`. ([@koic][])
 * [#8151](https://github.com/rubocop-hq/rubocop/pull/8151): Support autocorrection for `Style/NestedTernaryOperator`. ([@koic][])
 * [#8142](https://github.com/rubocop-hq/rubocop/pull/8142): Add `Lint/ConstantResolution` cop. ([@robotdana][])
+* [#8170](https://github.com/rubocop-hq/rubocop/pull/8170): Support autocorrection for `Lint/RegexpAsCondition`. ([@koic][])
 
 ### Bug fixes
 

config/default.yml

@@ -1677,6 +1677,7 @@ Lint/RegexpAsCondition:
                  The regexp literal matches `$_` implicitly.
   Enabled: true
   VersionAdded: '0.51'
+  VersionChanged: '0.86'
 
 Lint/RequireParentheses:
   Description: >-

docs/modules/ROOT/pages/cops_lint.adoc

@@ -2524,9 +2524,9 @@ end
 
 | Enabled
 | Yes
-| No
+| Yes
 | 0.51
-| -
+| 0.86
 |===
 
 This cop checks for regexp literals used as `match-current-line`.

lib/rubocop/cop/lint/regexp_as_condition.rb

@@ -23,6 +23,12 @@ class RegexpAsCondition < Cop
         def on_match_current_line(node)
           add_offense(node)
         end
+
+        def autocorrect(node)
+          lambda do |corrector|
+            corrector.replace(node, "#{node.source} =~ $_")
+          end
+        end
       end
     end
   end

spec/rubocop/cop/lint/regexp_as_condition_spec.rb

@@ -5,12 +5,17 @@
 
   let(:config) { RuboCop::Config.new }
 
-  it 'registers an offense for a regexp literal in `if` condition' do
+  it 'registers an offense and corrects for a regexp literal in `if` condition' do
     expect_offense(<<~RUBY)
       if /foo/
          ^^^^^ Do not use regexp literal as a condition. The regexp literal matches `$_` implicitly.
       end
     RUBY
+
+    expect_correction(<<~RUBY)
+      if /foo/ =~ $_
+      end
+    RUBY
   end
 
   it 'does not register an offense for a regexp literal outside conditions' do