reciper 0.0.4 → 0.1.0

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    reciper (0.0.3)
+    reciper (0.1.0)
 
 GEM
   remote: https://rubygems.org/

data/README.md

@@ -0,0 +1,25 @@
+# Reciper
+
+Suppose you're writing a book containing some programming recipes. It would be great to have a test suite that ensures that your snippets really are really working. But why we should write a test suite if RSpec already does an awesome job doing it ?
+
+Reciper is a collection of helpers that helps you write tests for your book recipes. It should be included on your described class and used whenever you need to copy files, copy line ranges, run tests, overwrite files, among other things.
+
+## Install
+
+On your `Gemfile`, just do this:
+
+```ruby
+gem "reciper"
+```
+     
+Run `bundle install` and on your described class just include `Reciper::Helpers`
+
+```ruby
+describe "My awesome recipe" do
+  include Reciper::Helpers
+  # (...)
+end
+```
+
+        
+

data/Rakefile

@@ -1,2 +1,10 @@
 require 'bundler'
-Bundler::GemHelper.install_tasks
+Bundler::GemHelper.install_tasks
+
+require 'rspec/core/rake_task'
+
+RSpec::Core::RakeTask.new(:spec) do |t|
+  t.rspec_opts = "--order rand -f d"
+end
+
+task :default => :spec

data/lib/reciper/helpers.rb

@@ -7,102 +7,177 @@ module Reciper
   class NoFileToBeOverriden < RuntimeError
   end
 
+  class NoFileOrMultipleFilesFound < RuntimeError
+  end
+
   module Helpers
+    # Copies the file from recipe to ruby_app. It is reversible.
+    #
+    # filename - The file to be copied relative to the recipe_path
+    # options - The hash options to configure the copy (default: {}):
+    #         :as - The copy name (default: the file name).
+    #         :to - The destination dir relative to the ruby_app_path. If the directory doesn't exists, it will be created (default: ruby_app_path root path).
+    #
+    # Examples
+    #
+    #   copy_file("a.rb")
+    #   copy_file("a.rb", :to => "app/models", :as => "person.rb")
+    #
+    # Returns nothing.
     def copy_file(filename, options={})
-      destination_dir = @ruby_app_path + "/" + (options[:to] || "")
+      destination_file_name = options[:as] || filename
+      destination_dir = options[:to] || ""
 
-      unless File.directory?(destination_dir)
-        FileUtils.mkdir_p(destination_dir)
-      end
+      destination = File.join(destination_dir , destination_file_name)
+      global_destination = File.join(@ruby_app_path, destination)
 
-      FileUtils.cp(@recipe_path + "/" + filename, destination_dir)
+      create_directory_if_not_exists(File.join(@ruby_app_path, destination_dir))
 
-      new_filename = options[:as] || filename
-
-      if(options[:as])
-        FileUtils.mv(destination_dir + "/" + filename, destination_dir + "/" + new_filename)
-      end
+      FileUtils.cp(File.join(@recipe_path, filename), global_destination)
 
-      @operations << [:copy, (options[:to] || "") + new_filename]
+      @operations << [:copy_file, { :destination => destination }]
     end
 
-    def run_tests(options={})
-      Dir.chdir(@ruby_app_path) do
-        response = `bundle exec rspec spec`
-
-        if response =~ /([.FE*]+)/
-          $1.split("").reject { |char| char == "." }.size
-        else
-          puts "Can't get any test output"
-          fail NoTestOutput
-        end
+    # Run the tests on the ruby app. It is NOT reversible.
+    #
+    # Examples
+    #
+    #  run_tests()
+    #  # => 2
+    #
+    # Returns the number of failures
+    def run_tests
+      result = run_command("bundle exec rspec spec")
+
+      if result[:response] =~ /([\.FE*]+)/
+        $1.split("").reject { |char| (char == "." || char == "*") }.size
+      else
+        puts "Can't get any test output"
+        fail NoTestOutput
       end
     end
 
+    # Run a rake task on the ruby app. It is NOT reversible.
+    #
+    # task - the desired task
+    #
+    # Examples
+    #
+    #  rake_task("db:migrate")
+    #  # => { :successful => true, :response => ""}
+    #  rake_task("db:migrate")
+    #  # => { :successful => false, :response => "Couldn't find the specified DB"}
+    #
+    # Returns a command execution hash
     def run_rake_task(task)
-      Dir.chdir(@ruby_app_path) do
-        spawn("bundle exec rake #{task}", :out => "/dev/null", :err => "/dev/null")
-
-        Process.wait
-      end
-
-      $?.exitstatus == 0
+      run_command("bundle exec rake #{task}")
     end
 
+    # Copies a range of lines from a file on the recipe path to a file on the ruby app path. It is reversible.
+    #
+    # from - the file, relative to the recipe path, that contains the file with the lines that will be copied
+    # to - the file, relative to the ruby app path, that contains the file that will receive the files. You can also specify a suffix like *.rb and if it matches only one file, it will insert the lines to it, otherwise it will raise a NoFileOrMultipleFilesFound exception
+    # options - some options related to the line copy (default: {})
+    #         :to_line - The line where the content will be inserted (default: raises an exception)
+    #         :lines - A range that specifies the lines that will be copied (default: whole file (0..-1))
+    #
+    # Examples
+    #
+    #  copy_line_range("a.rb", "app/models/person.rb", :to_line => 42)
+    #  copy_line_range("a.rb", "app/models/person.rb", :to_line => 42, :lines => (4..10))
+    #  copy_line_range("a.rb", "db/migrate/*create_users.rb", :to_line => 10, :lines => (1..5))
+    #
+    # Returns nothing.
     def copy_line_range(from, to, options={})
-      if options[:from_lines]
-        from_lines = Range.new(options[:from_lines].first - 1, options[:from_lines].last - 1)
-      else
-        from_lines = (0..-1)
-      end
+      original_file = filename_from_suffix(to)
 
-      from_file_lines = File.open(@recipe_path + "/" + from, "r").readlines
-      output_lines = File.read(@ruby_app_path + "/" + to).split("\n")
-      original_output = output_lines.dup
-      to_file_output = File.open(@ruby_app_path + "/" + to, "w")
+      original_content = File.read(original_file)
+      original = original_content.split("\n")
 
-      to_output = output_lines.insert(options[:to_line] - 1, from_file_lines.map(&:chomp).slice(from_lines)).flatten!.join("\n")
+      new_content = File.read(File.join(@recipe_path, from)).split("\n")
 
-      to_file_output.write(to_output)
+      range = options[:lines] || (0..-1)
 
-      to_file_output.close
+      original.insert(options[:to_line], new_content[range])
 
-      @operations << [:copy_range, to, original_output.join("\n")]
+      File.open(original_file, "w") do |file|
+        file.write(original.flatten.join("\n"))
+      end
+
+      @operations << [:copy_line_range, { :original_content => original_content, :original_file => original_file }]
     end
 
+    # Does the rollback on all reversible operations.
+    #
+    # Examples
+    #
+    #  rollback
+    #
+    # Returns nothing
     def rollback
       @operations.reverse.each do |operation|
-        if operation[0] == :copy
-          FileUtils.rm(@ruby_app_path + "/" + operation[1])
-        elsif operation[0] == :copy_range
-          File.open(@ruby_app_path + "/" + operation[1], "w") { |file| file.write(operation[2]) }
-        elsif operation[0] == :run_command
-          spawn(operation[1]) if operation[1]
-
-          Process.wait
-        elsif operation[0] == :override_file
-          FileUtils.cp(operation[1], @ruby_app_path + "/" + operation[2])
+        operation_name, arguments = operation
+
+        case operation_name
+        when :copy_file
+          then remove_file(arguments[:destination])
+        when :copy_range
+          then write_content_to_file(arguments[:original_file], arguments[:original_content])
+        when :run_command
+          then run_command(arguments[:rollback_command]) if arguments[:rollback_command]
+        when :override_file
+          then FileUtils.cp(arguments[:tmp_file], File.join(@ruby_app_path, arguments[:overriden_file]))
         end
       end
     end
 
+    # Runs a command using bundle exec on the ruby app path. If you specify a rollback command, it is reversible.
+    #
+    #  command - the command. It will run inside a bundle exec.
+    #  rollback_command - the command which rollbacks the command (default: nil)
+    #
+    # Examples
+    #
+    #  run_command("ls")
+    #  # => { :successful => true, response => "file.rb\n\file2.rb" }
+    #  run_command("rails g model user", "rails d model user")
+    #  # => { :successful => true, response => "model was generated" }
+    #  run_command("cp a.rb b.rb")
+    #  # => { :successful => false, response => "file a.rb doesn't exists" }
+    #
+    # Returns a command execution hash
     def run_command(command, rollback_command=nil)
-      Dir.chdir(@ruby_app_path) do
-        spawn("bundle exec #{command}", :out => "/dev/null", :err => "/dev/null")
+      response = ""
+      successful = ""
 
-        Process.wait
-      end
+      run_on_app_path do
+        IO.popen(command) do |io|
+          response = io.read
+        end
 
-      if $?.exitstatus == 0
-        @operations << [:run_command, rollback_command || nil]
-        true
-      else
-        false
+        successful = ($?.exitstatus == 0)
       end
+
+      @operations << [:run_command, { :rollback_command => rollback_command }]
+
+      {
+        :successful => successful,
+        :response => response
+      }
     end
 
+    # Overrides a file of the ruby app with a file from the recipe path. It is reversible.
+    #
+    #  file - The file that will override relative to the recipe path
+    #  file_to_be_overriden - The file that will be overriden
+    #
+    # Examples
+    #
+    #  override_file("a.rb", "app/controller/application_controller.rb")
+    #
+    # Returns nothing
     def override_file(file, file_to_be_overriden)
-      Dir.chdir(@ruby_app_path) do
+      run_on_app_path do
         fail NoFileToBeOverriden unless File.exists?(file_to_be_overriden)
 
         FileUtils.mkdir_p("/tmp/reciper")
@@ -111,10 +186,40 @@ module Reciper
 
         FileUtils.cp(file_to_be_overriden, tmp_file)
 
-        @operations << [:override_file, tmp_file, file_to_be_overriden]
+        @operations << [:override_file, { :tmp_file => tmp_file, :overriden_file => file_to_be_overriden }]
       end
 
-      FileUtils.cp(@recipe_path + "/" + file, @ruby_app_path + "/" + file_to_be_overriden)
+      FileUtils.cp(File.join(@recipe_path,file), File.join(@ruby_app_path, file_to_be_overriden))
+    end
+
+    private
+
+    def filename_from_suffix(suffix)
+      files = Dir.glob(File.join(@ruby_app_path, suffix))
+      fail NoFileOrMultipleFilesFound if files.size != 1
+      files.first
+    end
+
+    def create_directory_if_not_exists(directory)
+      FileUtils.mkdir_p(directory) unless File.directory?(directory)
+    end
+
+    def run_on_app_path
+      Dir.chdir(@ruby_app_path) do
+        yield
+      end
+    end
+
+    def remove_file(file)
+      run_on_app_path do
+        FileUtils.rm(file)
+      end
+    end
+
+    def write_content_to_file(filename, content)
+      File.open(filename, "w") do |file|
+        file.write(content)
+      end
     end
   end
 end

data/lib/reciper/version.rb

@@ -1,3 +1,3 @@
 module Reciper
-  VERSION = "0.0.4"
+  VERSION = "0.1.0"
 end

data/spec/reciper/helpers_spec.rb

@@ -14,229 +14,384 @@ describe Reciper::Helpers do
 
   describe ".copy" do
     it "copies the file from the recipe path to the ruby app root path " do
-      File.exists?("spec/fixtures/ruby_app/file.rb").should_not be
+      FileUtils.should_receive(:cp).with("spec/fixtures/recipe/file.rb", "spec/fixtures/ruby_app/file.rb")
 
       copy_file("file.rb")
-
-      File.exists?("spec/fixtures/ruby_app/file.rb").should be
-
-      FileUtils.rm("spec/fixtures/ruby_app/file.rb")
     end
 
     it "copies the file from the recipe path to the ruby app" do
-      File.exists?("spec/fixtures/ruby_app/lib/file.rb").should_not be
+      FileUtils.should_receive(:cp).with("spec/fixtures/recipe/file.rb", "spec/fixtures/ruby_app/lib/file.rb")
 
       copy_file("file.rb", :to => "lib")
-
-      File.exists?("spec/fixtures/ruby_app/lib/file.rb").should be
-
-      FileUtils.rm("spec/fixtures/ruby_app/lib/file.rb")
     end
 
     it "copies the file with the name as in defined in as" do
-      File.exists?("spec/fixtures/ruby_app/file.rb").should_not be
+      FileUtils.should_receive(:cp).with("spec/fixtures/recipe/file.rb", "spec/fixtures/ruby_app/another_file.rb")
 
       copy_file("file.rb", :as => "another_file.rb")
-
-      File.exists?("spec/fixtures/ruby_app/another_file.rb").should be
-
-      FileUtils.rm("spec/fixtures/ruby_app/another_file.rb")
     end
 
     it "if the dir doesn't exists, create it" do
-      File.exists?("spec/fixtures/ruby_app/lib/file.rb").should_not be
+      directory = @ruby_app_path + "/my_awesome_dir"
+      File.should_receive(:directory?).with("spec/fixtures/ruby_app/my_awesome_dir").and_return(false)
+      FileUtils.should_receive(:mkdir_p).with("spec/fixtures/ruby_app/my_awesome_dir")
+      FileUtils.should_receive(:cp).with("spec/fixtures/recipe/file.rb", "spec/fixtures/ruby_app/my_awesome_dir/file.rb")
 
       copy_file("file.rb", :to => "my_awesome_dir")
-
-      File.exists?("spec/fixtures/ruby_app/my_awesome_dir/file.rb").should be
-
-      FileUtils.rm_rf("spec/fixtures/ruby_app/my_awesome_dir")
     end
 
     it "adds the operation to @operation array" do
-      copy_file("file.rb")
+      FileUtils.should_receive(:cp).with("spec/fixtures/recipe/file.rb", "spec/fixtures/ruby_app/file.rb")
 
-      @operations.should include([:copy, "file.rb"])
+      copy_file("file.rb")
 
-      FileUtils.rm("spec/fixtures/ruby_app/file.rb")
+      @operations.should include([:copy_file, { :destination => "/file.rb" }])
     end
   end
 
   describe ".run_tests" do
     it "returns 0 if all tests pass" do
+      Dir.should_receive(:chdir).with("spec/fixtures/ruby_app").and_yield
+
+      test_output = <<-EOF
+      ....
+
+      Finished in 11.29 seconds
+      23 examples, 0 failures
+      EOF
+
+      io = double(:io, :read => test_output)
+      IO.should_receive(:popen).with("bundle exec rspec spec").and_yield(io)
+
       run_tests.should == 0
     end
 
     it "returns 1 if there is only one failure" do
-      copy_file("failing_spec.rb", :to => "spec")
+      Dir.should_receive(:chdir).with("spec/fixtures/ruby_app").and_yield
+
+      test_output = <<-EOF
+      FE..
 
-      run_tests.should == 1
+      Finished in 11.29 seconds
+      4 examples, 2 failures
+      EOF
 
-      FileUtils.rm("spec/fixtures/ruby_app/spec/failing_spec.rb")
+      io = double(:io, :read => test_output)
+      IO.should_receive(:popen).with("bundle exec rspec spec").and_yield(io)
+
+      run_tests.should == 2
     end
   end
 
   describe ".run_rake_task" do
-    it "returns true when the rake task has been run ok" do
-      run_rake_task("puts_something").should be
+    it "returns a hash with successful as true when the rake task has been run successfully" do
+      output = ""
+
+      Dir.should_receive(:chdir).with("spec/fixtures/ruby_app").and_yield
+
+      io = double(:io)
+      io.stub!(:read).and_return("")
+
+      IO.should_receive(:popen).
+        with("bundle exec rake puts_something").and_yield(io)
+
+      $?.should_receive(:exitstatus).and_return(0)
+
+      run_rake_task("puts_something").should == {
+        :response => "",
+        :successful => true
+      }
     end
 
-    it "returns false when the rake task hasn't been run ok" do
-      run_rake_task("idontexists").should_not be
+    it "returns a hash with successful as false when the rake task hasn't been run successfully" do
+      output = ""
+
+      io = double(:io)
+      io.stub!(:read).and_return("")
+
+      Dir.should_receive(:chdir).with("spec/fixtures/ruby_app").and_yield
+
+      IO.should_receive(:popen).
+        with("bundle exec rake puts_something").and_yield(io)
+      $?.should_receive(:exitstatus).and_return(1)
+
+      run_rake_task("puts_something").should == {
+        :response => "",
+        :successful => false
+      }
     end
   end
 
   describe ".copy_line_range" do
-    it "copies the entire input file to the output line " do
-      @expected_at_the_beginning = <<-EOF
-class MyClass
-end
+    it "copies the entire input file to the output line on a specified line" do
+      readme = <<-EOF
+d
+e
 EOF
-      File.read("spec/fixtures/ruby_app/lib/my_class.rb").should == @expected_at_the_beginning.chomp
-
-      expected_at_the_end = <<-EOF
-class MyClass
-def my_name
-  puts self.name
-end
-end
+
+      original = <<-EOF
+a
+b
+c
+f
 EOF
 
-      copy_line_range("my_name.rb", "lib/my_class.rb", :to_line => 2)
+      Dir.should_receive(:glob).with("spec/fixtures/ruby_app/README.md").
+        and_return(["spec/fixtures/ruby_app/README.md"])
+
+      File.should_receive(:read).with("spec/fixtures/recipe/README").
+        and_return(readme)
+
+      File.should_receive(:read).with("spec/fixtures/ruby_app/README.md").
+        and_return(original)
 
-      File.read("spec/fixtures/ruby_app/lib/my_class.rb").should == expected_at_the_end.chomp
+      file = double(:file)
+      file.should_receive(:write).with("a\nb\nc\nd\ne\nf")
+      File.should_receive(:open).with("spec/fixtures/ruby_app/README.md", "w").and_yield(file)
+
+      copy_line_range("README", "README.md", :to_line => 3)
     end
 
     it "copies only specified lines" do
-      @expected_at_the_beginning = <<-EOF
-class MyClass
-end
+      readme = <<-EOF
+a
+d
+e
+f
+g
 EOF
 
-      File.read("spec/fixtures/ruby_app/lib/my_class.rb").should == @expected_at_the_beginning.chomp
-
-      expected_at_the_end = <<-EOF
-class MyClass
-  puts self.name
-end
+      original = <<-EOF
+a
+b
+c
+f
 EOF
 
-      copy_line_range("my_name.rb", "lib/my_class.rb", :to_line => 2, :from_lines => (2..2))
+      Dir.should_receive(:glob).with("spec/fixtures/ruby_app/README.md").
+        and_return(["spec/fixtures/ruby_app/README.md"])
+
+      File.should_receive(:read).with("spec/fixtures/recipe/README").
+        and_return(readme)
 
-      File.read("spec/fixtures/ruby_app/lib/my_class.rb").should == expected_at_the_end.chomp
+      File.should_receive(:read).with("spec/fixtures/ruby_app/README.md").
+        and_return(original)
+
+      file = double(:file)
+      file.should_receive(:write).with("a\nb\nc\nd\ne\nf")
+      File.should_receive(:open).with("spec/fixtures/ruby_app/README.md", "w").and_yield(file)
+
+      copy_line_range("README", "README.md", :to_line => 3, :lines => (1..2))
     end
 
     it "adds an entry to operations" do
-      @expected_at_the_beginning = <<-EOF
-class MyClass
-end
+      readme = <<-EOF
+a
+d
+e
+f
+g
+EOF
+
+      original = <<-EOF
+a
+b
+c
+f
 EOF
 
-      copy_line_range("my_name.rb", "lib/my_class.rb", :to_line => 2)
+      Dir.stub!(:glob).with("spec/fixtures/ruby_app/README.md").
+        and_return(["spec/fixtures/ruby_app/README.md"])
+
+      File.stub!(:read).with("spec/fixtures/recipe/README").
+        and_return(readme)
+
+      File.stub!(:read).with("spec/fixtures/ruby_app/README.md").
+        and_return(original)
 
-      @operations.should include([:copy_range, "lib/my_class.rb", @expected_at_the_beginning.chomp])
+      file = double(:file)
+      file.stub!(:write).with("a\nb\nc\nd\ne\nf")
+      File.stub!(:open).with("spec/fixtures/ruby_app/README.md", "w").and_yield(file)
+
+      copy_line_range("README", "README.md", :to_line => 3, :lines => (1..2))
+
+      @operations.should include([:copy_line_range, { :original_content => original, :original_file => "spec/fixtures/ruby_app/README.md"}])
     end
 
-    after do
-      File.write("spec/fixtures/ruby_app/lib/my_class.rb", @expected_at_the_beginning.chomp)
+    context "suffix copy" do
+      it "works with only the suffix of the file when there is only one file" do
+        readme = ""
+        original = ""
+
+        Dir.should_receive(:glob).with("spec/fixtures/ruby_app/*.md").
+          and_return(["spec/fixtures/ruby_app/README.md"])
+
+        File.stub!(:read).with("spec/fixtures/recipe/README").
+          and_return(readme)
+
+        File.should_receive(:read).with("spec/fixtures/ruby_app/README.md").
+          and_return(original)
+
+        file = double(:file)
+        file.stub!(:write)
+        File.stub!(:open).with("spec/fixtures/ruby_app/README.md", "w").and_yield(file)
+
+        copy_line_range("README", "*.md", :to_line => 0)
+      end
+
+      it "raises an exception when given only the suffix of the file when there is more than one file" do
+        readme = ""
+        original = ""
+
+        Dir.should_receive(:glob).with("spec/fixtures/ruby_app/*.md").
+          and_return(["spec/fixtures/ruby_app/README.md", "spec/fixtures/ruby_app/README2.md"])
+
+        lambda {
+          copy_line_range("README", "*.md", :to_line => 0)
+        }.should raise_error(Reciper::NoFileOrMultipleFilesFound)
+      end
     end
   end
 
   describe ".run_command" do
-    it "runs a command on projects folder and returns true when successful" do
-      run_command("ls").should be
+    it "runs a command on projects folder and returns the command hash with the response and true when successful" do
+      Dir.should_receive(:chdir).with("spec/fixtures/ruby_app").and_yield
+
+      output = <<EOF
+a
+b
+EOF
+
+      io = double(:io)
+      io.should_receive(:read) do
+        "a\nb\n"
+      end
+
+      IO.should_receive(:popen).with("ls").and_yield(io)
+      $?.should_receive(:exitstatus).and_return(0)
+
+      run_command("ls").should == {
+        :response => "a\nb\n",
+        :successful => true
+      }
     end
 
-    it "runs a command on projects folder and returns not true when failure" do
-      run_command("cp").should_not be
+    it "runs a command on projects folder and returns the command hash with the response and false when not successful" do
+      Dir.should_receive(:chdir).with("spec/fixtures/ruby_app").and_yield
+
+      output = <<EOF
+a
+b
+EOF
+
+      io = double(:io)
+      io.should_receive(:read) do
+        "a\nb\n"
+      end
+
+      IO.should_receive(:popen).with("ls").and_yield(io)
+      $?.should_receive(:exitstatus).and_return(1)
+
+      run_command("ls").should == {
+        :response => "a\nb\n",
+        :successful => false
+      }
     end
 
     it "receives the rollback command together with the command and store it on @operations array" do
       run_command("ls", "ls -a")
 
-      @operations.should include([:run_command, "ls -a"])
+      @operations.should include([:run_command, { :rollback_command => "ls -a" }])
     end
-  end
 
-  describe ".rollback" do
-    it "removes the file when the operation is copy" do
-      File.open(@ruby_app_path + "/an_added_file.rb", "w") {
-        |f| f.write("OK")
-      }
+    it "doesn't require the rollback command to be informed" do
+      run_command("ls")
 
-      File.exists?("spec/fixtures/ruby_app/an_added_file.rb").should be
+      @operations.should include([:run_command, { :rollback_command => nil}])
+    end
+  end
 
-      @operations = [[:copy, "an_added_file.rb"]]
+  describe ".override_file" do
+    it "overrides the file with another file" do
+      Dir.should_receive(:chdir).with(@ruby_app_path).and_yield
+      File.should_receive(:exists?).with("README").and_return(true)
+      FileUtils.should_receive(:mkdir_p).with("/tmp/reciper")
 
-      rollback
+      FileUtils.should_receive(:cp).with("README", "/tmp/reciper/README")
+      FileUtils.should_receive(:cp).with("spec/fixtures/recipe/README",
+       "spec/fixtures/ruby_app/README")
 
-      File.exists?("spec/fixtures/ruby_app/an_added_file.rb").should_not be
+      override_file("README", "README")
     end
 
-    it "restores the old file when the operation is copy_range" do
-      File.open(@ruby_app_path + "/an_added_file.rb", "w") {
-        |f| f.write("OK")
-      }
-
-      File.read("spec/fixtures/ruby_app/an_added_file.rb").should == "OK"
+    it "raises an error when file doesn't exists" do
+      Dir.stub!(:chdir).with(@ruby_app_path).and_yield
+      File.should_receive(:exists?).with("README").and_return(false)
 
-      @operations = [[:copy_range, "an_added_file.rb", "Not OK"]]
+      lambda {
+        override_file("README", "README")
+      }.should raise_error(Reciper::NoFileToBeOverriden)
+    end
 
-      rollback
+    it "adds the operation to operations array" do
+      Dir.stub!(:chdir).with(@ruby_app_path).and_yield
+      File.stub!(:exists?).with("README").and_return(true)
+      FileUtils.stub!(:mkdir_p).with("/tmp/reciper")
 
-      File.read("spec/fixtures/ruby_app/an_added_file.rb").should == "Not OK"
+      FileUtils.stub!(:cp).with("README", "/tmp/reciper/README")
+      FileUtils.stub!(:cp).with("spec/fixtures/recipe/README",
+       "spec/fixtures/ruby_app/README")
 
-      FileUtils.rm("spec/fixtures/ruby_app/an_added_file.rb")
+      override_file("README", "README")
+      @operations.should include([:override_file, { :tmp_file => "/tmp/reciper/README", :overriden_file => "README"}])
     end
+  end
 
-    it "runs the rollback command when the operation is run_command and we have a rollback command" do
-      @operations = [[:run_command, "ls"]]
+  describe ".rollback" do
+    it "removes the file when the operation is copy_file" do
+      @operations = [[:copy_file, { :destination => "README" }]]
+
+      Dir.should_receive(:chdir).with("spec/fixtures/ruby_app").and_yield
 
-      self.should_receive(:spawn).with("ls")
-      Process.stub!(:wait)
+      FileUtils.should_receive(:rm).with("README")
 
       rollback
     end
 
-    it "runs the rollback command when the operation is override_file" do
-      begin
-        FileUtils.cp("spec/fixtures/ruby_app/README", "/tmp/README")
-        FileUtils.rm("spec/fixtures/ruby_app/README")
-
-        File.exists?("spec/fixtures/ruby_app/README").should_not be
-
-        @operations = [[:override_file, "/tmp/README", "README"]]
+    it "restores the old file when the operation is copy_range" do
+      @operations = [[:copy_range, { :original_content => "Not OK", :original_file => "an_added_file.rb"}]]
 
-        rollback
+      file = double(:file)
+      file.should_receive(:write).with("Not OK")
+      File.should_receive(:open).with("an_added_file.rb", "w").and_yield(file)
 
-        File.exists?("spec/fixtures/ruby_app/README").should be
-      ensure
-        FileUtils.cp("/tmp/README", "spec/fixtures/ruby_app/README") unless File.exists?("spec/fixtures/ruby_app/README")
-      end
+      rollback
     end
-  end
 
-  describe ".override_file" do
-    it "overrides the file with another file" do
-      FileUtils.cp("spec/fixtures/ruby_app/README", "/tmp/README")
+    it "runs the rollback command when the operation is run_command and we have a rollback command" do
+      @operations = [[:run_command, { :rollback_command => "ls"}]]
 
-      File.read("spec/fixtures/ruby_app/README").should == "some content"
+      Dir.should_receive(:chdir).with("spec/fixtures/ruby_app").and_yield
 
-      override_file("README", "README")
+      output = ""
 
-      File.read("spec/fixtures/ruby_app/README").should == ""
+      io = double(:io)
+      io.should_receive(:read) do
+        ""
+      end
 
-      FileUtils.mv("/tmp/README", "spec/fixtures/ruby_app/README")
-    end
+      IO.should_receive(:popen).with("ls").and_yield(io)
+      $?.should_receive(:exitstatus).and_return(0)
 
-    it "adds the operation to operations array" do
-      FileUtils.cp("spec/fixtures/ruby_app/README", "/tmp/README")
+      rollback
+    end
 
-      override_file("README", "README")
+    it "runs the rollback command when the operation is override_file" do
+      @operations = [[:override_file, { :tmp_file => "/tmp/reciper/my_file.rb", :overriden_file => "ola.rb"}]]
 
-      @operations.should include([:override_file, "/tmp/reciper/README", "README"])
+      FileUtils.should_receive(:cp).with("/tmp/reciper/my_file.rb", "spec/fixtures/ruby_app/ola.rb")
 
-      FileUtils.mv("/tmp/README", "spec/fixtures/ruby_app/README")
+      rollback
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: reciper
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.1.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-04-11 00:00:00.000000000 Z
+date: 2012-06-04 00:00:00.000000000 Z
 dependencies: []
 description: An awesome way to write recipes for a book chapter
 email:
@@ -22,21 +22,12 @@ files:
 - Gemfile
 - Gemfile.lock
 - LICENSE
+- README.md
 - Rakefile
 - lib/reciper.rb
 - lib/reciper/helpers.rb
 - lib/reciper/version.rb
 - reciper.gemspec
-- spec/fixtures/recipe/README
-- spec/fixtures/recipe/failing_spec.rb
-- spec/fixtures/recipe/file.rb
-- spec/fixtures/recipe/my_name.rb
-- spec/fixtures/ruby_app/Gemfile
-- spec/fixtures/ruby_app/Gemfile.lock
-- spec/fixtures/ruby_app/README
-- spec/fixtures/ruby_app/Rakefile
-- spec/fixtures/ruby_app/lib/my_class.rb
-- spec/fixtures/ruby_app/spec/true_spec.rb
 - spec/reciper/helpers_spec.rb
 - spec/spec_helper.rb
 homepage: http://github.com/rodrigoflores/reciper
@@ -51,28 +42,24 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: -3801167226663698293
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: -3801167226663698293
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.11
+rubygems_version: 1.8.23
 signing_key: 
 specification_version: 3
 summary: An awesome way to write recipes for a book chapter
 test_files:
-- spec/fixtures/recipe/README
-- spec/fixtures/recipe/failing_spec.rb
-- spec/fixtures/recipe/file.rb
-- spec/fixtures/recipe/my_name.rb
-- spec/fixtures/ruby_app/Gemfile
-- spec/fixtures/ruby_app/Gemfile.lock
-- spec/fixtures/ruby_app/README
-- spec/fixtures/ruby_app/Rakefile
-- spec/fixtures/ruby_app/lib/my_class.rb
-- spec/fixtures/ruby_app/spec/true_spec.rb
 - spec/reciper/helpers_spec.rb
 - spec/spec_helper.rb

data/spec/fixtures/recipe/failing_spec.rb

@@ -1,7 +0,0 @@
-require 'rspec'
-
-describe "true is true" do
-  it "is true" do
-    true.should_not be
-  end
-end

data/spec/fixtures/recipe/my_name.rb

@@ -1,3 +0,0 @@
-def my_name
-  puts self.name
-end

data/spec/fixtures/ruby_app/Gemfile

@@ -1,4 +0,0 @@
-source 'https://rubygems.org'
-
-gem "rake"
-gem "rspec"

data/spec/fixtures/ruby_app/Gemfile.lock

@@ -1,12 +0,0 @@
-GEM
-  remote: https://rubygems.org/
-  specs:
-    rake (0.9.2.2)
-    rspec-core (2.9.0)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  rake
-  rspec-core

data/spec/fixtures/ruby_app/README

@@ -1 +0,0 @@
-some content

data/spec/fixtures/ruby_app/Rakefile

@@ -1,5 +0,0 @@
-task :default => [:puts_something]
-
-task :puts_something do
-  sh "echo 'hello'"
-end

data/spec/fixtures/ruby_app/lib/my_class.rb

@@ -1,2 +0,0 @@
-class MyClass
-end

data/spec/fixtures/ruby_app/spec/true_spec.rb

@@ -1,7 +0,0 @@
-require 'rspec'
-
-describe "true is true" do
-  it "is true" do
-    true.should be
-  end
-end