Merge pull request #418 from koic/support_optimized_string_dup_for_pe…

…rformance_unfreeze_string [Fix #384] Support optimized `String#dup` for `Performance/UnfreezeString`
rubocop · Dec 7, 2023 · bf85c6d · bf85c6d
2 parents 76911d0 + 7ac5b60
commit bf85c6d
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 13 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -76,7 +76,7 @@ jobs:
           sed -e "/gem 'rubocop', github: 'rubocop\/rubocop'/d" \
               -e "/gem 'rubocop-rspec',/d" -i Gemfile
           cat << EOF > Gemfile.local
-            gem 'rubocop', '1.30.0' # Specify the oldest supported RuboCop version
+            gem 'rubocop', '1.48.1' # Specify the oldest supported RuboCop version
           EOF
       - name: set up Ruby
         uses: ruby/setup-ruby@v1

diff --git a/changelog/new_optimized_string_dup_for_performance_unfreeze_string.md b/changelog/new_optimized_string_dup_for_performance_unfreeze_string.md
@@ -0,0 +1 @@
+* [#384](https://github.com/rubocop/rubocop-performance/issues/384): Support optimized `String#dup` for `Performance/UnfreezeString` when Ruby 3.3+. ([@koic][])
diff --git a/lib/rubocop/cop/performance/unfreeze_string.rb b/lib/rubocop/cop/performance/unfreeze_string.rb
@@ -15,8 +15,8 @@ module Performance
       #
       # @example
       #   # bad
-      #   ''.dup
-      #   "something".dup
+      #   ''.dup          # when Ruby 3.2 or lower
+      #   "something".dup # when Ruby 3.2 or lower
       #   String.new
       #   String.new('')
       #   String.new('something')
@@ -45,7 +45,7 @@ class UnfreezeString < Base
         PATTERN
 
         def on_send(node)
-          return unless dup_string?(node) || string_new?(node)
+          return unless (dup_string?(node) && target_ruby_version <= 3.2) || string_new?(node)
 
           add_offense(node) do |corrector|
             string_value = "+#{string_value(node)}"

diff --git a/rubocop-performance.gemspec b/rubocop-performance.gemspec
@@ -30,6 +30,6 @@ Gem::Specification.new do |s|
     'rubygems_mfa_required' => 'true'
   }
 
-  s.add_runtime_dependency('rubocop', '>= 1.30.0', '< 2.0')
+  s.add_runtime_dependency('rubocop', '>= 1.48.1', '< 2.0')
   s.add_runtime_dependency('rubocop-ast', '>= 1.30.0', '< 2.0')
 end
diff --git a/spec/rubocop/cop/performance/unfreeze_string_spec.rb b/spec/rubocop/cop/performance/unfreeze_string_spec.rb
@@ -1,15 +1,25 @@
 # frozen_string_literal: true
 
 RSpec.describe RuboCop::Cop::Performance::UnfreezeString, :config do
-  it 'registers an offense and corrects for an empty string with `.dup`' do
-    expect_offense(<<~RUBY)
-      "".dup
-      ^^^^^^ Use unary plus to get an unfrozen string literal.
-    RUBY
+  context 'when Ruby <= 3.2', :ruby32 do
+    it 'registers an offense and corrects for an empty string with `.dup`' do
+      expect_offense(<<~RUBY)
+        "".dup
+        ^^^^^^ Use unary plus to get an unfrozen string literal.
+      RUBY
 
-    expect_correction(<<~RUBY)
-      +""
-    RUBY
+      expect_correction(<<~RUBY)
+        +""
+      RUBY
+    end
+  end
+
+  context 'when Ruby >= 3.3', :ruby33 do
+    it 'does not register an offense and corrects for an empty string with `.dup`' do
+      expect_no_offenses(<<~RUBY)
+        "".dup
+      RUBY
+    end
   end
 
   it 'registers an offense and corrects for a string with `.dup`' do