nickstenning-outback 0.0.3 → 0.0.4

data/README.markdown

@@ -0,0 +1,3 @@
+# Outback - a transactional shell runner
+
+Outback is a simple ruby library for running transactional pairs of shell commands ("rollout/rollback") in a known sequence. It's useful for running a set of dependent commands one after another with the ability to stop, find out what's gone wrong, and rollback all the steps so far if something goes awry.

data/Rakefile

@@ -0,0 +1,9 @@
+require 'rake'
+require 'spec/rake/spectask'
+
+desc "Run all examples with RCov"
+Spec::Rake::SpecTask.new('rcov_specs') do |t|
+  t.spec_files = FileList['spec/**/*_spec.rb']
+  t.rcov = true
+  t.rcov_opts = ['--exclude', 'spec']
+end

data/lib/outback.rb

@@ -1,11 +1,10 @@
 require 'outback/manager'
-require 'outback/shelltask'
-require 'outback/yaml'
+require 'outback/task_helper'
+require 'outback/task'
 
 module Outback
   
   class Error < RuntimeError; end
   class TransactionError < Error; end
-  class DuplicateNamedTaskError < Error; end
   
 end

data/lib/outback/manager.rb

@@ -1,104 +1,94 @@
 require 'yaml'
+require 'observer'
 
 module Outback
   
   class Manager
     
+    include Observable
+    
     ROLLOUT = 1
     ROLLBACK = -1
     
-    attr_reader :tasks, :position
-    attr_accessor :workdir
-    attr_writer :watcher
+    attr_reader :tasks
+    attr_accessor :direction, :position, :workdir, :cache
   
     def initialize
       @tasks = []
-      @names = {}
+      @cache = {}
       @position = 0
+      @direction = 1
     end
     
     def add_tasks( *tasks )
-      [*tasks].each do |task|
-        task.workdir = @workdir unless task.workdir
-        
-        if task.name
-          if @names.has_key?(task.name)
-            raise DuplicateNamedTaskError, "Cannot add a named task more than once!"
-          else
-            @names[task.name] = @tasks.length
-          end
-        end
-        
-        @tasks << task
-      end
+      @tasks += tasks
     end
     
-    alias_method :add_task, :add_tasks
-    
-    def find_task( name )
-      @tasks[@names[name]]
+    def add_task( task=nil, &block )
+      if block_given?
+        task = Outback::Task.new(&block)
+      end
+      add_tasks( task ) if task
     end
     
     def rollout!
       @direction = ROLLOUT
-      run
+      run_all
     end
     
     def rollback!
       @direction = ROLLBACK
-      run
+      run_all
     end
     
-    def rollout_from( task )
-      @position = @tasks.index(task)
-      rollout!
+    def run_all
+      cache.delete(:errors)
+      
+      tasks_to_run.each do |task|
+        temp = cache.dup
+        begin
+          run task, @direction
+        rescue => e
+          self.cache = temp
+          cache[:errors] ||= []
+          cache[:errors] << e
+          break false
+        end
+        @position += @direction
+        changed
+        notify_observers(state, cache)
+      end
     end
     
-    def rollback_from( task )
-      @position = @tasks.index(task) - 1
-      rollback!
+    def tasks_to_run
+      if @direction == ROLLOUT
+        @tasks[@position..-1]
+      elsif @direction == ROLLBACK
+        @tasks[0...@position].reverse
+      end
     end
     
-    def attempt( task )
-      method = {ROLLOUT => :rollout!, ROLLBACK => :rollback!}[@direction]
-      ret = task.send(method)
-      @watcher.notify(task) if @watcher
-      return ret
+    def state
+      { :position => @position,
+        :direction => @direction
+      }
     end
     
-    def current_task
-      @tasks[@position]
+    def restore_state( state )
+      @position = state[:position] || @position
+      @direction = state[:direction] || @direction
+      self
     end
+        
+    def run( task, direction=1 )
+      method = { ROLLOUT => :rollout!, ROLLBACK => :rollback! }[direction]
     
-    private
-    
-    def run
-      if @direction == ROLLOUT
-        task_list = @tasks[@position..-1]
-      elsif @direction == ROLLBACK
-        task_list = @tasks[0..@position + 1].reverse
-      end
-      task_list.each do |task|
-        @position = @tasks.index(task)
-        unless attempt current_task
-          fail
-          break
-        end
-      end
+      task.send(method, task_helper)
     end
     
-    def fail
-      case @direction
-      when ROLLOUT
-        #rollback_from current_task
-        raise Error, "Could not rollout task #{current_task}, attempting rollback."
-      when ROLLBACK
-        raise TransactionError, "Could not rollback task #{current_task}, aborting."
-      else
-        raise Error, "Unknown direction!"
-      end
+    def task_helper
+      @task_helper ||= TaskHelper.new(self)
     end
-    
   end
 
-end
+end

data/lib/outback/task.rb

@@ -0,0 +1,35 @@
+module Outback
+  class Task
+    def initialize
+      @rollback = proc {}
+      @rollout  = proc {}
+      if block_given?
+        yield self
+      end
+    end
+    
+    def rollout( &block )
+      if block_given?
+        @rollout = block
+      else
+        @rollout
+      end
+    end
+    
+    def rollback( &block )
+      if block_given?
+        @rollback = block
+      else
+        @rollback
+      end
+    end
+    
+    def rollout!( task_helper = nil )
+      @rollout.call(task_helper)
+    end
+    
+    def rollback!( task_helper = nil )
+      @rollback.call(task_helper)
+    end
+  end
+end

data/lib/outback/task_helper.rb

@@ -0,0 +1,39 @@
+require 'rubygems'
+require 'open4'
+
+module Outback
+  class TaskHelper  
+    def initialize( manager=nil )
+      @manager = manager
+    end
+    
+    def cache
+      if @manager and cache = @manager.cache
+        cache
+      else
+        {}
+      end
+    end
+    
+    def workdir
+      if @manager and workdir = @manager.workdir
+        workdir
+      else
+        Dir.getwd
+      end
+    end
+    
+    def sys( command )
+      result = {}
+      Dir.chdir(workdir) do
+        status = Open4.popen4(*command) do |pid, i, o, e|
+          result[:stdout] = o.read
+          result[:stderr] = e.read
+        end
+        result[:exit_status] = status.exitstatus
+      end
+      return result
+    end
+
+  end
+end

data/outback.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name = %q{outback}
-  s.version = "0.0.3"
+  s.version = "0.0.4"
  
   s.specification_version = 2 if s.respond_to? :specification_version=
  
@@ -8,7 +8,7 @@ Gem::Specification.new do |s|
   s.authors = ["Nick Stenning"]
   s.date = %q{2008-07-14}
   s.email = ["nick@whiteink.com"]
-  s.files = ["lib/outback/manager.rb", "lib/outback/shelltask.rb", "lib/outback/yaml.rb", "lib/outback.rb", "outback.gemspec", "spec/outback/manager_spec.rb", "spec/outback/shelltask_spec.rb", "spec/outback/yaml_spec.rb", "spec/spec_helper.rb"]
+  s.files = ["lib/outback/manager.rb", "lib/outback/task.rb", "lib/outback/task_helper.rb", "lib/outback.rb", "outback.gemspec", "Rakefile", "README.markdown", "spec/fixtures/example_manager.rb", "spec/outback/manager_spec.rb", "spec/outback/task_helper_spec.rb", "spec/outback/task_spec.rb", "spec/spec_helper.rb"] 
   s.has_rdoc = true
   s.homepage = %q{http://github.com/nickstenning/outback}
   s.require_paths = ["lib"]

data/spec/fixtures/example_manager.rb

@@ -0,0 +1,61 @@
+$: << File.expand_path(File.dirname(__FILE__) + '/../../lib')
+require 'outback'
+
+include Outback
+
+if $0 == __FILE__
+
+  @man = Manager.new
+
+  @man.add_task do |t|
+    t.rollout do |m|
+      m.cache[:result] = 1
+    end
+    t.rollback do |m|
+      m.cache.delete(:result)
+    end
+  end
+
+  @man.add_task do |t|
+    t.rollout do |m|
+      m.cache[:result] += 2
+    end
+    t.rollback do |m|
+      m.cache[:result] -= 2
+    end
+  end
+
+  @man.add_task do |t|
+    t.rollout do |m|
+      m.cache[:number_three] = m.cache[:result]
+    end
+    t.rollback do |m|
+      m.cache.delete(:number_three)
+    end
+  end
+
+  @man.add_task do |t|
+    t.rollout do |m|
+      m.cache[:b0rk] = "hello"
+    end
+    t.rollback do |m|
+      m.cache.delete(:b0rk)
+    end
+  end
+  
+  @man.add_task do |t|
+    t.rollout do |m|
+      m.cache[:syscall] = m.sys("pwd")[:stdout].strip
+    end
+    t.rollback do |m|
+      m.cache.delete(:syscall)
+    end
+  end
+  
+  @man.rollout!
+  y @man.cache
+  @man.rollback!
+  y @man.cache
+  
+
+end

data/spec/outback/manager_spec.rb

@@ -5,9 +5,9 @@ describe Outback::Manager do
   
   before do
     @obm = Outback::Manager.new
-    @task1 = mock("task1", :workdir => "/tmp", :name => "task1", :null_object => true)
-    @task2 = mock("task2", :workdir => nil, :name => nil, :null_object => true)
-    @task3 = mock("task3", :workdir => nil, :name => "task3", :null_object => true)
+    @task1 = mock(:task1, :null_object => true)
+    @task2 = mock(:task2, :null_object => true)
+    @task3 = mock(:task3, :null_object => true)
   end
 
   it "should have a list of tasks" do
@@ -22,6 +22,12 @@ describe Outback::Manager do
     @obm.should have(2).tasks
   end
   
+  it "should add a new Outback::Task and add it if #add_task is called with a block" do
+    @obm.add_task do
+    end
+    @obm.should have(1).tasks
+  end
+  
   it "should add multiple tasks to its list, in order, with #add_tasks" do
     @obm.add_tasks(@task1, @task2, @task3)
     @obm.should have(3).tasks
@@ -30,20 +36,20 @@ describe Outback::Manager do
     @obm.tasks[2].should == @task3
   end
   
-  it "should not allow more than one task with the same name" do
-    lambda { @obm.add_tasks(@task1, @task1) }.should raise_error(Outback::DuplicateNamedTaskError)
-    lambda { @obm.add_tasks(@task1, @task2, @task1) }.should raise_error(Outback::DuplicateNamedTaskError)
-    lambda { @obm.add_tasks(@task1, @task3, @task3) }.should raise_error(Outback::DuplicateNamedTaskError)
+  it "should give a hash representing its state on #state" do
+    @obm.state[:position].should == 0
+    @obm.state[:direction].should == 1
   end
   
-  it "should allow the same task twice if unnamed" do
-    lambda { @obm.add_tasks(@task1, @task2, @task2) }.should_not raise_error
+  it "should restore its state when passed in a state hash on #restore_state" do
+    @obm.restore_state({:position => 3, :direction => -1})
+    @obm.position.should == 3
+    @obm.direction.should == -1
   end
   
-  it "should return the named task with #find_task(name)" do
-    @obm.add_tasks(@task1, @task2, @task3)
-    @obm.find_task('task1').should == @task1
-    @obm.find_task('task3').should == @task3
+  it "should update its cache with #cache=" do
+    @obm.cache = {:foo => "bar"}
+    @obm.cache[:foo].should == "bar"
   end
   
   describe "(with a few tasks)" do
@@ -52,67 +58,111 @@ describe Outback::Manager do
       @obm.add_tasks(@task1, @task2)
     end
     
-    it "should call each task's #rollout! method, in order, on #rollout!" do
-      @task1.should_receive(:rollout!).and_return(true)
-      @task2.should_receive(:rollout!).and_return(true)
-      @obm.rollout!
+    it "should, by default, be rolling out" do
+      @obm.direction.should == Outback::Manager::ROLLOUT
     end
     
-    it "should call each task's #rollback! method, in reverse order, on #rollback!" do
-      @task1.should_receive(:rollout!).and_return(true)
-      @task2.should_receive(:rollout!).and_return(true)
-      @obm.rollout!
+    it "should start at the first task" do
+      @obm.position.should == 0
     end
     
-    it "should raise an Outback::Error if a rollout fails" do
-      @task1.stub!(:rollback!)
-      @task1.should_receive(:rollout!).and_return(false)
-      @task2.should_not_receive(:rollout!)
-      @task2.should_not_receive(:rollback!)
-      lambda { @obm.rollout! }.should raise_error(Outback::Error)
+    it "should report the tasks it will run in any given state with #tasks_to_run" do
+      @obm.add_task(@task3)
+      @obm.tasks_to_run.should == [@task1, @task2, @task3]
+      @obm.direction = -1
+      @obm.tasks_to_run.should be_empty
+      @obm.position = 3
+      @obm.tasks_to_run.should == [@task3, @task2, @task1]
+      @obm.position = 2
+      @obm.tasks_to_run.should == [@task2, @task1]
+      @obm.direction = 1
+      @obm.tasks_to_run.should == [@task3]
+    end
+        
+    it "should call each task's #rollout! method, in order, on #rollout!" do
+      @task1.should_receive(:rollout!)
+      @task2.should_receive(:rollout!)
+      @obm.rollout!.should == [@task1, @task2]
     end
     
-    it "should raise an Outback::TransactionError if a rollback subsequently fails" do
-      @task2.should_receive(:rollback!).and_return(false)
+    it "should not roll back any tasks if it isn't already rolled out" do
       @task1.should_not_receive(:rollback!)
-      lambda { @obm.rollback! }.should raise_error(Outback::TransactionError)
+      @task2.should_not_receive(:rollback!)
+      @obm.rollback!.should be_empty
     end
     
-  end
-  
-  describe "(with #workdir set)" do
+    it "should halt the rollout if any task's #rollout! method throws an error, and put the error in the cache" do
+      @error_task = mock(:error_task, :null_object => true)
+      @error_task.should_receive(:rollout!).and_raise("Just some runtime error.")
+      @obm.add_tasks(@error_task, @task2)
+      @obm.rollout!.should_not be_true
+      @obm.cache.should have(1).error
+      @obm.tasks_to_run.should == [@error_task, @task2]
+    end
     
-    before do
-      @task1 = Outback::ShellTask.new("", "")
-      @task1.workdir = "/tmp"
-      @task2 = Outback::ShellTask.new("", "")
-      @obm.workdir = "/nonexistent"
-      @obm.add_tasks(@task1, @task2)
+    it "should clear the cache errors before every rollout or rollback" do
+      @obm.cache[:errors] = "I'm an error."
+      @obm.rollout!
+      @obm.cache[:errors].should be_nil
+      @obm.cache[:errors] = "I'm an error."
+      @obm.rollback!
+      @obm.cache[:errors].should be_nil
+    end
+      
+    
+    it "should run each task in a transactional manner, so changes to the cache that are followed by an error are reversed" do
+      @error_task = Outback::Task.new do |t|
+        t.rollout do |m|
+          m.cache[:result] = "Foo"
+          raise "Some cockup!"
+        end
+      end
+      
+      @obm.add_tasks(@error_task, @task2)
+      @obm.rollout!.should_not be_true
+      @obm.cache[:result].should be_nil
+    end
+         
+    it "should call each task's #rollback! method, in reverse order, on #rollback!, provided it's already rolled out" do
+      @obm.position = 2
+      @task2.should_receive(:rollback!)
+      @task1.should_receive(:rollback!)
+      @obm.rollback!.should == [@task2, @task1]
     end
     
-    it "should set the workdir of each of its tasks if it is currently nil" do
-      @task1.workdir.should == "/tmp"
-      @task2.workdir.should == "/nonexistent"
+    it "should not roll out any tasks if it isn't already rolled back" do
+      @obm.position = 2
+      @task1.should_not_receive(:rollout!)
+      @task2.should_not_receive(:rollout!)
+      @obm.rollout!.should be_empty
+    end
+    
+    it "should pass an instance of TaskHelper into tasks' #rollout! and #rollback! methods" do
+      th = @obm.task_helper
+      @task1.should_receive(:rollout!).with(th)
+      @obm.rollout!
     end
     
   end
   
-  describe "(with #watcher set)" do
+  describe "(with observers)" do
     
     before do
-      @task1 = Outback::ShellTask.new("pwd", "")
-      @task2 = Outback::ShellTask.new("echo 'hello'", "")
-      require 'tempfile'
-      @obm.workdir = Dir.tmpdir
       @obm.add_tasks(@task1, @task2)
-      @watcher = mock("watcher")
-      @obm.watcher = @watcher
+
+      @watcher = mock(:watcher)
+      @watcher.stub!(:update)
+
+      @obm.add_observer(@watcher)   
     end
     
-    it "should call the watcher's #notify method after every task rollout/rollback with the task as an argument" do
-      @watcher.should_receive(:notify).ordered.with(@task1)
-      @watcher.should_receive(:notify).ordered.with(@task2)
+    it "should call the watcher's #update method after each rollout or rollback with a state hash and the results cache as parameters." do
+      @watcher.should_receive(:update).ordered.with({:position => 1, :direction => 1}, {})
+      @watcher.should_receive(:update).ordered.with({:position => 2, :direction => 1}, {})
+      @watcher.should_receive(:update).ordered.with({:position => 1, :direction => -1}, {})
+      @watcher.should_receive(:update).ordered.with({:position => 0, :direction => -1}, {})
       @obm.rollout!
+      @obm.rollback!
     end
     
   end

data/spec/outback/task_helper_spec.rb

@@ -0,0 +1,46 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Outback::TaskHelper do
+  
+  before do
+    @th = Outback::TaskHelper.new(@manager)
+  end
+  
+  it "should give an empty hash on #cache" do
+    @th.cache.should be_a_kind_of(Hash)
+    @th.cache.should be_empty
+  end
+    
+  it "should give the current working directory on #workdir" do
+    @th.workdir.should == Dir.getwd
+  end
+  
+  it "should provide a #sys method which makes a system call, returning a hash with stdout, stderr and exit_status" do
+    result = @th.sys("pwd")
+    result[:stdout].should == Dir.getwd + "\n"
+    result[:stderr].should == ""
+    result[:exit_status].should == 0
+  end
+  
+  describe "(with a manager set)" do
+    before do
+      @cache = {}
+      @manager = mock(:manager, :cache => @cache, :workdir => '/myworkdir')
+      @th = Outback::TaskHelper.new(@manager)
+    end
+    
+    it "should give the manager's cache on #cache" do
+      @th.cache.should == @cache
+    end
+    
+    it "should give the manager's workdir on #workdir" do
+      @th.workdir.should == '/myworkdir'
+    end
+    
+    it "should execute calls to sys in the workdir" do
+      @manager.should_receive(:workdir).and_return("/")
+      lambda { @result = @th.sys("pwd") }.should_not raise_error
+      @result[:stdout].should == "/\n"
+    end
+  end
+end

data/spec/outback/task_spec.rb

@@ -0,0 +1,41 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Outback::Task do
+  
+  before do
+    @task = Outback::Task.new
+  end
+
+  it "should take an optional block on its constructor, passing itself in as the block parameter" do
+    lambda { @task = Outback::Task.new }.should_not raise_error
+    temp = nil
+    @task = Outback::Task.new do |t|
+      temp = t
+    end
+    temp.should == @task
+  end
+    
+  it "should have blank rollout and rollback procs by default" do
+    @task.rollout.call.should be_nil
+    @task.rollback.call.should be_nil
+  end
+  
+  it "should define rollout and rollback tasks by calling #rollback and #rollout with a block" do
+    @task.rollout { 1 + 2 }
+    @task.rollout.call.should == 3
+    @task.rollback { 4 + 2 }
+    @task.rollback.call.should == 6
+  end
+  
+  describe "(with a rollout and rollback task defined)" do
+    before do
+      @task.rollout { 1 + 2 }
+      @task.rollback { 4 + 2 }
+    end
+    
+    it "should run the rollback/rollout procs when called with #rollout!/#rollback!, returning the return value of the proc" do
+      @task.rollout!.should == 3
+      @task.rollback!.should == 6
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: nickstenning-outback
 version: !ruby/object:Gem::Version 
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors: 
 - Nick Stenning
@@ -32,13 +32,16 @@ extra_rdoc_files: []
 
 files: 
 - lib/outback/manager.rb
-- lib/outback/shelltask.rb
-- lib/outback/yaml.rb
+- lib/outback/task.rb
+- lib/outback/task_helper.rb
 - lib/outback.rb
 - outback.gemspec
+- Rakefile
+- README.markdown
+- spec/fixtures/example_manager.rb
 - spec/outback/manager_spec.rb
-- spec/outback/shelltask_spec.rb
-- spec/outback/yaml_spec.rb
+- spec/outback/task_helper_spec.rb
+- spec/outback/task_spec.rb
 - spec/spec_helper.rb
 has_rdoc: true
 homepage: http://github.com/nickstenning/outback

data/lib/outback/shelltask.rb

@@ -1,53 +0,0 @@
-require 'rubygems'
-require 'open4'
-
-module Outback
-  
-  class ShellTask
-  
-    attr_accessor :rollout, :rollback, :workdir, :name
-    attr_reader :result, :errors, :exit_code
-    
-    def initialize(out, back)
-      @rollout, @rollback = out, back
-      @rolled_out = false
-    end
-    
-    def rollout!
-      run @rollout
-      @rolled_out = true
-      return @exit_code == 0
-    end
-    
-    def rollback!
-      run @rollback
-      @rolled_out = false
-      return @exit_code == 0
-    end
-    
-    def rolled_out?
-      @rolled_out
-    end
-        
-    def reset_strings
-      @result, @errors = "", ""
-    end
-    
-    def run( command )
-      reset_strings
-      Dir.chdir(@workdir || Dir.getwd) do
-        @status = Open4.popen4(*command) do |pid, i, o, e|
-          @result = o.read
-          @errors = e.read
-        end
-        @exit_code = @status.exitstatus
-      end
-    # Catch nonexistent commands at this point and return a sensible error
-    rescue Errno::ENOENT => e
-      @exit_code = 127
-      @errors = e.message
-    end
-    
-  end
-  
-end

data/lib/outback/yaml.rb

@@ -1,22 +0,0 @@
-module Outback
-  
-  module YAML
-    
-    def self.load( yaml_string )
-      manager = Manager.new
-      tasks = ::YAML.load(yaml_string)
-      tasks.each do |task|
-        task = [task['out'], task['back']]
-        manager.tasks << Outback::ShellTask.new(*task)
-      end
-      return manager
-    end
-    
-    def self.load_file( yaml_file )
-      yaml = File.read( yaml_file )
-      load( yaml )
-    end
-              
-  end
-  
-end

data/spec/outback/shelltask_spec.rb

@@ -1,92 +0,0 @@
-require File.dirname(__FILE__) + '/../spec_helper'
-
-describe Outback::ShellTask do
-  
-  before do
-    @task = Outback::ShellTask.new("out", "back")
-  end
-  
-  it "should have an attr_accessor for @name" do
-    @task.name.should be_nil
-    @task.name = "Henry"
-    @task.name.should == "Henry"
-  end
-  
-  it "should use the first parameter to the rollout command" do
-    @task.rollout.should == "out"
-  end
-  
-  it "should use the second parameter as the rollback command" do
-    @task.rollback.should == "back"
-  end
-  
-  it "should allow its rollout command to be changed with #rollout=" do
-    @task.rollout = "foo"
-    @task.rollout.should == "foo"
-  end
-  
-  it "should allow its rollback command to be changed with #rollback=" do
-    @task.rollout = "bar"
-    @task.rollout.should == "bar"
-  end
-  
-  it "should not be rolled out by default" do
-    @task.should_not be_rolled_out
-  end
-  
-  it "should rollout when called with #rollout!" do
-    @task.rollout!
-    @task.should be_rolled_out
-  end
-  
-  it "should run its commands in the current working directory if #workdir hasn't been set" do
-    @task = Outback::ShellTask.new("pwd", "")
-    @task.rollout!
-    @task.result.chomp.should == Dir.getwd
-  end
-  
-  it "should run its commands in the workdir, if set with #workdir=" do
-    @task = Outback::ShellTask.new("pwd", "")
-    @task.workdir = "/private/tmp" ## FIXME for *nix systems
-    @task.rollout!
-    @task.result.chomp.should == @task.workdir
-  end
-  
-  it "should return true on successful #rollout! or #rollback!" do
-    @task = Outback::ShellTask.new("echo", "echo")
-    @task.rollout!.should be_true
-    @task.rollback!.should be_true
-  end
-  
-  it "should return false on unsuccessful #rollout! or #rollback!" do
-    @task = Outback::ShellTask.new("I-am-a-command-most-unlikely-to-exist", "I-am-a-command-most-unlikely-to-exist")
-    @task.rollout!.should be_false
-    @task.rollback!.should be_false
-  end
-  
-  describe "(when rolled out)" do
-    
-    before do
-      @task = Outback::ShellTask.new('pwd', 'pwd')
-      @task.rollout!
-    end
-    
-    it "should be rolled out" do
-      @task.should be_rolled_out
-    end
-    
-    it "should have an integer exit code" do
-      @task.exit_code.should be_a_kind_of(Fixnum)
-    end
-    
-    it "should have a string result" do
-      @task.result.should be_a_kind_of(String)
-    end
-    
-    it "should have a string for errors (stderr)" do
-      @task.errors.should be_a_kind_of(String)
-    end
-    
-  end
-
-end

data/spec/outback/yaml_spec.rb

@@ -1,47 +0,0 @@
-require File.dirname(__FILE__) + '/../spec_helper'
-require 'yaml'
-
-describe Outback::YAML do
-  
-  before do
-    @yaml =<<-EOM.gsub(/^\s{4}/, '')
-    -
-      out: touch y
-      back: rm y
-    -
-      out: touch x
-      back: rm x
-    EOM
-    @file = '/tmp/outback_yaml_spec_tmp'
-    File.open(@file, 'w') { |f| f.puts @yaml }
-  end
-  
-  after do
-    File.delete(@file)
-  end
-  
-  describe do
-    it "should return an Outback::Manager object" do 
-      Outback::YAML.load(@yaml).should be_a_kind_of(Outback::Manager)
-      Outback::YAML.load_file(@file).should be_a_kind_of(Outback::Manager)
-    end
-    it "should return an Outback::Manager object with as many tasks as the YAML passed into it" do
-      Outback::YAML.load(@yaml).should have(2).tasks
-      Outback::YAML.load_file(@file).should have(2).tasks
-    end
-    it "should add instances of Outback::ShellTask to the Outback::Manager's task array" do
-      manager = Outback::YAML.load(@yaml)
-      manager.tasks.first.should be_a_kind_of(Outback::ShellTask)
-    end
-    it "should set the rollout_command to the first element of the YAML array" do
-      manager = Outback::YAML.load(@yaml)
-      manager.tasks[0].rollout.should == "touch y"
-      manager.tasks[1].rollout.should == "touch x"
-    end
-    it "should set the rollback_command to the second element of the YAML array" do
-      manager = Outback::YAML.load(@yaml)
-      manager.tasks[0].rollback.should == "rm y"
-      manager.tasks[1].rollback.should == "rm x"
-    end
-  end
-end