Fixed broken test with frozen objects

ruby · Jun 20, 2017 · 23f779f · 23f779f
1 parent 68ef914
commit 23f779f
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/lib/rake/ext/string.rb b/lib/rake/ext/string.rb
@@ -137,7 +137,7 @@ def pathmap_replace(patterns, &block)
     # This String extension comes from Rake
     def pathmap(spec=nil, &block)
       return self if spec.nil?
-      result = ""
+      result = "".dup
       spec.scan(/%\{[^}]*\}-?\d*[sdpfnxX%]|%-?\d+d|%.|[^%]+/) do |frag|
         case frag
         when "%f"

diff --git a/lib/rake/task.rb b/lib/rake/task.rb
@@ -321,7 +321,7 @@ def set_arg_names(args)
     # Return a string describing the internal state of a task.  Useful for
     # debugging.
     def investigation
-      result = "------------------------------\n"
+      result = "------------------------------\n".dup
       result << "Investigating #{name}\n"
       result << "class: #{self.class}\n"
       result <<  "task needed: #{needed?}\n"

diff --git a/test/test_rake_application.rb b/test/test_rake_application.rb
@@ -37,7 +37,7 @@ def test_display_exception_details
 
   def test_display_exception_details_bad_encoding
     begin
-      raise "El Niño is coming!".force_encoding("US-ASCII")
+      raise "El Niño is coming!".dup.force_encoding("US-ASCII")
     rescue => ex
     end
 

diff --git a/test/test_rake_path_map_partial.rb b/test/test_rake_path_map_partial.rb
@@ -3,7 +3,7 @@
 
 class TestRakePathMapPartial < Rake::TestCase
   def test_pathmap_partial
-    @path = "1/2/file"
+    @path = "1/2/file".dup
     def @path.call(n)
       pathmap_partial(n)
     end

diff --git a/test/test_trace_output.rb b/test/test_trace_output.rb
@@ -9,7 +9,7 @@ class PrintSpy
     attr_reader :result, :calls
 
     def initialize
-      @result = ""
+      @result = "".dup
       @calls = 0
     end