boxgrinder-build 0.10.0 → 0.10.1

data/spec/plugins/delivery/local/local-plugin-spec.rb

@@ -16,8 +16,6 @@
 # Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
 # 02110-1301 USA, or see the FSF site: http://www.fsf.org.
 
-require 'rubygems'
-require 'rspec'
 require 'boxgrinder-build/plugins/delivery/local/local-plugin'
 require 'hashery/opencascade'
 

data/spec/plugins/delivery/s3/s3-plugin-spec.rb

@@ -16,8 +16,6 @@
 # Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
 # 02110-1301 USA, or see the FSF site: http://www.fsf.org.
 
-require 'rubygems'
-require 'rspec'
 require 'boxgrinder-build/plugins/delivery/s3/s3-plugin'
 require 'hashery/opencascade'
 require 'boxgrinder-core/models/config'
@@ -109,10 +107,10 @@ module BoxGrinder
         @key1.should_receive(:exists?).and_return(true)
         @key2.should_receive(:exists?).and_return(false)
 
-        @s3helper.should_receive(:stub_s3obj).with(@bucket, 'name/fedora/14/1.0-SNAPSHOT-1/x86_64/').and_return(@key1)
-        @s3helper.should_receive(:stub_s3obj).with(@bucket, 'name/fedora/14/1.0-SNAPSHOT-2/x86_64/').and_return(@key2)
+        @s3helper.should_receive(:stub_s3obj).with(@bucket, 'name/fedora/14/1.0-snapshot-1/x86_64/').and_return(@key1)
+        @s3helper.should_receive(:stub_s3obj).with(@bucket, 'name/fedora/14/1.0-snapshot-2/x86_64/').and_return(@key2)
 
-        @plugin.ami_key("name", "/").should == "name/fedora/14/1.0-SNAPSHOT-2/x86_64"
+        @plugin.ami_key("name", "/").should == "name/fedora/14/1.0-snapshot-2/x86_64"
       end
 
       it "should return valid ami_key with snapshot and overwrite enabled" do
@@ -121,10 +119,10 @@ module BoxGrinder
         @key1.should_receive(:exists?).and_return(true)
         @key2.should_receive(:exists?).and_return(false)
 
-        @s3helper.should_receive(:stub_s3obj).with(@bucket, 'name/fedora/14/1.0-SNAPSHOT-1/x86_64/').and_return(@key1)
-        @s3helper.should_receive(:stub_s3obj).with(@bucket, 'name/fedora/14/1.0-SNAPSHOT-2/x86_64/').and_return(@key2)
+        @s3helper.should_receive(:stub_s3obj).with(@bucket, 'name/fedora/14/1.0-snapshot-1/x86_64/').and_return(@key1)
+        @s3helper.should_receive(:stub_s3obj).with(@bucket, 'name/fedora/14/1.0-snapshot-2/x86_64/').and_return(@key2)
 
-        @plugin.ami_key("name", "/").should == "name/fedora/14/1.0-SNAPSHOT-1/x86_64"
+        @plugin.ami_key("name", "/").should == "name/fedora/14/1.0-snapshot-1/x86_64"
       end
     end
 

data/spec/plugins/delivery/sftp/sftp-plugin-spec.rb

@@ -16,8 +16,6 @@
 # Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
 # 02110-1301 USA, or see the FSF site: http://www.fsf.org.
 
-require 'rubygems'
-require 'rspec'
 require 'boxgrinder-build/plugins/delivery/sftp/sftp-plugin'
 
 module BoxGrinder

data/spec/plugins/os/centos/centos-plugin-spec.rb

@@ -16,8 +16,6 @@
 # Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
 # 02110-1301 USA, or see the FSF site: http://www.fsf.org.
 
-require 'rubygems'
-require 'rspec'
 require 'boxgrinder-build/plugins/os/centos/centos-plugin'
 require 'hashery/opencascade'
 

data/spec/plugins/os/fedora/fedora-plugin-spec.rb

@@ -16,8 +16,6 @@
 # Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
 # 02110-1301 USA, or see the FSF site: http://www.fsf.org.
 
-require 'rubygems'
-require 'rspec'
 require 'boxgrinder-build/plugins/os/fedora/fedora-plugin'
 require 'hashery/opencascade'
 

data/spec/plugins/os/rhel/rhel-plugin-spec.rb

@@ -16,8 +16,6 @@
 # Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
 # 02110-1301 USA, or see the FSF site: http://www.fsf.org.
 
-require 'rubygems'
-require 'rspec'
 require 'boxgrinder-build/plugins/os/rhel/rhel-plugin'
 require 'hashery/opencascade'
 

data/spec/plugins/os/rpm-based/kickstart-spec.rb

@@ -16,8 +16,6 @@
 # Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
 # 02110-1301 USA, or see the FSF site: http://www.fsf.org.
 
-require 'rubygems'
-require 'rspec'
 require 'boxgrinder-build/plugins/os/rpm-based/kickstart'
 require 'hashery/opencascade'
 

data/spec/plugins/os/rpm-based/rpm-based-os-plugin-spec.rb

@@ -16,8 +16,6 @@
 # Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
 # 02110-1301 USA, or see the FSF site: http://www.fsf.org.
 
-require 'rubygems'
-require 'rspec'
 require 'boxgrinder-build/plugins/os/rpm-based/rpm-based-os-plugin'
 require 'hashery/opencascade'
 
@@ -51,6 +49,19 @@ module BoxGrinder
       @log = @plugin.instance_variable_get(:@log)
     end
 
+    it "should create the /etc/yum.repos.d directory if it does not exist" do
+      @appliance_config.stub!(:repos).and_return([
+        {'name' => 'cirras', 'baseurl' => "http://repo.boxgrinder.org/packages/fedora/11/RPMS/x86_64"}
+      ])
+
+      guestfs = mock("guestfs").as_null_object
+
+      guestfs.stub!(:exists).and_return(0)
+      guestfs.should_receive(:mkdir_p).with("/etc/yum.repos.d/")
+
+      @plugin.install_repos(guestfs)
+    end
+
     it "should install repos" do
       @appliance_config.should_receive(:repos).and_return(
           [
@@ -58,7 +69,8 @@ module BoxGrinder
               {'name' => 'abc', 'baseurl' => 'http://abc', 'mirrorlist' => "http://abc.org/packages/fedora/11/RPMS/x86_64"},
           ])
 
-      guestfs = mock("guestfs")
+      guestfs = mock("guestfs").as_null_object
+
       guestfs.should_receive(:write_file).with("/etc/yum.repos.d/cirras.repo", "[cirras]\nname=cirras\nenabled=1\ngpgcheck=0\nbaseurl=http://repo.boxgrinder.org/packages/fedora/11/RPMS/x86_64\n", 0)
       guestfs.should_receive(:write_file).with("/etc/yum.repos.d/abc.repo", "[abc]\nname=abc\nenabled=1\ngpgcheck=0\nbaseurl=http://abc\nmirrorlist=http://abc.org/packages/fedora/11/RPMS/x86_64\n", 0)
 
@@ -72,7 +84,7 @@ module BoxGrinder
               {'name' => 'cirras', 'baseurl' => "http://repo.boxgrinder.org/packages/fedora/11/RPMS/x86_64", 'ephemeral' => true}
           ])
 
-      guestfs = mock("guestfs")
+      guestfs = mock("guestfs").as_null_object
       guestfs.should_receive(:write_file).with("/etc/yum.repos.d/abc.repo", "[abc]\nname=abc\nenabled=1\ngpgcheck=0\nbaseurl=http://abc\nmirrorlist=http://abc.org/packages/fedora/11/RPMS/x86_64\n", 0)
 
       @plugin.install_repos(guestfs)

data/spec/plugins/os/rpm-based/rpm-dependency-validator-spec.rb

@@ -16,8 +16,6 @@
 # Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
 # 02110-1301 USA, or see the FSF site: http://www.fsf.org.
 
-require 'rubygems'
-require 'rspec'
 require 'boxgrinder-build/plugins/os/rpm-based/rpm-dependency-validator'
 require 'hashery/opencascade'
 

data/spec/plugins/platform/ec2/ec2-plugin-spec.rb

@@ -16,8 +16,6 @@
 # Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
 # 02110-1301 USA, or see the FSF site: http://www.fsf.org.
 
-require 'rubygems'
-require 'rspec'
 require 'boxgrinder-build/plugins/platform/ec2/ec2-plugin'
 require 'boxgrinder-core/helpers/log-helper'
 require 'hashery/opencascade'

data/spec/plugins/platform/virtualbox/virtualbox-plugin-spec.rb

@@ -16,11 +16,9 @@
 # Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
 # 02110-1301 USA, or see the FSF site: http://www.fsf.org.
 
-require 'rubygems'
-require 'rspec'
 require 'boxgrinder-build/plugins/platform/virtualbox/virtualbox-plugin'
 require 'hashery/opencascade'
-require 'spec/rspec-plugin-helper.rb'
+require 'rspec-plugin-helper'
 
 module BoxGrinder
   describe VirtualBoxPlugin do

data/spec/plugins/platform/virtualpc/virtualpc-plugin-spec.rb

@@ -16,7 +16,6 @@
 # Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
 # 02110-1301 USA, or see the FSF site: http://www.fsf.org.
 
-require 'rubygems'
 require 'boxgrinder-build/plugins/platform/virtualpc/virtualpc-plugin.rb'
 
 module BoxGrinder

data/spec/plugins/platform/vmware/vmware-plugin-spec.rb

@@ -16,8 +16,6 @@
 # Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
 # 02110-1301 USA, or see the FSF site: http://www.fsf.org.
 
-require 'rubygems'
-require 'rspec'
 require 'boxgrinder-build/plugins/platform/vmware/vmware-plugin'
 require 'hashery/opencascade'
 
@@ -122,7 +120,7 @@ module BoxGrinder
 
         vmdk_image = @plugin.change_vmdk_values("vmfs")
 
-        vmdk_image.scan(/^createType="(.*)"\s?$/).to_s.should == "vmfs"
+        vmdk_image.match(/^createType="(.*)"\s?$/)[1].should == "vmfs"
 
         disk_attributes = vmdk_image.scan(/^RW (.*) (.*) "(.*).raw" (.*)\s?$/)[0]
 
@@ -131,11 +129,11 @@ module BoxGrinder
         disk_attributes[2].should == "full"
         disk_attributes[3].should == ""
 
-        vmdk_image.scan(/^ddb.geometry.cylinders = "(.*)"\s?$/).to_s.should == "652"
-        vmdk_image.scan(/^ddb.geometry.heads = "(.*)"\s?$/).to_s.should == "255"
-        vmdk_image.scan(/^ddb.geometry.sectors = "(.*)"\s?$/).to_s.should == "63"
+        vmdk_image.match(/^ddb.geometry.cylinders = "(.*)"\s?$/)[1].should == "652"
+        vmdk_image.match(/^ddb.geometry.heads = "(.*)"\s?$/)[1].should == "255"
+        vmdk_image.match(/^ddb.geometry.sectors = "(.*)"\s?$/)[1].should == "63"
 
-        vmdk_image.scan(/^ddb.virtualHWVersion = "(.*)"\s?$/).to_s.should == "7"
+        vmdk_image.match(/^ddb.virtualHWVersion = "(.*)"\s?$/)[1].should == "7"
       end
 
       it "should change vmdk data (flat)" do
@@ -145,7 +143,7 @@ module BoxGrinder
 
         vmdk_image = @plugin.change_vmdk_values("monolithicFlat")
 
-        vmdk_image.scan(/^createType="(.*)"\s?$/).to_s.should == "monolithicFlat"
+        vmdk_image.match(/^createType="(.*)"\s?$/)[1].should == "monolithicFlat"
 
         disk_attributes = vmdk_image.scan(/^RW (.*) (.*) "(.*).raw" (.*)\s?$/)[0]
 
@@ -154,12 +152,12 @@ module BoxGrinder
         disk_attributes[2].should == "full"
         disk_attributes[3].should == "0"
 
-        vmdk_image.scan(/^ddb.geometry.cylinders = "(.*)"\s?$/).to_s.should == "652"
-        vmdk_image.scan(/^ddb.geometry.heads = "(.*)"\s?$/).to_s.should == "255"
-        vmdk_image.scan(/^ddb.geometry.sectors = "(.*)"\s?$/).to_s.should == "63"
+        vmdk_image.match(/^ddb.geometry.cylinders = "(.*)"\s?$/)[1].should == "652"
+        vmdk_image.match(/^ddb.geometry.heads = "(.*)"\s?$/)[1].should == "255"
+        vmdk_image.match(/^ddb.geometry.sectors = "(.*)"\s?$/)[1].should == "63"
 
-        vmdk_image.scan(/^ddb.virtualHWVersion = "(.*)"\s?$/).to_s.should == "7"
-        vmdk_image.scan(/^ddb.thinProvisioned = "(.*)"\s?$/).to_s.should == "0"
+        vmdk_image.match(/^ddb.virtualHWVersion = "(.*)"\s?$/)[1].should == "7"
+        vmdk_image.match(/^ddb.thinProvisioned = "(.*)"\s?$/)[1].should == "0"
       end
 
       it "should change vmdk data (flat) enabling thin disk" do
@@ -169,7 +167,7 @@ module BoxGrinder
 
         vmdk_image = @plugin.change_vmdk_values("monolithicFlat")
 
-        vmdk_image.scan(/^ddb.thinProvisioned = "(.*)"\s?$/).to_s.should == "1"
+        vmdk_image.match(/^ddb.thinProvisioned = "(.*)"\s?$/)[1].should == "1"
       end
     end
 
@@ -177,16 +175,19 @@ module BoxGrinder
       prepare_image({'thin_disk' => false, 'type' => 'enterprise'})
 
       vmx_file = @plugin.change_common_vmx_values
-
-      vmx_file.scan(/^guestOS = "(.*)"\s?$/).to_s.should == "linux"
-      vmx_file.scan(/^displayName = "(.*)"\s?$/).to_s.should == "full"
-      vmx_file.scan(/^annotation = "(.*)"\s?$/).to_s.scan(/^full | Version: 1\.0 | Built by: BoxGrinder 1\.0\.0/).should_not == nil
-      vmx_file.scan(/^guestinfo.vmware.product.long = "(.*)"\s?$/).to_s.should == "full"
-      vmx_file.scan(/^guestinfo.vmware.product.url = "(.*)"\s?$/).to_s.should == "http://boxgrinder.org"
-      vmx_file.scan(/^numvcpus = "(.*)"\s?$/).to_s.should == "1"
-      vmx_file.scan(/^memsize = "(.*)"\s?$/).to_s.should == "256"
-      vmx_file.scan(/^log.fileName = "(.*)"\s?$/).to_s.should == "full.log"
-      vmx_file.scan(/^scsi0:0.fileName = "(.*)"\s?$/).to_s.should == "full.vmdk"
+      
+      vmx_file.match(/^guestOS = "(.*)"\s?$/)[1].should == "linux"
+      vmx_file.match(/^displayName = "(.*)"\s?$/)[1].should == "full"
+
+      vmx_file.match(/^annotation = "(.*)"\s?$/)[1].
+        match(/^full | Version: 1\.0 | Built by: BoxGrinder 1\.0\.0/).should_not == nil
+      
+      vmx_file.match(/^guestinfo.vmware.product.long = "(.*)"\s?$/)[1].should == "full"
+      vmx_file.match(/^guestinfo.vmware.product.url = "(.*)"\s?$/)[1].should == "http://boxgrinder.org"
+      vmx_file.match(/^numvcpus = "(.*)"\s?$/)[1].should == "1"
+      vmx_file.match(/^memsize = "(.*)"\s?$/)[1].should == "256"
+      vmx_file.match(/^log.fileName = "(.*)"\s?$/)[1].should == "full.log"
+      vmx_file.match(/^scsi0:0.fileName = "(.*)"\s?$/)[1].should == "full.vmdk"
     end
 
     describe ".build_vmware_personal" do

data/spec/rcov_helper.rb

@@ -0,0 +1,2 @@
+require 'rubygems'
+require 'rspec'

data/spec/spec_helper.rb

@@ -0,0 +1,9 @@
+if ENV['COVERAGE']
+  require 'simplecov'
+
+  FILTER_DIRS = ['spec']
+
+  SimpleCov.start do
+    FILTER_DIRS.each{ |f| add_filter f }
+  end
+end

data/spec/util/concurrent/get-set-spec.rb

@@ -0,0 +1,43 @@
+require "rspec"
+
+require 'boxgrinder-build/util/concurrent/get_set'
+
+describe GetSet do
+  subject{ GetSet.new }
+  its(:get_set){ should be_false }
+
+  context "#initialize" do
+    it "should have a value of true with default init" do
+      GetSet.new.get_set.should be_false
+    end
+
+    it "should have an arbitrary value after setting initial_state" do
+      GetSet.new(:boxgrinder).get_set.should equal(:boxgrinder)
+    end
+  end
+
+  context "#get_set" do
+    context "without block" do
+      it "should return the existing value when none is set" do
+        subject.get_set(:some_value)
+        subject.get_set.should equal(:some_value)
+      end
+
+      it "should return the set non-nil value when explicitly provided" do
+        subject.get_set(:boxgrinder).should equal(:boxgrinder)
+      end
+    end
+
+    context "with block" do
+      it "yield existing the value and set block return as the new value" do
+        expect{ subject.get_set(:blah){|v| :boo} }.
+          to change(subject, :get_set).from(false).to(:boo)
+      end
+
+      it "parameter should be ignored" do
+        expect{ subject.get_set(:blah){|v| v} }.
+          not_to change(subject, :get_set)
+      end
+    end
+  end
+end

data/spec/util/permissions/fs-monitor-spec.rb

@@ -0,0 +1,233 @@
+#
+# Copyright 2012 Red Hat, Inc.
+#
+# This is free software; you can redistribute it and/or modify it
+# under the terms of the GNU Lesser General Public License as
+# published by the Free Software Foundation; either version 3 of
+# the License, or (at your option) any later version.
+#
+# This software is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+# Lesser General Public License for more details.
+#
+# You should have received a copy of the GNU Lesser General Public
+# License along with this software; if not, write to the Free
+# Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+# 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+
+require 'boxgrinder-build/util/permissions/fs-monitor'
+
+module BoxGrinder
+  describe FSMonitor do     
+
+    before(:each) do
+      # Singleton
+      @fs_monitor = FSMonitor.instance
+    end
+
+    after(:each) do
+      # Reset the singleton to avoid tests being serial
+      @fs_monitor.reset
+    end
+
+    subject{ @fs_monitor }
+ 
+    let(:observer){ mock('FSObserver-mock-1', :respond_to? => true) }
+    let(:observer_2){ mock('FSObserver-mock-2', :respond_to? => true) }
+
+    context "after singleton #initialize" do
+      its(:reset){ should be_nil }
+      its(:stop){ should be_false }
+      its(:trigger){ should be_false } 
+    
+      it "should insert wrappers for File{#open, #new, #rename, #symlink, #link}, 
+         Dir#mkdir." do
+        File.should respond_to(:__alias_open, :__alias_new, :__alias_rename, 
+          :__alias_symlink, :__alias_link)
+        File.should respond_to(:open, :new, :rename, :symlink, :link)
+        Dir.should respond_to(:__alias_mkdir)
+        Dir.should respond_to(:mkdir)
+      end
+    end
+
+    context "#capture" do
+      let(:mock_return){ mock('mock_return_value') }
+      let(:dumb_observer){ mock('dumb_observer').as_null_object }
+      let(:dumb_observer_2){ mock('dumb_observer_2').as_null_object }
+      
+      before(:each) do
+        # Just reflect the path back so we can differentiate the paths
+        subject.stub(:realpath){ |reflect| reflect }
+      end
+
+      it "should add observers to its observer set" do
+        expect {
+          subject.capture(dumb_observer, dumb_observer_2){}
+        }.to change(subject, :count_observers).
+          from(0).to(2)
+      end
+
+      def basic_capture_test_setup(klazz=File, meth_s=:open, &blk)
+        klazz.stub("__alias_#{meth_s}").and_return(mock_return)
+
+        observer.should_receive(:update).twice.ordered.
+          with({ :command => :stop_capture })
+
+        blk.call(observer)
+      end
+
+      it "should not capture outside of #capture block{}" do
+        basic_capture_test_setup do |observer|
+          # Ignore unscoped operations
+          observer.should_receive(:update).once.
+            with({ :data => '/a/y/r/s', :command => :add_path })
+
+          File.open('/m/a/s', 'w+')
+
+          subject.capture(observer) do 
+            File.open('/a/y/r/s', 'a') 
+          end
+
+          File.open('/c/y/s', 'w')
+        end
+      end
+
+      it "should capture with manual #start, #stop delimiters" do
+        basic_capture_test_setup do |observer|
+          observer.should_receive(:update).once.
+            with({ :data => '/m/a/w', :command => :add_path })
+
+          File.open('no-capture', 'w+')
+          
+          # Note: without block, hence no auto-stop
+          subject.capture(observer) 
+          File.open('/m/a/w', 'a') 
+          subject.stop
+
+          File.open('capture-no', 'w')
+        end        
+      end
+      
+      def wrapper_spec_builder(klazz, method, pnum, include, exclude=[], &blk)
+        basic_capture_test_setup(klazz, method) do |observer|
+
+          include.each do |arg|
+            observer.should_receive(:update).once.
+              with({ :data => arg[pnum], :command => :add_path})       
+          end
+
+          subject.capture(observer) do
+            (include+exclude).each{ |arg| klazz.send(method, *arg) }
+          end
+        end
+      end
+
+      let(:mode_new_inc) do
+          [['/ham/jam', 'a+'], ['/ham/jam', 'a'], 
+          ['/spam', 'w+'], ['/spam', 'w']] 
+      end
+
+      let(:mode_new_exc){ [['/nocapture', 'r']] }
+
+      it "should capture File#new when mode=/^(t|b)?((w|a)[+]?)(t|b)?$/" do
+        wrapper_spec_builder(File, :new, 0, mode_new_inc, 
+          mode_new_exc)
+      end
+
+      it "should capture File#open when mode=/^(t|b)?((w|a)[+]?)(t|b)?$/" do
+        wrapper_spec_builder(File, :open, 0, mode_new_inc, 
+          mode_new_exc)
+      end
+      
+      it "should capture File#rename new path" do
+        wrapper_spec_builder(File, :rename, 1, 
+          # original -> expectation
+          [['/old/path', '/new/path']])
+      end
+
+      it "should capture File#symlink" do
+        wrapper_spec_builder(File, :symlink, 1,
+          [['/some/file', '/the/symlink']])
+      end
+
+      it "should capture File#link" do
+        wrapper_spec_builder(File, :link, 1,
+          [['/some/file', '/the/hardlink']])
+      end
+
+      it "should capture Dir#mkdir" do
+        basic_capture_test_setup(Dir, :mkdir) do |observer|
+          observer.should_receive(:update).once.
+            with({ :data => '/a_new_dir/', :command => :add_path })
+          
+          subject.capture(observer) do
+            Dir.mkdir('/a_new_dir/and/other/junk')
+          end
+        end
+      end
+    end
+
+    context "#add" do      
+      context "valid usage" do
+        before(:each) do
+          subject.stub(:realpath).and_return('/custard/creme')
+          # Stub the 'real' method, use generated wrapper.
+          Dir.stub(:__alias_mkdir).and_return(0)
+        end
+
+        context "no observers provided" do
+          it "should return false, but most not raise any errors" do
+            subject.add_path('/exploding/sausage').should be_false
+          end
+        end
+
+        # NOTE: Normally #stop would only be received once, but due to
+        # resetting in after(:each) it is received twice in these
+        # specs.
+        it "should send captured paths to observers when capturing stopped" do
+          observer.should_receive(:update).once.
+            with({ :data => '/custard/creme', :command => :add_path }) 
+
+          observer.should_receive(:update).twice.
+            with({ :command => :stop_capture })
+
+          subject.capture(observer){ Dir.mkdir('custard/creme') }
+        end
+
+        it "should accept and update multiple observers" do
+          [observer, observer_2].each do |o| 
+            o.should_receive(:update).once.
+              with({ :data => '/custard/creme', :command => :add_path })
+          end
+
+          [observer, observer_2].each do |o| 
+            o.should_receive(:update).twice.
+              with({ :command => :stop_capture }) 
+          end
+
+          subject.capture(observer, observer_2) do
+            Dir.mkdir('custard/creme')
+          end
+        end
+      end
+    end
+    
+    context "#trigger" do
+      before(:each) do
+        subject.stub(:_stop)
+      end
+
+      it "should return false when there are no observers, but *not* fail" do
+        expect{ subject.trigger }.to be_true
+      end
+
+      it "should send a :chown update to observers" do
+        observer.should_receive(:update).with({:command => :chown})
+
+        subject.send(:add_observer, observer)
+        subject.trigger
+      end
+    end
+  end
+end