fairchild-poolparty 1.1.5 → 1.2.12

data/spec/poolparty/extra/deployments_spec.rb

@@ -1,68 +1,68 @@
-#TODO: Reimplement
-require File.dirname(__FILE__) + '/../spec_helper'
-
-describe "deployments" do
-  describe "class" do
-    before(:each) do
-      @klass = PoolParty::Extra::Deployments
-      @file = "hanky_danky_ranky.pool"
-      ::File.stub!(:file?).with(@file).and_return false
-    end
-    it "should have the singleton method include_deployment" do
-      @klass.respond_to?(:include_deployment).should == true
-    end
-    it "should return nil if the file doesn't exist" do
-      @klass.include_deployment(@file).should == nil
-    end
-    describe "existing file" do
-      before(:each) do
-        ::File.stub!(:file?).and_return true
-        @contents = "'hello goober'"
-        @klass.stub!(:open).with(@file).and_return @file
-        @file.stub!(:read).and_return @contents
-        @klass.include_deployment(@file).should_not == nil
-      end
-      it "should not be nil if the file does exist" do        
-        @klass.include_deployment(@file).should_not == nil
-      end
-      it "should create a class called PoolPartyHankyDankyRankyClass" do
-        @klass.include_deployment(@file).nil?.should == false
-        Object.const_defined?(:HankyDankyRankyClass).should == true
-      end
-      it "should create a method on the class called enable" do
-        HankyDankyRankyClass.new.respond_to?(:enable).should == true
-      end
-      it "should add the contents of the file to be eval'd on the method" do
-        # puts HankyDankyRankyClass.new.enable
-        # HankyDankyRankyClass.new.enable.should == "hello goober"
-        pending
-      end
-    end
-    describe "include_deployments" do
-      before(:each) do
-        ::File.stub!(:directory?).and_return false
-        @dir = "/deployments"
-      end
-      it "should have the method include_deployments" do
-        @klass.respond_to?(:include_deployments).should == true
-      end
-      it "should return nil if the directory does not exist" do
-        @klass.include_deployments(@dir).should == nil
-      end
-      describe "existing directory" do
-        before(:each) do          
-          ::File.stub!(:directory?).and_return true
-          @contents = []
-          Dir.stub!(:[]).and_return @contents
-        end
-        it "should not be nil if the directory exists" do
-          @klass.include_deployments(@dir).nil?.should == false
-        end
-        it "should call Dir.[] on the directory" do
-          Dir.should_receive(:[]).with("#{@dir}/*").and_return []
-          @klass.include_deployments(@dir)
-        end
-      end
-    end
-  end
-end
+# #TODO: Reimplement
+# require File.dirname(__FILE__) + '/../spec_helper'
+# 
+# describe "deployments" do
+#   describe "class" do
+#     before(:each) do
+#       @klass = PoolParty::Extra::Deployments
+#       @file = "hanky_danky_ranky.pool"
+#       ::File.stub!(:file?).with(@file).and_return false
+#     end
+#     it "should have the singleton method include_deployment" do
+#       @klass.respond_to?(:include_deployment).should == true
+#     end
+#     it "should return nil if the file doesn't exist" do
+#       @klass.include_deployment(@file).should == nil
+#     end
+#     describe "existing file" do
+#       before(:each) do
+#         ::File.stub!(:file?).and_return true
+#         @contents = "'hello goober'"
+#         @klass.stub!(:open).with(@file).and_return @file
+#         @file.stub!(:read).and_return @contents
+#         @klass.include_deployment(@file).should_not == nil
+#       end
+#       it "should not be nil if the file does exist" do        
+#         @klass.include_deployment(@file).should_not == nil
+#       end
+#       it "should create a class called PoolPartyHankyDankyRankyClass" do
+#         @klass.include_deployment(@file).nil?.should == false
+#         Object.const_defined?(Plugin::HankyDankyRanky).should == true
+#       end
+#       it "should create a method on the class called enable" do
+#         HankyDankyRanky.new.respond_to?(:enable).should == true
+#       end
+#       it "should add the contents of the file to be eval'd on the method" do
+#         # puts HankyDankyRankyClass.new.enable
+#         # HankyDankyRankyClass.new.enable.should == "hello goober"
+#         pending
+#       end
+#     end
+#     describe "include_deployments" do
+#       before(:each) do
+#         ::File.stub!(:directory?).and_return false
+#         @dir = "/deployments"
+#       end
+#       it "should have the method include_deployments" do
+#         @klass.respond_to?(:include_deployments).should == true
+#       end
+#       it "should return nil if the directory does not exist" do
+#         @klass.include_deployments(@dir).should == nil
+#       end
+#       describe "existing directory" do
+#         before(:each) do          
+#           ::File.stub!(:directory?).and_return true
+#           @contents = []
+#           Dir.stub!(:[]).and_return @contents
+#         end
+#         it "should not be nil if the directory exists" do
+#           @klass.include_deployments(@dir).nil?.should == false
+#         end
+#         it "should call Dir.[] on the directory" do
+#           Dir.should_receive(:[]).with("#{@dir}/*").and_return []
+#           @klass.include_deployments(@dir)
+#         end
+#       end
+#     end
+#   end
+# end

data/spec/poolparty/fixtures/clouds.json

@@ -34,7 +34,7 @@
         "default_properties_hash_filename": "cloud.json",
         "poolparty_home_path": "/Users/mfairchild/.poolparty",
         "keypairs": ["/Users/mfairchild/.ec2oncourse/r_and_d.pem", "/Users/mfairchild/.ssh/id_rsa"],
-        "availabilty_zone": "us-east-1a",
+        "availability_zone": "us-east-1a",
         "fileserver_base": "puppet://master/files",
         "debugging": false,
         "minimum_instances": 1,

data/spec/poolparty/fixtures/fake_key.pub

@@ -0,0 +1 @@
+ssh-rsa totally_aPublicKey auser@my.laptop.local

data/spec/poolparty/helpers/optioner_spec.rb

@@ -7,11 +7,11 @@ describe "Option Parser" do
       @op = PoolParty::Optioner.new([], {:abstract => true})
       @op.parse_options
     end
-    it "should set the options as an Hash" do
-      @op.options.class.should == Hash
+    it "should set the dsl_options as an Hash" do
+      @op.dsl_options.class.should == Hash
     end
     it "should have the verbose option set to false by default" do
-      @op.verbose.should == false
+      @op.verbose?.should == false
     end
     it "should call a method called on it that is not defined on the options if they exist" do
       @op.dsl_options.should_receive(:[]).with(:verbose).at_least(1).and_return true
@@ -40,11 +40,4 @@ describe "Option Parser" do
       @op.flagged_args.should == ["-v", "-i", "1", "-x"]
     end
   end
-  
-  it "should be able to take a block and set some options on the block" do
-    PoolParty::Optioner.new(["-w"], {:abstract => false, :load_pools => false}) do |opts, optioner|
-      opts.on('-w', '--wee')    { optioner.wee "wee" }
-      opts.on('-t t', '--teatime tea')    { optioner.tea "time" }
-    end.wee.should == "wee"
-  end
-end
+end

data/spec/poolparty/modules/cloud_resourcer_spec.rb

@@ -15,7 +15,7 @@ class TestParentClass < PoolParty::Cloud::Cloud
 end
 describe "CloudResourcer" do
   before(:each) do
-    @tc = ResourcerTestClass.new :bank do
+    @tc = Cloud.new :bank do
     end
   end
   it "should have the method instances" do

data/spec/poolparty/modules/searchable_paths_spec.rb

@@ -0,0 +1,83 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+PATH_TEST_ROOT = Default.storage_directory / "path_test"
+PATH_ONE       = PATH_TEST_ROOT / "one"
+PATH_TWO       = PATH_TEST_ROOT / "two"
+
+class TestFile
+  include SearchablePaths
+  has_searchable_paths(:dir => "templates", :paths => [PATH_ONE, PATH_TWO])
+end
+
+class TestFile2
+  include SearchablePaths
+  has_searchable_paths(:dirs => ["clouds", "/"], :prepend_paths => [PATH_ONE, PATH_TWO])
+end
+
+def File.write_to_file(name, content="")
+  open(name, "w") {|f| f.print content}
+end
+
+describe "FileWriter" do
+  before(:each) do
+    # create path one
+    [PATH_ONE/:clouds, PATH_ONE/:templates, PATH_ONE/:foo].each {|dir| FileUtils.mkdir_p(dir) }
+    [PATH_ONE/'clouds.rb', PATH_ONE/:clouds/'special.rb', PATH_ONE/:templates/'apache.conf'].each do |f|
+      File.write_to_file(f)
+    end
+
+    # create path two
+    [PATH_TWO/:clouds, PATH_TWO/:templates, PATH_TWO/:foo].each {|dir| FileUtils.mkdir_p(dir) }
+    [PATH_TWO/'swing.rb', PATH_TWO/'clouds.rb', PATH_TWO/:clouds/'common.rb', PATH_TWO/:templates/'mysql.conf'].each do |f|
+      File.write_to_file(f)
+    end
+    
+    @template = TestFile.new
+    @cloud    = TestFile2.new
+  end
+
+  describe "class methods" do
+    it "should have class options set" do
+      TestFile.searchable_paths_dir.should   == "templates"
+      TestFile.searchable_paths_dirs.should  == ["templates"]
+      TestFile2.searchable_paths_dirs.should == ["clouds", "/"]
+    end
+  end
+
+  describe "#find_file" do
+    it "should be able to find a template" do
+      @template.find_file("apache.conf").should_not be_nil
+      @template.find_file("apache.conf").should == PATH_ONE/:templates/'apache.conf'
+      @template.find_file("mysql.conf").should == PATH_TWO/:templates/'mysql.conf'
+    end
+
+    it "shouldn't find a template that doesn't exist" do
+      @template.find_file("post-office.conf").should be_nil
+    end
+
+    it "should find something with prepended paths" do
+      @cloud.find_file("clouds.rb").should == PATH_ONE/'clouds.rb'
+      @cloud.find_file("swing.rb").should == PATH_TWO/'swing.rb'
+    end
+
+    it "should find things in the right order" do
+      @cloud.find_file("clouds.rb").should == PATH_ONE/'clouds.rb'
+    end
+  end
+  
+  describe "#search_in_known_locations" do
+    before(:each) do
+      # create path one
+      FileUtils.mkdir_p(PATH_ONE/'extra')
+      File.write_to_file(PATH_ONE/'extra'/'clouds.rb')
+    end
+    
+    it "should look in the additional search paths first" do
+      @cloud.find_file("clouds.rb", [PATH_ONE/'extra']).should == PATH_ONE/'extra'/'clouds.rb'
+    end    
+  end
+
+  after(:each) do
+    FileUtils.rm_rf(PATH_TEST_ROOT)
+  end
+end

data/spec/poolparty/net/remote_instance_spec.rb

@@ -5,8 +5,7 @@ include PoolParty::Remote
 
 describe "Remote Instance" do
   before(:each) do
-    setup
-    @valid_hash = {:ip => "127.0.0.1", :name => "master", :responding => "true"}
+    @valid_hash = {:public_ip=>'33.44.55.66', :ip => "127.0.0.1", :name => "master", :responding => "true"}
   end
 
   it "should have tests for the abstaract RemoteInstance class" do

data/spec/poolparty/net/remoter_base_spec.rb

@@ -7,14 +7,10 @@ end
 
 describe "RemoterBase" do
   before(:each) do
-    setup
+    @tr = TestRemoteClass.new
   end
-  describe "methods" do
-    before(:each) do
-      @tr = TestRemoteClass.new
-    end
-    %w(launch_new_instance! terminate_instance describe_instance instances_list).each do |method|
-      eval <<-EOE
+  %w(launch_new_instance! terminate_instance describe_instance instances_list).each do |method|
+    eval <<-EOE
         it "should raise an exception if #{method} is not defined as a method" do
           # pending # Weird .should raise_error
           lambda { @tr.class.#{method} }.should raise_error
@@ -28,8 +24,8 @@ describe "RemoterBase" do
             @tr.#{method}
           }.should_not raise_error
         end
-      EOE
-    end
-    
+EOE
+        
   end
-end
+end
+    

data/spec/poolparty/net/remoter_bases/ec2_mocks_and_stubs.rb

@@ -3,22 +3,20 @@ class TestEC2Class < Ec2
   include CloudDsl
   include Dslify
   
-  default_options PoolParty::Default.default_options.merge({:access_key => "Not an access key", 
-    :secret_access_key => "not a secret access key", :keypair => 'id_rsa'})
+  default_options :access_key => "Not an access key", 
+                  :secret_access_key => "not a secret access key", 
+                  :keypair => 'id_rsa'
   
-  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
+  # def ami;"ami-abc123";end
+  # def size; "small";end
+  # def security_group; "default";end
+  # def ebs_volume_id; "ebs_volume_id";end
+  # def availability_zone; "us-east-1a";end
+  # def verbose; false; end
   def describe_instances(o={})
     response_list_of_instances
   end
 end
 
 class TestEc2RemoteInstance < PoolParty::Remote::Ec2RemoteInstance
-  def initialize(opts, p=nil)
-    super
-  end
 end

data/spec/poolparty/net/remoter_bases/ec2_remote_instance_spec.rb

@@ -2,30 +2,21 @@ require File.join(File.dirname(__FILE__), '../../spec_helper')
 require File.dirname(__FILE__) + '/ec2_mocks_and_stubs.rb'
 
 include PoolParty::Remote
-# include Aska
-
-# require File.dirname(__FILE__) + '/ec2_mocks_and_stubs.rb'
 
 
 describe "Ec2RemoteInstance" do
   before(:each) do
     @cloud = TestCloud.new :test_remoter_base_cloud
-    @ec2 = TestEc2RemoteInstance.new( {:name => "node3"}, TestEC2Class.new(@cloud) )
-    # node3 stubbed as  {:ip=>"192.168.0.3", :keypair=>"fake_keypair", :status=>"pending", :name=>"node3", :launching_time=>Time.now
-    @ec2.my_cloud.stub!(:describe_instances).and_return response_list_of_instances
-  end
-  it "should require options" do
-    lambda {TestEc2RemoteInstance.new()}.should raise_error
+    @tec2 = TestEC2Class.new(@cloud.dsl_options)
+    @ec2 = TestEc2RemoteInstance.new( @tec2.dsl_options.merge(:name => "node3", :status => "pending", :ip => "192.168.0.3"))
   end
   it "should have the name set in the options" do
-    @ec2.respond_to?(:name).should == false
-    @ec2.options.keys.include?(:name).should == true
+    @ec2.respond_to?(:name).should == true
+    @ec2.dsl_options.keys.include?(:name).should == true
     @ec2.name.should == 'node3'
   end
   it "should have access to parent cloud options" do
-    @ec2.my_cloud.class.should ==  TestEC2Class
-    @ec2.my_cloud.should respond_to(:describe_instances)
-    @ec2.my_cloud.size.should == 'small'    
+    @ec2.instance_type.should == 'm1.small'
   end
   it "should have instance description hash mapped to methods" do
     # puts  "<ol>"
@@ -34,43 +25,14 @@ describe "Ec2RemoteInstance" do
     @ec2.status.should == 'pending'
     @ec2.name.should  == 'node3'
     @ec2.ip.should == '192.168.0.3'
-    @ec2.launching_time.should_not be_nil
-    @ec2.valid?.should be_true
-  end
-  it "should not be the master" do
-    @ec2.master?.should_not be_true
   end
 end
 
-
-
-
-
-
 describe "Remote Instance" do
   before(:each) do
-    setup
     @valid_hash = {:ip => "127.0.0.1", :name => "master"}
   end
 
-  describe "configurable" do
-    it "should set the options sent in the options, overiding passed option with found instance values" do
-      @ec2 = TestEc2RemoteInstance.new(@valid_hash)
-      @ec2.ip.should == "127.0.0.1"
-    end
-    it "should set the options sent by the parent" do
-      @obj = Object.new
-      @obj.stub!(:options).and_return({:dude => "tte"})
-      @ec2 = @ec2 = TestEc2RemoteInstance.new(@valid_hash, @obj)
-      @ec2.dude.should == "tte"
-    end
-    it "should not overwrite the options that are already set" do
-      @obj = Object.new
-      @obj.stub!(:options).and_return({:ip => "172.176.0.1"})
-      @ec2 = TestEc2RemoteInstance.new(@valid_hash, @obj)
-      @ec2.ip.should == "127.0.0.1"
-    end
-  end
   it "should create a remote instance with a Hash" do
     @ec2 = TestEc2RemoteInstance.new(@valid_hash)
     @ec2.valid?.should == true
@@ -105,20 +67,8 @@ describe "Remote Instance" do
       @ec2.stub!(:launching_time).and_return(30.minutes.ago)
       @ec2.elapsed_runtime.should be >= 1800
     end
-    it "should be say that it is the master if the name is master" do
-      @ec2.name.should == "master"
-      @ec2.master?.should == true
-    end
     it "should say that it is responding? if responding is not nil" do
       @ec2.running?.should == true
-      @ec2.responding?.should == true
-      # @ec2.responding.should_not be_nil
-    end
-    it "should have the puppet_runner_command" do
-      @ec2.respond_to?(:puppet_runner_command).should == true
-    end
-    it "should return the puppet_runner_command with puppetd" do
-      @ec2.puppet_runner_command.should =~ /puppetrun/
     end
   end
-end
+end

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

@@ -5,7 +5,7 @@ require File.dirname(__FILE__) + '/ec2_mocks_and_stubs.rb'
 describe "ec2 remote base" do
   before(:each) do
     @cloud = TestCloud.new :test_remoter_base_cloud
-    @tr = TestEC2Class.new(@cloud)
+    @tr = TestEC2Class.new( TestCloud.default_options )
     stub_remoter_for(@tr)
     # @tr.stub!(:get_instances_description).and_return response_list_of_instances
   end
@@ -17,9 +17,12 @@ describe "ec2 remote base" do
     EOE
   end
   describe "helpers" do
-    it "should be able to convert an ec2 ip to a real ip" do
+    it "should be able to convert an american ec2 ip to a real ip" do
       "ec2-72-44-36-12.compute-1.amazonaws.com".convert_from_ec2_to_ip.should == "72.44.36.12"
     end
+    it "should be able to convert an european ec2 ip to a real ip" do
+      "ec2-72-44-36-12.eu-west-1.compute.amazonaws.com".convert_from_ec2_to_ip.should == "72.44.36.12"
+    end
     it "should not throw an error if another string is returned" do
       "72.44.36.12".convert_from_ec2_to_ip.should == "72.44.36.12"
     end
@@ -49,7 +52,7 @@ describe "ec2 remote base" do
     #   @tr.launch_new_instance!
     # end
     it "should get the hash response from EC2ResponseObject" do
-      EC2ResponseObject.should_receive(:get_hash_from_response).and_return @ret_hash
+      EC2ResponseObject.should_receive(:describe_instance).and_return @ret_hash
       @tr.launch_new_instance! :keypair => "keys"
     end
   end
@@ -87,12 +90,11 @@ describe "ec2 remote base" do
       @resp = {"requestId"=>"be9bd2e9-4f8c-448f-993d-c21fe537e322", "addressesSet"=>{"item"=>[{"instanceId"=>nil, "publicIp"=>"174.129.212.93"}, {"instanceId"=>nil, "publicIp"=>"182.199.200.201"}]}, "xmlns"=>"http://ec2.amazonaws.com/doc/2008-12-01/"}
       @tr.ec2.stub!(:describe_addresses).and_return @resp
     end
-    it "should have the next available elastic_ip" do
-      @tr.next_unused_elastic_ip.should == "174.129.212.93"
+    it "should have the next available elastic_ip as nil if there are no ips set" do
+      @tr.next_unused_elastic_ip.should == nil
     end
     it "should use only the elastic ips set on the cloud" do
-      @cloud.stub!(:elastic_ips?).and_return true
-      @cloud.stub!(:elastic_ips).and_return ["182.199.200.201"]
+      @tr.stub!(:elastic_ips).and_return ["182.199.200.201"]
       @tr.stub!(:cloud).and_return @cloud
       @tr.next_unused_elastic_ip.should == "182.199.200.201"
     end
@@ -131,4 +133,5 @@ describe "ec2 remote base" do
       @tr.create_snapshot.should == nil
     end
   end
+
 end

data/spec/poolparty/net/remoter_spec.rb

@@ -2,10 +2,9 @@ require File.dirname(__FILE__) + '/../spec_helper'
 
 describe "Remoter" do
   before(:each) do
-    setup
     @cloud = cloud :app do;end
     @tc = TestClass.new
-    @tc.parent = @cloud
+    # @tc.parent = @cloud
     @sample_instances_list = [{:ip => "192.168.0.1", :name => "master"}, {:ip => "192.168.0.2", :name => "node1"}]
   end
   describe "ssh_string" do
@@ -130,4 +129,4 @@ describe "Remoter" do
   #   end
   # end
 
-end
+end

data/spec/poolparty/plugins/authorized_key_spec.rb

@@ -0,0 +1,23 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe "Remote Instance" do
+  before(:each) do
+    @tc = TestBaseClass.new do
+    end
+  end
+  it "should have the method has_line_in_file available on the class" do
+    @tc.respond_to?(:has_line_in_file).should == true
+  end
+  describe "call" do
+    before(:each) do
+      @tc.run_in_context do
+        has_authorized_key({:for_user => 'alerner', 
+                            :public_key_file => "#{::File.dirname(__FILE__)}/../fixtures/fake_key.pub"})
+      end
+      @compiled = ChefResolver.new(@tc.to_properties_hash).compile
+    end
+    it "should have the line in the file from ChefResolver" do
+      @compiled.should match(/~alerner\/\.ssh/)
+    end
+  end
+end