Merge pull request #505 from vcavallo/feature/dont-replace-nonexistant-files

Feature/dont replace nonexistant files

Author: rafaelfranca

lib/thor/actions/inject_into_file.rb

@@ -53,7 +53,11 @@ def invoke!
           replacement + '\0'
         end
 
-        replace!(/#{flag}/, content, config[:force])
+        if File.exist?(destination)
+          replace!(/#{flag}/, content, config[:force])
+        else
+          raise Thor::Error, "The file #{ destination } does not appear to exist"
+        end
       end
 
       def revoke!

spec/actions/inject_into_file_spec.rb

@@ -71,6 +71,15 @@ def file
       expect(File.read(file)).to eq("__start__\nREADME\nmore content\n__end__\n")
     end
 
+    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
+      end.to raise_error(Thor::Error, /does not appear to exist/)
+      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"