chef-provisioning-fog 0.14.0 → 0.15.0

data/spec/unit/chef/provisioning/fog_driver/driver_spec.rb

@@ -0,0 +1,71 @@
+require 'chef/provisioning/fog_driver/driver'
+
+describe Chef::Provisioning::FogDriver::Driver do
+  let(:driver) { Chef::Provisioning::FogDriver::Driver.new("fog:OpenStack", {}) }
+
+  before(:each) do
+    Chef::Provisioning::FogDriver::Driver.send(:public, *Chef::Provisioning::FogDriver::Driver.protected_instance_methods)
+  end
+
+  describe "#determine_remote_host" do
+    let(:machine_spec) { double("machine_spec", :reference => reference, :name => 'name') }
+    let(:server) { double("server", :private_ip_address => 'private', :public_ip_address => 'public', :ip_addresses => ['first_ip_address'])}
+
+    context "when 'use_private_ip_for_ssh' is specified in the machine_spec.reference" do
+      let(:reference) { { 'use_private_ip_for_ssh' => true } }
+      it "returns the private ip" do
+        expect(driver.determine_remote_host(machine_spec, server)).to eq('private')
+        expect(reference).to eq( {'transport_address_location' => :private_ip} )
+      end
+    end
+
+    context "when 'transport_address_location' is set to :private_ip" do
+      let(:reference) { { 'transport_address_location' => :private_ip } }
+      it "returns the private ip" do
+        expect(driver.determine_remote_host(machine_spec, server)).to eq('private')
+      end
+    end
+
+    context "when 'transport_address_location' is set to :ip_addresses" do
+      let(:reference) { { 'transport_address_location' => :ip_addresses } }
+      it "returns the first ip_address from array" do
+        expect(driver.determine_remote_host(machine_spec, server)).to eq('first_ip_address')
+      end
+    end
+
+    context "when 'transport_address_location' is set to :public_ip" do
+      let(:reference) { { 'transport_address_location' => :public_ip } }
+      it "returns the public ip" do
+        expect(driver.determine_remote_host(machine_spec, server)).to eq('public')
+      end
+    end
+
+    context "when machine_spec.reference does not specify the transport type" do
+      let(:reference) { Hash.new }
+
+      context "when the machine does not have a public_ip_address" do
+        let(:server) { double("server", :private_ip_address => 'private', :public_ip_address => nil, :ip_addresses => ['first_ip_address'])}
+
+        it "returns the private ip" do
+          expect(driver.determine_remote_host(machine_spec, server)).to eq('private')
+        end
+      end
+
+      context "when the machine has a public_ip_address" do
+        let(:server) { double("server", :private_ip_address => 'private', :public_ip_address => 'public', :ip_addresses => ['first_ip_address'])}
+
+        it "returns the public ip" do
+          expect(driver.determine_remote_host(machine_spec, server)).to eq('public')
+        end
+      end
+
+      context "when the machine does not have a public_ip_address or private_ip_address" do
+        let(:server) { double("server", :private_ip_address => nil, :public_ip_address => nil, :ip_addresses => ['first_ip_address'], :id => 'id')}
+
+        it "raises an error" do
+          expect {driver.determine_remote_host(machine_spec, server)}.to raise_error("Server #{server.id} has no private or public IP address!")
+        end
+      end
+    end
+  end
+end

data/spec/unit/fog_driver_spec.rb

@@ -1,32 +1,32 @@
-require 'spec_helper'
-require 'chef/provisioning/fog_driver'
-
-describe Chef::Provisioning::FogDriver do
-
-  describe ".from_url" do
-    subject { Chef::Provisioning::FogDriver.from_provider('TestDriver', {}) }
-
-    it "should return the correct class" do
-      expect(subject).to be_an_instance_of Chef::Provisioning::FogDriver::Providers::TestDriver
-    end
-
-    it "should call the target compute_options_for" do
-      expect(Chef::Provisioning::FogDriver::Providers::TestDriver).to receive(:compute_options_for)
-        .with('TestDriver', anything, {}).and_return([{}, 'test']).twice
-      subject
-    end
-
-  end
-
-  describe "when creating a new class" do
-    it "should return the correct class" do
-      test = Chef::Provisioning::FogDriver.new('fog:TestDriver:foo', {})
-      expect(test).to be_an_instance_of Chef::Provisioning::FogDriver::Providers::TestDriver
-    end
-
-    it "should populate config" do
-      test = Chef::Provisioning::FogDriver.new('fog:TestDriver:foo', {test: "chef_provisioning"})
-      expect(test.config[:test]).to eq "chef_provisioning"
-    end
-  end
-end
+require 'spec_helper'
+require 'chef/provisioning/fog_driver'
+
+describe Chef::Provisioning::FogDriver do
+
+  describe ".from_url" do
+    subject { Chef::Provisioning::FogDriver.from_provider('TestDriver', {}) }
+
+    it "should return the correct class" do
+      expect(subject).to be_an_instance_of Chef::Provisioning::FogDriver::Providers::TestDriver
+    end
+
+    it "should call the target compute_options_for" do
+      expect(Chef::Provisioning::FogDriver::Providers::TestDriver).to receive(:compute_options_for)
+        .with('TestDriver', anything, {}).and_return([{}, 'test']).twice
+      subject
+    end
+
+  end
+
+  describe "when creating a new class" do
+    it "should return the correct class" do
+      test = Chef::Provisioning::FogDriver.new('fog:TestDriver:foo', {})
+      expect(test).to be_an_instance_of Chef::Provisioning::FogDriver::Providers::TestDriver
+    end
+
+    it "should populate config" do
+      test = Chef::Provisioning::FogDriver.new('fog:TestDriver:foo', {test: "chef_provisioning"})
+      expect(test.config[:test]).to eq "chef_provisioning"
+    end
+  end
+end

data/spec/unit/providers/aws/credentials_spec.rb

@@ -1,45 +1,45 @@
-require 'chef/provisioning/fog_driver/providers/aws/credentials'
-
-describe Chef::Provisioning::FogDriver::Providers::AWS::Credentials do
-  describe "#load_ini" do
-    let(:aws_credentials_ini_file) { File.join(File.expand_path('../../../../support', __FILE__), 'aws/ini-file.ini') }
-
-    before do
-      described_class.load_ini(aws_credentials_ini_file)
-    end
-
-    it "should load a default profile" do
-      expect(described_class['default']).to include(:aws_access_key_id)
-    end
-
-    it "should load the correct values" do
-      expect(described_class['default'][:aws_access_key_id]).to eq "12345"
-      expect(described_class['default'][:aws_secret_access_key]).to eq "abcde"
-      expect(described_class['default'][:region]).to eq "us-east-1"
-      expect(described_class['default'][:aws_session_token]).to eq "mysecret"
-    end
-
-    it "should load several profiles" do
-      expect(described_class.keys.length).to eq 2
-    end
-  end
-
-  describe "#load_csv" do
-    let(:aws_credentials_csv_file) { File.join(File.expand_path('../../../../support', __FILE__), 'aws/config-file.csv') }
-    before do
-      described_class.load_csv(aws_credentials_csv_file)
-    end
-
-    it "should load a single profile" do
-      expect(described_class['default']).to include(:aws_access_key_id)
-    end
-
-    it "should load the correct values" do
-      expect(described_class['default'][:aws_access_key_id]).to eq "12345"
-    end
-
-    it "should load several profiles" do
-      expect(described_class.keys.length).to eq 2
-    end
-  end
-end
+require 'chef/provisioning/fog_driver/providers/aws/credentials'
+
+describe Chef::Provisioning::FogDriver::Providers::AWS::Credentials do
+  describe "#load_ini" do
+    let(:aws_credentials_ini_file) { File.join(File.expand_path('../../../../support', __FILE__), 'aws/ini-file.ini') }
+
+    before do
+      described_class.load_ini(aws_credentials_ini_file)
+    end
+
+    it "should load a default profile" do
+      expect(described_class['default']).to include(:aws_access_key_id)
+    end
+
+    it "should load the correct values" do
+      expect(described_class['default'][:aws_access_key_id]).to eq "12345"
+      expect(described_class['default'][:aws_secret_access_key]).to eq "abcde"
+      expect(described_class['default'][:region]).to eq "us-east-1"
+      expect(described_class['default'][:aws_session_token]).to eq "mysecret"
+    end
+
+    it "should load several profiles" do
+      expect(described_class.keys.length).to eq 2
+    end
+  end
+
+  describe "#load_csv" do
+    let(:aws_credentials_csv_file) { File.join(File.expand_path('../../../../support', __FILE__), 'aws/config-file.csv') }
+    before do
+      described_class.load_csv(aws_credentials_csv_file)
+    end
+
+    it "should load a single profile" do
+      expect(described_class['default']).to include(:aws_access_key_id)
+    end
+
+    it "should load the correct values" do
+      expect(described_class['default'][:aws_access_key_id]).to eq "12345"
+    end
+
+    it "should load several profiles" do
+      expect(described_class.keys.length).to eq 2
+    end
+  end
+end

data/spec/unit/providers/rackspace_spec.rb

@@ -1,16 +1,16 @@
-require 'chef/provisioning/fog_driver'
-require 'chef/provisioning/fog_driver/providers/rackspace'
-
-describe Chef::Provisioning::FogDriver::Providers::Rackspace do
-  subject { Chef::Provisioning::FogDriver.from_provider('Rackspace',{}) }
-
-  it "returns the correct driver" do
-    expect(subject).to be_an_instance_of Chef::Provisioning::FogDriver::Providers::Rackspace
-  end
-
-  it "has a Fog backend" do
-    pending unless Fog.mock?
-    expect(subject.compute).to be_an_instance_of Fog::Compute::RackspaceV2::Mock
-  end
-
-end
+require 'chef/provisioning/fog_driver'
+require 'chef/provisioning/fog_driver/providers/rackspace'
+
+describe Chef::Provisioning::FogDriver::Providers::Rackspace do
+  subject { Chef::Provisioning::FogDriver.from_provider('Rackspace',{}) }
+
+  it "returns the correct driver" do
+    expect(subject).to be_an_instance_of Chef::Provisioning::FogDriver::Providers::Rackspace
+  end
+
+  it "has a Fog backend" do
+    pending unless Fog.mock?
+    expect(subject.compute).to be_an_instance_of Fog::Compute::RackspaceV2::Mock
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: chef-provisioning-fog
 version: !ruby/object:Gem::Version
-  version: 0.14.0
+  version: 0.15.0
 platform: ruby
 authors:
 - John Keiser
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-08-12 00:00:00.000000000 Z
+date: 2015-09-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: chef-provisioning
@@ -111,6 +111,7 @@ files:
 - lib/chef/provisioning/fog_driver/providers/rackspace.rb
 - lib/chef/provisioning/fog_driver/providers/softlayer.rb
 - lib/chef/provisioning/fog_driver/providers/vcair.rb
+- lib/chef/provisioning/fog_driver/providers/xenserver.rb
 - lib/chef/provisioning/fog_driver/recipe_dsl.rb
 - lib/chef/provisioning/fog_driver/version.rb
 - lib/chef/resource/fog_key_pair.rb
@@ -118,6 +119,7 @@ files:
 - spec/support/aws/config-file.csv
 - spec/support/aws/ini-file.ini
 - spec/support/chef_metal_fog/providers/testdriver.rb
+- spec/unit/chef/provisioning/fog_driver/driver_spec.rb
 - spec/unit/fog_driver_spec.rb
 - spec/unit/providers/aws/credentials_spec.rb
 - spec/unit/providers/rackspace_spec.rb
@@ -140,7 +142,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.7
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
 summary: Driver for creating Fog instances in Chef Provisioning.