Merge pull request #574 from yhirano55/fix_pretend_handling

Fix bug and refactoring

Author: rafaelfranca

lib/thor/actions/inject_into_file.rb

@@ -53,10 +53,12 @@ def invoke!
           replacement + '\0'
         end
 
-        if File.exist?(destination)
+        if exists?
           replace!(/#{flag}/, content, config[:force])
         else
-          raise Thor::Error, "The file #{ destination } does not appear to exist"
+          unless pretend?
+            raise Thor::Error, "The file #{ destination } does not appear to exist"
+          end
         end
       end
 
@@ -95,7 +97,7 @@ def say_status(behavior)
       # Adds the content to the file.
       #
       def replace!(regexp, string, force)
-        return if base.options[:pretend]
+        return if pretend?
         content = File.read(destination)
         if force || !content.include?(replacement)
           content.gsub!(regexp, string)

spec/actions/inject_into_file_spec.rb

@@ -73,13 +73,23 @@ def file
 
     it "does not attempt to change the file if it doesn't exist - instead raises Thor::Error" do
       expect do
-	invoke! "idontexist", :before => "something" do
-	  "any content"
-	end
+        invoke! "idontexist", :before => "something" do
+          "any content"
+        end
       end.to raise_error(Thor::Error, /does not appear to exist/)
       expect(File.exist?("idontexist")).to be_falsey
     end
 
+    it "does not attempt to change the file if it doesn't exist and pretending" do
+      expect do
+        invoker :pretend => true
+        invoke! "idontexist", :before => "something" do
+          "any content"
+        end
+      end.not_to raise_error
+      expect(File.exist?("idontexist")).to be_falsey
+    end
+
     it "does change the file if already includes content and :force is true" do
       invoke! "doc/README", :before => "__end__" do
         "more content\n"