automateit 0.70923 → 0.70928

data/spec/integration/edit_manager_spec.rb

@@ -0,0 +1,52 @@
+require File.join(File.dirname(File.expand_path(__FILE__)), "/../spec_helper.rb")
+
+describe "AutomateIt::EditManager for files" do
+  before :all do
+    @a = AutomateIt.new(:verbosity => Logger::WARN)
+  end
+
+  it "should make a backup if editing a file" do
+    @a.mktempdircd do
+      target = "myfile"
+      @a.render(:text => "Whatever", :to => target)
+      entries = Dir.entries(".").select{|t| t =~ /^#{target}/}
+      entries.size.should == 1
+      
+      @a.edit(:file => target) do
+        append "Hello"
+      end.should be_true
+      
+      entries = Dir.entries(".").select{|t| t =~ /^#{target}/}
+      entries.size.should == 2
+      File.size(entries[0]).should_not == File.size(entries[1])
+    end    
+  end
+
+  it "should not make a backup if not editing a file" do
+    @a.mktempdircd do
+      target = "myfile"
+      @a.render(:text => "Whatever", :to => target)
+      entries = Dir.entries(".").select{|t| t =~ /^#{target}/}
+      entries.size.should == 1
+      
+      @a.edit(:file => target) do
+      end.should be_false
+      
+      entries = Dir.entries(".").select{|t| t =~ /^#{target}/}
+      entries.size.should == 1
+    end    
+  end
+  
+  it "should not make a backup for a newly created file" do
+    @a.mktempdircd do
+      target = "myfile"
+      
+      @a.edit(:file => target, :create => true) do
+        append "Hello"
+      end.should be_true
+      
+      entries = Dir.entries(".").select{|t| t =~ /^#{target}/}
+      entries.size.should == 1
+    end  
+  end
+end

data/spec/integration/shell_manager_spec.rb

@@ -5,6 +5,10 @@ describe AutomateIt::ShellManager, :shared => true do
     @a = AutomateIt.new(:verbosity => Logger::WARN)
     @m = @a.shell_manager
   end
+
+  before(:each) do
+    @a.preview = false
+  end
 end
 
 describe AutomateIt::ShellManager, " with sh and which" do
@@ -42,6 +46,60 @@ end
 describe AutomateIt::ShellManager, " in general" do
   it_should_behave_like "AutomateIt::ShellManager"
 
+  it "should backup a file (backup)" do
+    @m.mktempdircd do
+      source = "myfile"
+      @m.touch(source)
+
+      target = @m.backup(source)
+
+      target.should_not be_nil
+      target.should_not == source
+      File.exists?(target).should be_true
+    end
+  end
+
+  it "should backup a directory with files (backup)" do
+    @m.mktempdircd do
+      dir = "mydir"
+      file = File.join(dir, "myfile")
+      @m.mkdir(dir)
+      @m.touch(file)
+
+      target_dir = @m.backup(dir)
+
+      target_dir.should_not be_nil
+      target_dir.should_not == dir
+      File.exists?(target_dir).should be_true
+      File.directory?(target_dir).should be_true
+
+      target_file = File.join(target_dir, File.basename(file))
+      File.exists?(target_file).should be_true
+      File.directory?(target_file).should be_false
+    end
+  end
+
+  it "should backup multiple files and directories (backup)" do
+    @m.mktempdircd do
+      @m.mkdir_p("foo/bar/baz")
+      @m.touch("foo/bar/baz/feh")
+      @m.mkdir_p("foo/bar/qux")
+      @m.touch("foo/bar/qux/meh")
+      @m.mkdir_p("qux/foo/bar")
+      @m.touch("qux/foo/bar/bah")
+
+      results = @m.backup("foo", "qux")
+      results.should_not be_blank
+
+      results[0].should =~ %r{#{File::SEPARATOR}foo\..+\.bak$}
+      File.exists?(results[0]+"/bar/baz/feh")
+      File.exists?(results[0]+"/bar/baz/meh")
+
+      results[1].should =~ %r{#{File::SEPARATOR}qux\..+\.bak$}
+      File.exists?(results[1]+"/foo/bar/bah")
+    end
+  end
+
   it "should change directories (cd)" do
     before = Dir.pwd
     target = Pathname.new("/").expand_path.to_s
@@ -60,6 +118,38 @@ describe AutomateIt::ShellManager, " in general" do
     Dir.pwd.should == before
   end
 
+  it "should change back to a directory when there's an error (cd)" do
+    before = Dir.pwd
+    target = Pathname.new("/").expand_path.to_s
+    lambda {
+      @m.cd(target) do
+        raise "42"
+      end
+    }.should raise_error("42")
+    Dir.pwd.should == before
+  end
+
+  it "should fail to change to a non-existend directory (cd)" do
+    lambda{ @m.cd("qweljkrusfauweqr") }.should raise_error(Errno::ENOENT)
+  end
+
+  it "should not fail to change to a non-existent directory in preview mode (cd)" do
+    @m.preview = true
+    @m.cd("qweljkrusfauweqr").should_not be_nil
+  end
+
+  it "should return if there's an error when pretending to cd into a non-existent directory in preview mode (cd)" do
+    @m.preview = true
+    before = Dir.pwd
+    lambda {
+      @m.cd("qweljkrusfauweqr") do
+        Dir.pwd.should == before
+        raise "42"
+      end
+    }.should raise_error("42")
+    Dir.pwd.should == before
+  end
+
   it "should locate the current directory (pwd)" do
     @m.pwd.should == Dir.pwd
   end
@@ -76,6 +166,26 @@ describe AutomateIt::ShellManager, " in general" do
     end
   end
 
+  it "should create a directory and cd into it (mkdir)" do
+    @m.mktempdircd do
+      target = "foo"
+      @m.mkdir(target) do |path|
+        path.should == target
+        File.basename(Dir.pwd).should == target
+      end
+    end
+  end
+
+  it "should fail to cd into multiple created directories (mkdir)" do
+    @m.mktempdircd do
+      lambda {
+        @m.mkdir(%w(foo bar baz)) do |path|
+          violated
+        end
+      }.should raise_error(ArgumentError)
+    end
+  end
+
   it "should create nested directories when needed (mkdir_p)" do
     @m.mktempdircd do
       target = "foo/bar/baz"
@@ -98,6 +208,20 @@ describe AutomateIt::ShellManager, " in general" do
     end
   end
 
+  it "should create a temporary file (mktemp)" do
+    @m.mktempdir do
+      item = nil
+      @m.mktemp do |path|
+        item = path
+        File.exists?(item).should be_true
+      end
+      File.exists?(item).should be_false
+    end
+  end
+
+  #it "should create a temporary directory (mktempdir)"
+  #it "should create a temporary directory and cd into it (mktempdircd)"
+
   it "should install a file to a file (install)" do
     @m.mktempdircd do
       source = "foo"
@@ -188,10 +312,6 @@ describe AutomateIt::ShellManager, " in general" do
     end
   end
 
-  # TODO implement umask spec
-
-  # TODO implement gap
-
   it "should move files (mv)" do
     @m.mktempdircd do
       file1 = "foo"
@@ -252,6 +372,25 @@ describe AutomateIt::ShellManager, " in general" do
     end
   end
 
+  it "should create files and change their timestamps (touch)" do
+    @m.mktempdircd do
+      target = "foo"
+      File.exists?(target).should be_false
+
+      @m.touch(target)
+      File.exists?(target).should be_true
+      before = File.mtime(target)
+
+      @m.touch(target)
+      after = File.mtime(target)
+      before.should <= after
+    end
+  end
+end
+
+describe AutomateIt::ShellManager, " when managing modes" do
+  it_should_behave_like "AutomateIt::ShellManager"
+
   unless INTERPRETER.shell_manager.provides_mode?
     puts "NOTE: Can't check permission modes on this platform, #{__FILE__}"
   else
@@ -288,22 +427,10 @@ describe AutomateIt::ShellManager, " in general" do
         @m.chmod_R(mode, dir).should be_false
       end
     end
-  end
 
-  it "should create files and change their timestamps (touch)" do
-    @m.mktempdircd do
-      target = "foo"
-      File.exists?(target).should be_false
-
-      @m.touch(target)
-      File.exists?(target).should be_true
-      before = File.mtime(target)
-
-      @m.touch(target)
-      after = File.mtime(target)
-      before.should <= after
-    end
+    #it "should set the default mask (umask)" # TODO 
   end
+
 end
 
 describe AutomateIt::ShellManager, " when managing permissions" do

data/spec/integration/template_manager_erb_spec.rb

@@ -9,7 +9,11 @@ describe "AutomateIt::TemplateManager::ERB" do
     @d.available?.should be_true
   end
 
-  it "should set mode when rendering" do
+  before(:each) do
+    @a.preview = false
+  end
+
+  it "should set file's mode when rendering" do
     @a.mktempdircd do
       source = "foo"
       target = "bar"
@@ -28,4 +32,17 @@ describe "AutomateIt::TemplateManager::ERB" do
       end
     end
   end
+
+  it "should fail to render non-existent file" do
+    @a.mktempdircd do
+      lambda { @a.render(:file => "foo", :to => "bar") }.should raise_error(Errno::ENOENT)
+    end
+  end
+
+  it "should not raise error with non-existent file in preview mode" do
+    @a.mktempdircd do
+      @a.preview = true
+      @a.render(:file => "foo", :to => "bar").should be_true
+    end
+  end
 end

data/spec/integration/tempster_spec.rb

@@ -0,0 +1,65 @@
+require File.join(File.dirname(File.expand_path(__FILE__)), "/../spec_helper.rb")
+
+describe Tempster do
+  it "should create a temporary file" do
+    result = nil
+    begin
+      result = Tempster.mktemp(:verbose => false)
+      File.exists?(result).should be_true
+    ensure
+      File.unlink(result) if result
+    end
+  end
+
+  it "should create a temporary file and remove it with a block" do
+    result = nil
+    begin
+      Tempster.mktemp(:verbose => false) do |path|
+        result = path
+        File.exists?(result).should be_true
+      end
+      File.exists?(result).should be_false
+    ensure
+      File.unlink(result) if result and File.exists?(result)
+    end
+  end
+
+  it "should create a temporary directory" do
+    result = nil
+    begin
+      result = Tempster.mktempdir(:verbose => false)
+      File.directory?(result).should be_true
+    ensure
+      Dir.rmdir(result) if result
+    end
+  end
+
+  it "should create a temporary directory and remove it with a block" do
+    result = nil
+    begin
+      Tempster.mktempdir(:verbose => false) do |path|
+        result = path
+        File.directory?(result).should be_true
+      end
+      File.directory?(result).should be_false
+    ensure
+      Dir.rmdir(result) if result and File.directory?(result)
+    end
+  end
+
+  it "should create a temporary directory, cd into it and remove it with a block" do
+    result = nil
+    previous = Dir.pwd
+    begin
+      Tempster.mktempdircd(:verbose => false) do |path|
+        result = path
+        File.directory?(result).should be_true
+        Dir.pwd.should_not == previous
+      end
+      File.directory?(result).should be_false
+      Dir.pwd.should == previous
+    ensure
+      Dir.rmdir(result) if result and File.directory?(result)
+    end
+  end
+end

data/spec/unit/edit_manager_spec.rb

@@ -21,12 +21,18 @@ describe "AutomateIt::EditManager for strings" do
     end
   end
 
-  it "should find contained lines" do
+  it "should find if buffer contains lines by regexp" do
     @a.edit(:text => @input) do
       contains?(/This/).should == true
     end
   end
 
+  it "should find if buffer contains lines by string" do
+    @a.edit(:text => @input) do
+      contains?("This").should == true
+    end
+  end
+
   it "should prepend lines to the top" do
     output = @a.edit(:text => @input) do
       prepend "PREPEND"
@@ -117,7 +123,7 @@ describe "AutomateIt::EditManager for strings" do
   end
 
   it "should raise exceptions for invalid methods" do
-    lambda { 
+    lambda {
       @a.edit(:text => @input) do
         qwoiuerjzxiuo
       end
@@ -138,7 +144,7 @@ describe "AutomateIt::EditManager for files" do
   it "should edit a file" do
     File.should_receive(:read).with(@filename).and_return(@input)
     File.should_receive(:open).with(@filename, "w+").and_return(true)
-    result = @a.edit(:file => @filename) do
+    result = @a.edit(:file => @filename, :backup => false) do
       append "APPEND"
     end
     result.should be_true
@@ -146,7 +152,7 @@ describe "AutomateIt::EditManager for files" do
 
   it "should not rewrite an unchanged file" do
     File.should_receive(:read).with(@filename).and_return(@input)
-    result = @a.edit(:file => @filename) do
+    result = @a.edit(:file => @filename, :backup => false) do
       # Do nothing
     end
     result.should be_false
@@ -154,9 +160,47 @@ describe "AutomateIt::EditManager for files" do
 
   it "should default to editing a file" do
     File.should_receive(:read).with(@filename).and_return(@input)
-    result = @a.edit(@filename) do
+    result = @a.edit(@filename, :backup => false) do
+      # Do nothing
+    end
+    result.should be_false
+  end
+
+  it "should fail to editing non-existent file" do
+    lambda {
+      @a.edit(@filename) do
+        # Do nothing
+      end
+    }.should raise_error(Errno::ENOENT)
+  end
+end
+
+describe "AutomateIt::EditManager for files in preview mode" do
+  before(:all) do
+    @a = AutomateIt.new(:verbosity => Logger::WARN)
+    @a.preview = true
+    @filename = "input"
+  end
+
+  it "should not fail to editing a non-existent file" do
+    @a.edit(@filename) do
+      # Do nothing
+    end
+  end
+
+  it "should not fail when reading a non-existent file" do
+    result = @a.edit(@filename, :backup => false) do
       # Do nothing
     end
     result.should be_false
   end
+
+  it "should not write changes" do
+    File.should_receive(:exists?).any_number_of_times.with(@filename).and_return(true)
+    File.should_receive(:read).with(@filename).and_return(@input)
+    result = @a.edit(:file => @filename, :backup => false) do
+      append "APPEND"
+    end
+    result.should be_true
+  end
 end