ey-core 3.4.5 → 3.5.0

data/spec/ey-core/cli/recipes/download_spec.rb

@@ -1,93 +1,93 @@
-require 'spec_helper'
-require 'ey-core/cli/recipes/download'
-
-describe Ey::Core::Cli::Recipes::Download do
-  set_up_cli
-
-  before(:each) do
-    allow_any_instance_of(described_class).
-      to receive(:untar).
-      with(any_args).
-      and_return(true)
-
-    allow_any_instance_of(described_class).
-      to receive(:ungzip).
-      with(any_args).
-      and_return(true)
-  end
-
-  context 'ey-core recipes download' do
-    context 'with an existing cookbooks directory' do
-      before(:each) do
-        allow(File).to receive(:exist?).with("cookbooks").and_return(true)
-      end
-
-      it 'fails out, advising that the cookbooks already exist locally' do
-        status = execute
-
-        expect(error_output).
-          to include(
-            'Cannot download recipes, cookbooks directory already exists.'
-          )
-
-        expect(status).to eql(255)
-      end
-    end
-
-    context 'with no cookbooks directory' do
-      before(:each) do
-        allow(File).to receive(:exist?).with("cookbooks").and_return(false)
-
-        allow(environment).
-          to receive(:download_recipes).
-          and_return('cookbooks.tar.gz')
-
-        allow(cli).
-          to receive(:ungzip)
-
-        allow(cli).
-          to receive(:untar)
-      end
-
-      it 'advises that the cookbooks are being downloaded' do
-        execute
-
-        expect(standard_output).to include('Downloading recipes'.green)
-      end
-
-      it 'downloads the cookbooks archive' do
-        expect(environment).
-          to receive(:download_recipes).
-          and_call_original
-
-        execute
-      end
-
-      it 'advises that the downloaded archive is being extracted' do
-        execute
-
-        expect(standard_output).
-          to match(/Extracting recipes to 'cookbooks\/'/)
-      end
-
-      it 'unarchives the downloaded archive' do
-        allow(environment).to receive(:download_recipes).and_return('cookbooks.tar.gz')
-        expect(cli).
-          to receive(:ungzip).
-          with('cookbooks.tar.gz').
-          and_return('cookbooks.tar')
-
-        expect(cli).
-          to receive(:untar).
-          with('cookbooks.tar', './').
-          and_return(true)
-
-        execute
-      end
-
-      it 'exits cleanly' do
-        expect(execute).to eql(0)
-      end
-    end
-  end
-end
+require 'spec_helper'
+require 'ey-core/cli/recipes/download'
+
+describe Ey::Core::Cli::Recipes::Download do
+  set_up_cli
+
+  before(:each) do
+    allow_any_instance_of(described_class).
+      to receive(:untar).
+      with(any_args).
+      and_return(true)
+
+    allow_any_instance_of(described_class).
+      to receive(:ungzip).
+      with(any_args).
+      and_return(true)
+  end
+
+  context 'ey-core recipes download' do
+    context 'with an existing cookbooks directory' do
+      before(:each) do
+        allow(File).to receive(:exist?).with("cookbooks").and_return(true)
+      end
+
+      it 'fails out, advising that the cookbooks already exist locally' do
+        status = execute
+
+        expect(error_output).
+          to include(
+            'Cannot download recipes, cookbooks directory already exists.'
+          )
+
+        expect(status).to eql(255)
+      end
+    end
+
+    context 'with no cookbooks directory' do
+      before(:each) do
+        allow(File).to receive(:exist?).with("cookbooks").and_return(false)
+
+        allow(environment).
+          to receive(:download_recipes).
+          and_return('cookbooks.tar.gz')
+
+        allow(cli).
+          to receive(:ungzip)
+
+        allow(cli).
+          to receive(:untar)
+      end
+
+      it 'advises that the cookbooks are being downloaded' do
+        execute
+
+        expect(standard_output).to include('Downloading recipes'.green)
+      end
+
+      it 'downloads the cookbooks archive' do
+        expect(environment).
+          to receive(:download_recipes).
+          and_call_original
+
+        execute
+      end
+
+      it 'advises that the downloaded archive is being extracted' do
+        execute
+
+        expect(standard_output).
+          to match(/Extracting recipes to 'cookbooks\/'/)
+      end
+
+      it 'unarchives the downloaded archive' do
+        allow(environment).to receive(:download_recipes).and_return('cookbooks.tar.gz')
+        expect(cli).
+          to receive(:ungzip).
+          with('cookbooks.tar.gz').
+          and_return('cookbooks.tar')
+
+        expect(cli).
+          to receive(:untar).
+          with('cookbooks.tar', './').
+          and_return(true)
+
+        execute
+      end
+
+      it 'exits cleanly' do
+        expect(execute).to eql(0)
+      end
+    end
+  end
+end

data/spec/ey-core/cli/recipes/upload_spec.rb

@@ -1,80 +1,80 @@
-require 'spec_helper'
-require 'ey-core/cli/recipes/upload'
-
-describe Ey::Core::Cli::Recipes::Upload do
-  set_up_cli
-
-  before(:each) do
-    allow_any_instance_of(described_class).
-      to receive(:run_chef).
-      with(any_args).
-      and_return(true)
-
-    allow_any_instance_of(described_class).
-      to receive(:upload_recipes).
-      with(any_args).
-      and_return(true)
-  end
-
-  context 'ey-core recipes upload' do
-    it 'advises that it is uploading recipes for the current environment' do
-      execute
-
-      expect(standard_output).
-        to match(/Uploading custom recipes for /)
-    end
-
-    it 'uploads the recipes' do
-      expect(cli).to receive(:upload_recipes).with(environment, 'cookbooks/')
-
-      execute
-    end
-    
-    context 'upon uploading successfully' do
-      it 'advises that the upload completed' do
-        execute
-
-        expect(standard_output).
-          to match(/Uploading custom recipes complete/)
-      end
-    end
-
-    context 'upon failing to upload' do
-      before(:each) do
-        allow(cli).
-          to receive(:upload_recipes).
-          with(environment, 'cookbooks/').
-          and_raise('big bada boom')
-      end
-
-      it 'aborts, advising that the upload failed' do
-        status = execute
-
-        expect(error_output).
-          to match(/There was a problem uploading the recipes/)
-
-        expect(status).not_to eql(0)
-      end
-    end
-  end
-
-  context 'ey-core recipes upload --apply' do
-    arguments '--apply'
-
-    it 'performs a custom chef run' do
-      expect(cli).to receive(:run_chef).with('main', environment, {no_wait: nil, verbose: nil, watch: nil})
-
-      execute
-    end
-  end
-
-  context 'ey-core recipes upload --path /some/path' do
-    arguments '--file /some/path'
-
-    it 'uploads the recipes from the given path' do
-      expect(cli).to receive(:upload_recipes).with(environment, '/some/path')
-
-      execute
-    end
-  end
-end
+require 'spec_helper'
+require 'ey-core/cli/recipes/upload'
+
+describe Ey::Core::Cli::Recipes::Upload do
+  set_up_cli
+
+  before(:each) do
+    allow_any_instance_of(described_class).
+      to receive(:run_chef).
+      with(any_args).
+      and_return(true)
+
+    allow_any_instance_of(described_class).
+      to receive(:upload_recipes).
+      with(any_args).
+      and_return(true)
+  end
+
+  context 'ey-core recipes upload' do
+    it 'advises that it is uploading recipes for the current environment' do
+      execute
+
+      expect(standard_output).
+        to match(/Uploading custom recipes for /)
+    end
+
+    it 'uploads the recipes' do
+      expect(cli).to receive(:upload_recipes).with(environment, 'cookbooks/')
+
+      execute
+    end
+    
+    context 'upon uploading successfully' do
+      it 'advises that the upload completed' do
+        execute
+
+        expect(standard_output).
+          to match(/Uploading custom recipes complete/)
+      end
+    end
+
+    context 'upon failing to upload' do
+      before(:each) do
+        allow(cli).
+          to receive(:upload_recipes).
+          with(environment, 'cookbooks/').
+          and_raise('big bada boom')
+      end
+
+      it 'aborts, advising that the upload failed' do
+        status = execute
+
+        expect(error_output).
+          to match(/There was a problem uploading the recipes/)
+
+        expect(status).not_to eql(0)
+      end
+    end
+  end
+
+  context 'ey-core recipes upload --apply' do
+    arguments '--apply'
+
+    it 'performs a custom chef run' do
+      expect(cli).to receive(:run_chef).with('main', environment, {no_wait: nil, verbose: nil, watch: nil})
+
+      execute
+    end
+  end
+
+  context 'ey-core recipes upload --path /some/path' do
+    arguments '--file /some/path'
+
+    it 'uploads the recipes from the given path' do
+      expect(cli).to receive(:upload_recipes).with(environment, '/some/path')
+
+      execute
+    end
+  end
+end

data/spec/features_spec.rb

@@ -1,79 +1,79 @@
-require 'spec_helper'
-
-describe 'as a user' do
-  let!(:client) { create_client }
-
-  let!(:public_feature) do
-    if Ey::Core::Client.mocking?
-      client.create_feature(
-                     :id      => "public_feature",
-                     :privacy => "public",
-                     :name    => "A Public Feature",
-                    )
-    else
-      client.features.all.first
-    end
-  end
-
-  let!(:other_feature)  do
-    if Ey::Core::Client.mocking?
-      client.create_feature(
-                     :id      => "other_feature",
-                     :privacy => "private",
-                     :name    => "Another Feature",
-                    )
-    else
-      client.features.all.last
-    end
-  end
-
-  context "with an account" do
-    let!(:account) { create_account(client: client) }
-
-    it "should list account features" do
-      client.features.get(other_feature.identity).enable!(account)
-
-      expect(account.features.all.size).to be > 0
-
-      feature = account.features.first(id: other_feature.id)
-
-      expect(feature.id).to          eq(other_feature.id)
-      expect(feature.name).to        eq(other_feature.name)
-      expect(feature.description).to eq(other_feature.description)
-    end
-
-    it "should enable feature" do
-      expect {
-        client.features.first.enable!(account)
-      }.to change { account.features.all.size }.by(1)
-    end
-
-    it "should disable feature" do
-      client.features.first.enable!(account)
-      expect {
-        client.features.first.disable!(account)
-      }.to change { account.features.all.size }.by(-1)
-    end
-  end
-
-  context "without an account" do
-    it "should list features" do
-      expect(client.features.all).to include(public_feature)
-    end
-
-    it "should show a feature" do
-      feature = client.features.get(public_feature.identity)
-
-      expect(feature).not_to be_nil
-
-      expect(feature.id).to          eq(public_feature.identity)
-      expect(feature.name).to        eq(public_feature.name)
-      expect(feature.description).to eq(public_feature.description)
-    end
-  end
-
-  context "with multiple accounts" do
-    it "should list all enabled features"
-    it "should list an account's enabled features"
-  end
-end
+require 'spec_helper'
+
+describe 'as a user' do
+  let!(:client) { create_client }
+
+  let!(:public_feature) do
+    if Ey::Core::Client.mocking?
+      client.create_feature(
+                     :id      => "public_feature",
+                     :privacy => "public",
+                     :name    => "A Public Feature",
+                    )
+    else
+      client.features.all.first
+    end
+  end
+
+  let!(:other_feature)  do
+    if Ey::Core::Client.mocking?
+      client.create_feature(
+                     :id      => "other_feature",
+                     :privacy => "private",
+                     :name    => "Another Feature",
+                    )
+    else
+      client.features.all.last
+    end
+  end
+
+  context "with an account" do
+    let!(:account) { create_account(client: client) }
+
+    it "should list account features" do
+      client.features.get(other_feature.identity).enable!(account)
+
+      expect(account.features.all.size).to be > 0
+
+      feature = account.features.first(id: other_feature.id)
+
+      expect(feature.id).to          eq(other_feature.id)
+      expect(feature.name).to        eq(other_feature.name)
+      expect(feature.description).to eq(other_feature.description)
+    end
+
+    it "should enable feature" do
+      expect {
+        client.features.first.enable!(account)
+      }.to change { account.features.all.size }.by(1)
+    end
+
+    it "should disable feature" do
+      client.features.first.enable!(account)
+      expect {
+        client.features.first.disable!(account)
+      }.to change { account.features.all.size }.by(-1)
+    end
+  end
+
+  context "without an account" do
+    it "should list features" do
+      expect(client.features.all).to include(public_feature)
+    end
+
+    it "should show a feature" do
+      feature = client.features.get(public_feature.identity)
+
+      expect(feature).not_to be_nil
+
+      expect(feature.id).to          eq(public_feature.identity)
+      expect(feature.name).to        eq(public_feature.name)
+      expect(feature.description).to eq(public_feature.description)
+    end
+  end
+
+  context "with multiple accounts" do
+    it "should list all enabled features"
+    it "should list an account's enabled features"
+  end
+end

data/spec/firewalls_spec.rb

@@ -1,84 +1,84 @@
-require 'spec_helper'
-
-describe "firewalls" do
-  let(:client)       { create_client }
-  let!(:account)     { create_account(client: client) }
-  let!(:provider)    { create_provider(account: account) }
-  let!(:environment) { create_environment(account: account, name: Faker::Name.first_name) }
-
-  it "creates a firewall" do
-    pending "Not implemented"
-
-    name     = Faker::Name.first_name
-    location = "us-west-2"
-
-    firewall = client.firewalls.create!(
-      :provider    => provider,
-      :name        => name,
-      :location    => location,
-      :environment => environment.id,
-    ).resource!
-
-    expect(firewall.id).not_to be_nil
-    expect(firewall.provisioned_id).not_to be_nil
-    expect(firewall.provider).to eq(provider)
-    expect(firewall.name).to     eq(name)
-    expect(firewall.location).to eq(location)
-    expect(firewall.environment).to eq(environment)
-  end
-
-  context "with a firewall" do
-    before(:each) { pending "Not implemented" }
-
-    let!(:firewall) { client.firewalls.create!(provider: provider, name: Faker::Name.first_name, location: "us-west-2", environment: environment.id).resource! }
-
-    it "destroys a firewall" do
-      firewall.destroy.ready!
-
-      expect(firewall.reload.deleted_at).not_to be_nil
-    end
-
-    it "should create a rule" do
-      rule = client.firewall_rules.create!(
-        :firewall_id => firewall.id,
-      ).resource!
-
-      expect(rule.source).to eq("0.0.0.0/0")
-      expect(rule.port_range).to eq(0..65535)
-    end
-
-    it "creates a rule with a firewall as the source" do
-      other_firewall = client.firewalls.create!(provider: provider, name: Faker::Name.first_name, location: "us-west-2").resource!
-
-      rule = client.firewall_rules.create!(
-        :firewall_id => firewall.id,
-        :source      => other_firewall.id,
-        :port_range  => "80-443",
-      ).resource!
-
-      expect(rule.source.to_i).to     eq(other_firewall.id)
-      expect(rule.source_firewall).to eq(other_firewall)
-      expect(rule.port_range).to      eq(80..443)
-    end
-
-    context "with a firewall rule" do
-      let!(:rule) { client.firewall_rules.create!(firewall_id: firewall.id).resource! }
-
-      it "destroys a rule" do
-        expect { rule.destroy.ready! }.to change { client.firewall_rules.all.size }.by(-1)
-        expect(rule.reload.deleted_at).not_to be_nil
-      end
-
-      it "is part of the firewall's rules list" do
-        expect(firewall.rules.count).to eq(1)
-        expect(firewall.rules.first.id).to eq(rule.id)
-      end
-    end
-  end
-
-  context "with multiple firewalls" do
-    it "lists all firewalls"
-    it "lists firewalls by name"
-    it "lists a firewall's rules"
-  end
-end
+require 'spec_helper'
+
+describe "firewalls" do
+  let(:client)       { create_client }
+  let!(:account)     { create_account(client: client) }
+  let!(:provider)    { create_provider(account: account) }
+  let!(:environment) { create_environment(account: account, name: Faker::Name.first_name) }
+
+  it "creates a firewall" do
+    pending "Not implemented"
+
+    name     = Faker::Name.first_name
+    location = "us-west-2"
+
+    firewall = client.firewalls.create!(
+      :provider    => provider,
+      :name        => name,
+      :location    => location,
+      :environment => environment.id,
+    ).resource!
+
+    expect(firewall.id).not_to be_nil
+    expect(firewall.provisioned_id).not_to be_nil
+    expect(firewall.provider).to eq(provider)
+    expect(firewall.name).to     eq(name)
+    expect(firewall.location).to eq(location)
+    expect(firewall.environment).to eq(environment)
+  end
+
+  context "with a firewall" do
+    before(:each) { pending "Not implemented" }
+
+    let!(:firewall) { client.firewalls.create!(provider: provider, name: Faker::Name.first_name, location: "us-west-2", environment: environment.id).resource! }
+
+    it "destroys a firewall" do
+      firewall.destroy.ready!
+
+      expect(firewall.reload.deleted_at).not_to be_nil
+    end
+
+    it "should create a rule" do
+      rule = client.firewall_rules.create!(
+        :firewall_id => firewall.id,
+      ).resource!
+
+      expect(rule.source).to eq("0.0.0.0/0")
+      expect(rule.port_range).to eq(0..65535)
+    end
+
+    it "creates a rule with a firewall as the source" do
+      other_firewall = client.firewalls.create!(provider: provider, name: Faker::Name.first_name, location: "us-west-2").resource!
+
+      rule = client.firewall_rules.create!(
+        :firewall_id => firewall.id,
+        :source      => other_firewall.id,
+        :port_range  => "80-443",
+      ).resource!
+
+      expect(rule.source.to_i).to     eq(other_firewall.id)
+      expect(rule.source_firewall).to eq(other_firewall)
+      expect(rule.port_range).to      eq(80..443)
+    end
+
+    context "with a firewall rule" do
+      let!(:rule) { client.firewall_rules.create!(firewall_id: firewall.id).resource! }
+
+      it "destroys a rule" do
+        expect { rule.destroy.ready! }.to change { client.firewall_rules.all.size }.by(-1)
+        expect(rule.reload.deleted_at).not_to be_nil
+      end
+
+      it "is part of the firewall's rules list" do
+        expect(firewall.rules.count).to eq(1)
+        expect(firewall.rules.first.id).to eq(rule.id)
+      end
+    end
+  end
+
+  context "with multiple firewalls" do
+    it "lists all firewalls"
+    it "lists firewalls by name"
+    it "lists a firewall's rules"
+  end
+end