poolparty 0.2.69 → 0.2.84

data/spec/poolparty/modules/cloud_resourcer_spec.rb

@@ -2,7 +2,11 @@ require File.dirname(__FILE__) + '/../spec_helper'
 
 class ResourcerTestClass
   include CloudResourcer
-  include Configurable  
+  include Configurable
+  
+  default_options({
+    :minimum_runtime => 50.minutes
+  })
   
   def initialize(&block)
     store_block(&block) if block
@@ -35,10 +39,23 @@ describe "CloudResourcer" do
     @tc.instances 4..10
     @tc.minimum_instances.should == 4
   end
-  it "should set the max to the maximum instances to the last" do
+  it "should be able to accept a Fixnum and set the minimum_instances and maximum_instances" do
+    @tc.instances 1
+    @tc.minimum_instances.should == 1
+    @tc.maximum_instances.should == 1
+  end
+  it "should set the max to the maximum instances to the last in a given range" do
     @tc.instances 4..10
     @tc.maximum_instances.should == 10
   end
+  it "should have default minimum_runtime of 50 minutes (3000 seconds)" do
+    Base.stub!(:minimum_runtime).and_return 50.minutes
+    @tc.minimum_runtime.should ==  50.minutes
+  end
+  it "should have minimum_runtime" do
+    @tc.minimum_runtime 40.minutes
+    @tc.minimum_runtime.should == 40.minutes
+  end
   describe "keypair_path" do
     before(:each) do
     end

data/spec/poolparty/modules/configurable_spec.rb

@@ -1,12 +1,12 @@
 require File.dirname(__FILE__) + '/../spec_helper'
 
-class TestClass
+class TestClass2
   include Configurable
 end
 
 describe "configurable" do
   before(:each) do
-    @tc = TestClass.new
+    @tc = TestClass2.new
   end
   it "should set the name as frank" do
     @tc.configure({:name => "frank"})

data/spec/poolparty/net/remote_bases/ec2_spec.rb

@@ -2,7 +2,7 @@ require File.dirname(__FILE__) + '/../../spec_helper'
 
 # include Remote
 
-class TestClass  
+class TestEC2Class  
   include PoolParty::Remote::RemoterBase
   include Ec2
   include CloudResourcer
@@ -14,6 +14,9 @@ class TestClass
   
   def ami;"ami-abc123";end
   def size; "small";end
+  def security_group; "default";end
+  def ebs_volume_id; "ebs_volume_id";end
+  def availabilty_zone; "us-east-1a";end
   def verbose
     false
   end
@@ -24,7 +27,7 @@ end
 describe "ec2 remote base" do
   before(:each) do
     setup
-    @tr = TestClass.new
+    @tr = TestEC2Class.new
     stub_remoter_for(@tr)
     @tr.stub!(:get_instances_description).and_return response_list_of_instances
   end

data/spec/poolparty/net/remote_instance_spec.rb

@@ -1,6 +1,7 @@
 require File.dirname(__FILE__) + '/../spec_helper'
 
 include Remote
+include Aska
 
 describe "Remote Instance" do
   before(:each) do
@@ -58,6 +59,10 @@ describe "Remote Instance" do
     before(:each) do
       @ri = RemoteInstance.new(@valid_hash, nil)
     end
+    it "should give the elapsed time" do
+      @ri.stub!(:launching_time).and_return(30.minutes.ago)
+      @ri.elapsed_runtime.should be >= 1800
+    end
     it "should be say that it is the master if the name is master" do
       @ri.name.should == "master"
       @ri.master?.should == true

data/spec/poolparty/net/remote_spec.rb

@@ -10,20 +10,11 @@ module Hype
   end
   register_remote_base :Hype
 end
-class TestClass
-  include Remote
-  include RemoterBase
-  
-  def keypair
-    "fake_keypair"
-  end
-  def verbose
-    false
-  end
-end
 
 describe "Remote" do
   before(:each) do
+    @cloud = cloud :test_cloud do;end
+    
     @tc = TestClass.new
     @tc.stub!(:verbose).and_return false
     setup
@@ -32,6 +23,9 @@ describe "Remote" do
     @tc.respond_to?(:using).should == true
   end
   it "should include the module with using" do
+    @tc.instance_eval do
+      @remote_base = nil
+    end
     @tc.should_receive(:extend).with("Hype".preserved_module_constant).once
     @tc.using :hype
   end
@@ -59,6 +53,9 @@ describe "Remote" do
       @tc.using :hype
     end
     it "should only include the remote class once" do
+      @tc.instance_eval do
+        @remote_base = nil
+      end
       @tc.should_receive(:extend).with(Hype).once
       @tc.using :hype
       @tc.using :hype
@@ -106,6 +103,8 @@ describe "Remote" do
       end
       it "should be true if there are" do
         add_stub_instance_to(@tc, 8)
+        add_stub_instance_to(@tc, 9)
+        add_stub_instance_to(@tc, 10)
         @tc.minimum_number_of_instances_are_running?.should == true
       end
     end
@@ -191,10 +190,6 @@ describe "Remote" do
         @tc.request_termination_of_non_master_instance
       end
     end
-    describe "should_expand_cloud?" do
-    end
-    describe "should_contract_cloud?" do
-    end
     describe "expansions" do
       before(:each) do
         setup
@@ -214,13 +209,21 @@ describe "Remote" do
         before(:each) do
           setup
           stub_list_from_remote_for(@tc)
-          @tc.stub!(:request_launch_new_instances).and_return PoolParty::Remote::RemoteInstance.new(:ip => "127.0.0.1", :num => 1)
+          @ri = PoolParty::Remote::RemoteInstance.new(:ip => "127.0.0.1", :num => 1, :name => "master")
+          @tc.stub!(:request_launch_new_instances).and_return @ri
           @tc.stub!(:can_start_a_new_instance).and_return true
           @tc.stub!(:list_of_pending_instances).and_return []
           @tc.stub!(:prepare_for_configuration).and_return true
           @tc.stub!(:build_and_store_new_config_file).and_return true          
           PoolParty::Provisioner.stub!(:provision_slaves).and_return true
-          Kernel.stub!(:system).and_return true
+          @cloud.stub!(:master).and_return @ri
+          @cloud.stub!(:list_of_nonterminated_instances).and_return [@ri]
+          @cloud.stub!(:full_keypair_path).and_return "keyairs"
+          
+          @provisioner = PoolParty::Provisioner::Capistrano.new(@ri, @cloud, :ubuntu)
+          PoolParty::Provisioner::Capistrano.stub!(:new).and_return @provisioner
+          @provisioner.stub!(:install).and_return true
+          @provisioner.stub!(:configure).and_return true
         end
         it "should receive can_start_a_new_instance?" do
           @tc.should_receive(:can_start_a_new_instance?).once
@@ -234,12 +237,7 @@ describe "Remote" do
         end
         it "should call a new slave provisioner" do
           @tc.stub!(:should_expand_cloud?).once.and_return true
-          PoolParty::Provisioner.should_receive(:provision_slave).at_least(1)
-          # Kernel.should_receive(:system).with(". /etc/profile && cloud-provision -i 5 2>&1 > /dev/null &").and_return true
-        end
-        it "should call reconfigure on the master to pick up the new slave" do
-          @tc.stub!(:should_expand_cloud?).once.and_return true
-          PoolParty::Provisioner.should_receive(:reconfigure_master).once
+          @provisioner.should_receive(:install).at_least(1)
         end
         after(:each) do
           @tc.expand_cloud_if_necessary
@@ -248,9 +246,10 @@ describe "Remote" do
       describe "contract_cloud_if_necessary" do
         before(:each) do
           @tc.stub!(:request_termination_of_non_master_instance).and_return true
-          @tc.stub!(:can_shutdown_an_instance?).and_return true
+          @tc.stub!(:are_any_nodes_exceeding_minimum_runtime?).and_return true
           @tc.stub!(:wait).and_return true
           @tc.stub!(:valid_rules?).and_return false
+          @tc.stub!(:can_shutdown_an_instance?).and_return true
         end
         it "should receive can_shutdown_an_instance?" do
           @tc.should_receive(:can_shutdown_an_instance?).once
@@ -259,7 +258,7 @@ describe "Remote" do
           @tc.should_receive(:should_contract_cloud?).once.and_return false
         end
         it "should call request_termination_of_non_master_instance if we should_contract_cloud?" do
-          @tc.should_receive(:should_contract_cloud?).once.and_return true
+          @tc.stub!(:should_contract_cloud?).and_return true
           @tc.should_receive(:request_termination_of_non_master_instance).once.and_return true
         end
         after(:each) do
@@ -275,12 +274,6 @@ describe "Remote" do
         @obj = Object.new
         @obj.stub!(:ip).and_return "192.168.0.1"
       end
-      it "should raise an exception if it cannot find the keypair" do
-        @tc.stub!(:keypair_path).and_return nil
-        lambda {
-          @tc.rsync_storage_files_to(@tc.master)
-        }.should raise_error
-      end
       it "should call exec on the kernel" do
         @tc.stub!(:keypair).and_return "funky"
         ::File.stub!(:exists?).with("#{File.expand_path(Base.base_keypair_path)}/id_rsa-funky").and_return true

data/spec/poolparty/net/remoter_base_spec.rb

@@ -1,9 +1,12 @@
 require File.dirname(__FILE__) + '/../spec_helper'
 
-include Remote
+class TestRemoteClass
+  include CloudResourcer
+  include PoolParty::Remote
 
-class TestRemoter
-  include RemoterBase  
+  def keypair
+    "fake_keypair"
+  end
 end
 
 describe "RemoterBase" do
@@ -12,7 +15,7 @@ describe "RemoterBase" do
   end
   describe "methods" do
     before(:each) do
-      @tr = TestRemoter.new      
+      @tr = TestRemoteClass.new
     end
     %w(launch_new_instance! terminate_instance describe_instance instances_list).each do |method|
       eval <<-EOE
@@ -22,7 +25,7 @@ describe "RemoterBase" do
         it "should not raise an exception if #{method} is defined as a method" do
           lambda {
             @tr.instance_eval do
-              def #{method}                
+              def #{method}      
               end
             end
             @tr.#{method}
@@ -32,6 +35,7 @@ describe "RemoterBase" do
     end
     describe "lists" do
       before(:each) do
+        @tr = TestClass.new
         stub_list_of_instances_for(@tr)
       end
       it "should gather a list of the running instances" do
@@ -47,7 +51,7 @@ describe "RemoterBase" do
         @tr.list_of_nonterminated_instances.map {|a| a.name }.should == ["master", "node1", "node3"]
       end
       it "should return a list of remote instances" do
-        @tr.remote_instances_list.first.class.should == RemoteInstance
+        @tr.remote_instances_list.first.class.should == PoolParty::Remote::RemoteInstance
       end
       describe "by keypairs" do
         it "should be able to grab all the alist keypairs" do
@@ -74,6 +78,7 @@ describe "RemoterBase" do
       before(:each) do
         @master = Object.new
         @master.stub!(:ip).and_return "192.68.0.1"
+        @tr = TestClass.new
         @tr.stub!(:master).and_return @master
       end
       it "should have the method custom_install_tasks" do;@tr.respond_to?(:custom_install_tasks_for).should == true;end

data/spec/poolparty/net/remoter_spec.rb

@@ -1,21 +1,5 @@
 require File.dirname(__FILE__) + '/../spec_helper'
 
-include PoolParty::Remote
-
-def valid_rules?(*args)
-  false
-end
-
-class TestClass
-  include CloudResourcer
-  include Remote
-  using :ec2
-  attr_accessor :parent
-    
-  def keypair
-    "fake_keypair"
-  end
-end
 describe "Remoter" do
   before(:each) do
     setup
@@ -27,7 +11,7 @@ describe "Remoter" do
   end
   describe "ssh_string" do
     it "should have the ssh command" do
-      @tc.ssh_string.should =~ /ssh -o StrictHostKeyChecking=no -l '#{Base.user}' -i/
+      @tc.ssh_string.should =~ /ssh -o StrictHostKeyChecking=no -l/
     end
     it "should have the keypair in the ssh_string" do
       @tc.ssh_string.should =~ /#{@tc.keypair}/
@@ -37,9 +21,6 @@ describe "Remoter" do
     it "should have StrictHostKeyChecking set to no" do
       @tc.ssh_array.include?("-o StrictHostKeyChecking=no").should == true
     end
-    it "should have the user set to the base user class" do
-      @tc.ssh_array.include?("-l '#{Base.user}'").should == true
-    end
     it "should have the keypair path in the ssh_array" do
       @tc.ssh_array.include?('-i "'+@tc.full_keypair_path+'"').should == true
     end
@@ -50,7 +31,7 @@ describe "Remoter" do
       @ri.stub!(:ip).and_return "192.168.0.22"
     end
     it "should have rsync in the rsync_command" do
-      @tc.rsync_command.should == "rsync -azP --exclude cache -e '#{@tc.ssh_string}'"
+      @tc.rsync_command.should == "rsync -azP --exclude cache -e '#{@tc.ssh_string} -l #{Base.user}'"
     end
     it "should be able to rsync storage commands" do
       @tc.rsync_storage_files_to_command(@ri).should == "#{@tc.rsync_command} #{Base.storage_directory}/ 192.168.0.22:/var/poolparty"
@@ -60,15 +41,22 @@ describe "Remoter" do
     before(:each) do
       @tc.stub!(:wait).and_return true
       stub_list_from_remote_for(@tc)
+      stub_remoting_methods_for(@tc)
       @tc.stub!(:maximum_instances).and_return 5
       @tc.stub!(:list_of_pending_instances).and_return []
+      @tc.stub!(:list_of_nonterminated_instances).and_return []
       @tc.stub!(:list_of_running_instances).and_return []
       @tc.stub!(:master).and_return ris.first
       @tc.stub!(:after_launched).and_return true
       @tc.stub!(:verbose).and_return false
-      Provisioner.stub!(:provision_master).and_return true
-      Provisioner.stub!(:reconfigure_master).and_return true
-      Provisioner.stub!(:clear_master_ssl_certs).and_return true
+      ::File.stub!(:exists?).and_return true
+      
+      @pb = PoolParty::Provisioner::Capistrano.new(nil, @tc)
+      PoolParty::Provisioner::Capistrano.stub!(:new).and_return @pb
+      @pb.stub!(:setup_runner)
+      @pb.stub!(:install).and_return true
+      @pb.stub!(:configure).and_return true
+      @pb.stub!(:create_roles).and_return true
     end
     it "should have the method launch_master!" do
       @tc.respond_to?(:launch_and_configure_master!).should == true
@@ -84,12 +72,52 @@ describe "Remoter" do
       @tc.stub!(:can_start_a_new_instance?).and_return true
       @tc.stub!(:is_master_running?).and_return false
     end
-    it "should tell the provisioner to provision_master" do
-      Provisioner.should_receive(:provision_master).once.and_return true
-    end
     after(:each) do
       @tc.launch_and_configure_master!
     end
   end
+  describe "expansions and contractions" do
+    before(:each) do
+      @tc = TestClass.new
+      stub_list_from_remote_for @tc # sets the list of instances to 2
+    end
+    describe "list_of_nodes_exceeding_minimum_runtime" do
+      before(:each) do
+        @tc.stub!(:minimum_runtime).and_return 3000
+      end
+      it "should not be empty" do
+        @tc.list_of_running_instances.size.should == 2
+        @tc.list_of_running_instances.first.elapsed_runtime.should be > 3000
+        @tc.list_of_nodes_exceeding_minimum_runtime.size.should be > 0
+      end
+      it "should return a RemoteInstance" do
+        @tc.list_of_nodes_exceeding_minimum_runtime.first.should be_instance_of(PoolParty::Remote::RemoteInstance)
+      end
+      it "are_any_nodes_exceeding_minimum_runtime? should be true" do
+        @tc.are_any_nodes_exceeding_minimum_runtime?.should == true
+      end
+    end
+    
+    describe "are_too_few_instances_running?" do
+      it "should be false if the number of running instances is larger than the minimum instances" do
+        @tc.stub!(:minimum_instances).and_return 1
+        @tc.are_too_few_instances_running?.should == false
+      end
+      it "should be true if the number of running instances is smaller than the minimum instances" do
+        @tc.stub!(:minimum_instances).and_return 5
+        @tc.are_too_few_instances_running?.should == true
+      end
+    end
+    describe "are_too_many_instances_running?" do
+      it "should be true if the number of running instances is larger than the maximum instances" do
+        @tc.stub!(:maximum_instances).and_return 1
+        @tc.are_too_many_instances_running?.should == true
+      end
+      it "should be false if the number of running instances is smaller than the maximum instances" do
+        @tc.stub!(:maximum_instances).and_return 5
+        @tc.are_too_many_instances_running?.should == false
+      end
+    end
+  end
 
 end

data/spec/poolparty/plugins/deploydirectory_spec.rb

@@ -8,6 +8,7 @@ end
 describe "Remote Instance" do
   before(:each) do
     reset_resources!
+    Kernel.stub!(:system).and_return true
   end
   describe "wrapped" do
     before(:each) do

data/spec/poolparty/plugins/git_spec.rb

@@ -0,0 +1,45 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+include PoolParty::Resources
+
+class TestGitClass
+  include PoolParty::Resources
+  
+  def options(h={})
+    {}
+  end
+end
+describe "Remote Instance" do
+  before(:each) do
+    reset_resources!
+  end
+  describe "wrapped" do
+    before(:each) do
+      @tc = TestGitClass.new
+    end
+    it "should be a string" do
+      @tc.has_git(:at => "/var/www/", :name => "gitrepos.git", :source => "git://source.git").to_string.should =~ /exec/
+    end
+    it "should included the flushed out options" do
+      @tc.has_git({:name => "git.git", :source => "git://source.git", :requires_user => "finger", :at => "/var/www/"}).to_string.should =~ /finger@git:/
+    end
+    it "should not include the user if none is given" do
+      @tc.has_git({:name => "git.git", :source => "git://source.git",:at => "/var/www/"}).to_string.should =~ /git clone git:/
+    end
+    describe "in resource" do
+      before(:each) do
+        @tc.instance_eval do
+          has_git(:name => "gittr") do
+            source "git://source.git"
+            path "/var/www/xnot.org"
+            symlink "/var/www/xnot.org/public"
+            at "/var/www"
+          end
+        end
+      end
+      it "should have the path set within the resource" do
+        @tc.resource(:git).first.to_string.should =~ /exec \{ \"git-gittr/
+      end
+    end
+  end
+end

data/spec/poolparty/{pool → poolparty}/cloud_spec.rb

@@ -302,6 +302,7 @@ describe "Cloud" do
           describe "prepare_for_configuration" do
             before(:each) do
               @cloud.stub!(:copy_ssh_key).and_return true
+              @cloud.stub!(:before_configuration_tasks).and_return []
             end
             it "should make_base_directory" do
               @cloud.should_receive(:make_base_directory).at_least(1)
@@ -312,6 +313,12 @@ describe "Cloud" do
             it "should copy_custom_monitors" do
               @cloud.should_receive(:copy_custom_monitors).once
             end
+            it "should call before_configuration_tasks callback" do
+              @cloud.should_receive(:before_configuration_tasks).once
+            end
+            it "should call call write_unique_cookie" do
+              @cloud.should_receive(:write_unique_cookie).once
+            end
             describe "copy_custom_monitors" do
               before(:each) do                
                 Base.stub!(:custom_monitor_directories).and_return ["/tmp/monitors/custom_monitor.rb"]
@@ -360,6 +367,40 @@ describe "Cloud" do
             end
           end
         end
+        describe "minimum_runnable_options" do
+          it "should be an array on the cloud" do
+            @cloud.minimum_runnable_options.class.should == Array
+          end
+          ["keypair","minimum_instances","maximum_instances",
+            "expand_when","contract_when","set_master_ip_to"].each do |k|
+            eval <<-EOE
+              it "should have #{k} in the minimum_runnable_options" do
+                @cloud.minimum_runnable_options.include?(:#{k}).should == true
+              end
+            EOE
+          end
+          it "should include the custom_minimum_runnable_options" do
+            @cloud.stub!(:custom_minimum_runnable_options).and_return [:blank]
+            @cloud.minimum_runnable_options.include?(:blank).should == true
+          end
+        end
+        describe "unique_cookie" do
+          it "should have the method generate generate_unique_cookie_string" do
+            @cloud.respond_to?(:generate_unique_cookie_string).should == true
+          end
+          it "should call hexdigest to digest/sha" do
+            Digest::SHA256.should_receive(:hexdigest).with("#{@cloud.full_keypair_name}#{@cloud.name}")
+            @cloud.generate_unique_cookie_string
+          end
+          it "should generate the same cookie string every time" do
+            older = @cloud.generate_unique_cookie_string
+            old = @cloud.generate_unique_cookie_string
+            new_one = @cloud.generate_unique_cookie_string
+            older.should == old
+            old.should == new_one
+            new_one.should == older
+          end
+        end
       end
 
       describe "instances" do

data/spec/poolparty/{pool → poolparty}/plugin_spec.rb

@@ -100,7 +100,7 @@ describe "Plugin" do
           Kernel::PoolPartyApacheClass.stub!(:new).and_return @plugin          
         end
         it "should call enable on calling realize! when there is no block included" do
-          @plugin.should_receive(:enable).and_return true
+          @plugin.should_receive(:enable).at_least(1).and_return true
           @cloud = cloud :app do
             apache
           end

data/spec/poolparty/{pool → poolparty}/pool_spec.rb

@@ -25,7 +25,7 @@ describe "Pool" do
       @pool.respond_to?(:cloud).should == true
     end
     it "should evaluate the block when creating a new pool" do
-      Proc.should_receive(:new).once
+      Proc.should_receive(:new).at_least(1)
       Pool.new(:test) do
         Proc.new {puts "hi"}
       end

data/spec/poolparty/{pool → poolparty}/resource_spec.rb

@@ -240,6 +240,11 @@ describe "Resource" do
         it "should say there is 1 resource because the lower-level resources should be contained on the parenting resource" do
           @cloud1.resources.size.should == 1
         end
+        it "should set the exec to notify apache2" do
+          has_service("apache2")
+          has_exec(:name => "reload-page", :notify => get_service("apache2"))
+          resource(:exec).get_named("reload-page").first.options[:notify].to_s.should == "Service['apache2']"
+        end
         it "should say there is one resource on the outer resource" do
           @dir.resources.size.should == 1
         end
@@ -250,6 +255,14 @@ describe "Resource" do
           @file.parent.to_s.should == @dir.to_s
         end
       end
+      describe "handle_option_value" do
+        it "should turn a string into a hash with the string as the key name" do
+          handle_option_values("vat of cats").should == {:name => "vat of cats"}
+        end
+        it "should not change a hash into anything" do
+          handle_option_values({:name => "poe"}).should == {:name => "poe"}
+        end
+      end
       describe "fetching" do
         before(:each) do
           @file = file(:name => "pancakes")
@@ -283,10 +296,10 @@ describe "Resource" do
         end
         describe "virtual_resources" do
           before(:each) do
-            @virtual_resource = git(:name => "tank", :source => "git://github.com/auser/testgit.git")
+            @virtual_resource = git(:at => "/var/paint/cans", :name => "tank", :source => "git://github.com/auser/testgit.git")
           end
           it "should fetch a virtual resource the same" do
-            git(:name => "tank", :source => "git://github.com/auser/testgit.git").to_s.should == @virtual_resource.to_s
+            git(:at => "/var/paint/cans", :name => "tank", :source => "git://github.com/auser/testgit.git").to_s.should == @virtual_resource.to_s
           end
         end
       end

data/spec/poolparty/poolparty/resources/user_spec.rb

@@ -0,0 +1,38 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+include PoolParty::Resources
+
+describe "User" do
+  describe "instances" do
+    before(:each) do
+      @user = remote_user({:name => "fred"})
+    end
+    it "should turn the one hash instance into a string" do
+      @user.to_string.should =~ /"fred":/
+    end
+    it "should turn the two hash instance into a string" do
+      @user = remote_user do
+        name "bob"
+        home "/home/bob"
+      end
+      @user.to_string.should =~ /"bob":/
+      @user.to_string.should =~ /home => '\/home\/bob'/
+    end
+    describe "as included" do            
+      before(:each) do
+        @user = remote_user({:rent => "low"}) do
+          name "/www/conf/httpd.conf"
+        end
+      end
+      it "should use default values" do
+        @user.name.should == "/www/conf/httpd.conf"
+      end
+      it "should keep the default values for the user" do
+        @user.shell.should == "/bin/sh"
+      end
+      it "should also set options through a hash" do
+        @user.rent.should == "low"
+      end
+    end
+  end
+end

data/spec/poolparty/poolparty_spec.rb

@@ -28,6 +28,6 @@ describe "PoolParty" do
     end
   end
   it "should have a logger" do
-    PoolParty.logger.should_not be_nil
+    PoolParty.log.should_not be_nil
   end
 end

data/spec/poolparty/provisioners/capistrano/capistrano_spec.rb

@@ -0,0 +1,27 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe "Capistrano provisioner" do
+  before(:each) do
+    @cloud = cloud :app do;end
+    @remote_instance = PoolParty::Remote::RemoteInstance.new({:ip => "192.168.0.1", :status => "running", :name => "master"}, @cloud)
+    stub_list_from_remote_for(@cloud)
+  end
+  describe "instance" do
+    before(:each) do
+      @pb = PoolParty::Provisioner::Capistrano.new(@remote_instance, @cloud)
+    end
+    it "should create the config on the initialize" do    
+      @pb.config.class.should == ::Capistrano::Configuration
+    end
+    describe "config" do
+      it "should create the config at ::Capistrano::Logger::INFO if the cloud is verbose" do
+        @cloud.stub!(:verbose).and_return true
+        PoolParty::Provisioner::Capistrano.new(nil, @cloud).config.logger.level.should == ::Capistrano::Logger::INFO
+      end
+      it "should create the config at ::Capistrano::Logger::IMPORTANT if the cloud is not verbose" do
+        @cloud.stub!(:verbose).and_return false
+        PoolParty::Provisioner::Capistrano.new(nil, @cloud).config.logger.level.should == ::Capistrano::Logger::IMPORTANT
+      end      
+    end
+  end
+end