Merge pull request #708 from scambra/patch-1

Fix for #707
rails · Jun 8, 2021 · efe79b5 · efe79b5
2 parents dcc2bf3 + baf7e46
commit efe79b5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/lib/thor/actions/inject_into_file.rb b/lib/thor/actions/inject_into_file.rb
@@ -106,15 +106,14 @@ def say_status(behavior, warning: nil, color: nil)
       # Adds the content to the file.
       #
       def replace!(regexp, string, force)
-        return if pretend?
         content = File.read(destination)
         before, after = content.split(regexp, 2)
         snippet = (behavior == :after ? after : before).to_s
 
         if force || !snippet.include?(replacement)
           success = content.gsub!(regexp, string)
 
-          File.open(destination, "wb") { |file| file.write(content) }
+          File.open(destination, "wb") { |file| file.write(content) } unless pretend?
           success
         end
       end

diff --git a/spec/actions/inject_into_file_spec.rb b/spec/actions/inject_into_file_spec.rb
@@ -72,6 +72,11 @@ def file
       expect(invoke!("doc/README", "\nmore content", :after => "__start__")).to eq("      insert  doc/README\n")
     end
 
+    it "logs status if pretending" do
+      invoker(:pretend => true)
+      expect(invoke!("doc/README", "\nmore content", :after => "__start__")).to eq("      insert  doc/README\n")
+    end
+
     it "does not change the file if pretending" do
       invoker :pretend => true
       invoke! "doc/README", "\nmore content", :after => "__start__"