af 0.3.22 → 0.5.0.beta.1

data/spec/vmc/cli/organization/rename_spec.rb

@@ -0,0 +1,113 @@
+require 'spec_helper'
+require "vmc/cli/organization/rename"
+
+describe VMC::Organization::Rename do
+  let(:global) { { :color => false, :quiet => true } }
+  let(:inputs) { {} }
+  let(:given) { {} }
+  let(:organizations) { fake_list(:organization, 3) }
+  let(:client) { fake_client(:organizations => organizations) }
+  let(:new_name) { "some-new-name" }
+
+  before do
+    any_instance_of(VMC::CLI) do |cli|
+      stub(cli).client { client }
+      stub(cli).precondition { nil }
+    end
+  end
+
+  subject { Mothership.new.invoke(:rename_org, inputs, given, global) }
+
+  describe 'metadata' do
+    let(:command) { Mothership.commands[:rename_org] }
+
+    describe 'command' do
+      subject { command }
+      its(:description) { should eq "Rename an organization" }
+      it { expect(Mothership::Help.group(:organizations)).to include(subject) }
+    end
+
+    describe 'inputs' do
+      subject { command.inputs }
+
+      it "is not missing any descriptions" do
+        subject.each do |input, attrs|
+          expect(attrs[:description]).to be
+          expect(attrs[:description].strip).to_not be_empty
+        end
+      end
+    end
+
+    describe 'arguments' do
+      subject { command.arguments }
+      it 'has the correct argument order' do
+        should eq([
+          { :type => :optional, :value => nil, :name => :organization },
+          { :type => :optional, :value => nil, :name => :name }
+        ])
+      end
+    end
+  end
+
+  context 'when there are no organizations' do
+    let(:organizations) { [] }
+
+    context 'and an organization is given' do
+      let(:given) { { :organization => "some-invalid-organization" } }
+      it { expect { subject }.to raise_error(VMC::UserError, "Unknown organization 'some-invalid-organization'.") }
+    end
+
+    context 'and an organization is not given' do
+      it { expect { subject }.to raise_error(VMC::UserError, "No organizations.") }
+    end
+  end
+
+  context 'when there are organizations' do
+    let(:renamed_organization) { organizations.first }
+
+    context 'when the defaults are used' do
+      it 'asks for the organization and new name and renames' do
+        mock_ask("Rename which organization?", anything) { renamed_organization }
+        mock_ask("New name") { new_name }
+        mock(renamed_organization).name=(new_name)
+        mock(renamed_organization).update!
+        subject
+      end
+    end
+
+    context 'when no name is provided, but an organization is' do
+      let(:given) { { :organization => renamed_organization.name } }
+
+      it 'asks for the new name and renames' do
+        dont_allow_ask("Rename which organization?", anything)
+        mock_ask("New name") { new_name }
+        mock(renamed_organization).name=(new_name)
+        mock(renamed_organization).update!
+        subject
+      end
+    end
+
+    context 'when an organization is provided and a name' do
+      let(:inputs) { { :organization => renamed_organization, :name => new_name } }
+
+      it 'renames the organization' do
+        mock(renamed_organization).update!
+        subject
+      end
+
+      it 'displays the progress' do
+        mock_with_progress("Renaming to #{new_name}")
+        mock(renamed_organization).update!
+
+        subject
+      end
+
+      context 'and the name already exists' do
+        it 'fails' do
+          mock(renamed_organization).update! { raise CFoundry::OrganizationNameTaken.new("Bad error", 200) }
+          expect { subject }.to raise_error(CFoundry::OrganizationNameTaken)
+        end
+      end
+    end
+  end
+end

data/spec/vmc/cli/route/map_spec.rb

@@ -0,0 +1,138 @@
+require 'spec_helper'
+
+describe VMC::Route::Map do
+  let(:inputs) { {} }
+  let(:global) { { :color => false } }
+  let(:given) { {} }
+  let(:client) { fake_client }
+  let!(:cli) { described_class.new }
+
+  before do
+    stub(cli).client { client }
+    stub_output(cli)
+  end
+
+  let(:app){ fake(:app, :space => space, :name => "app-name") }
+  let(:space) { fake(:space, :name => "space-name", :domains => space_domains) }
+  let(:domain) { fake(:domain, :name => domain_name ) }
+  let(:domain_name) { "some-domain.com" }
+  let(:host_name) { "some-host" }
+  let(:space_domains) { [] }
+
+  subject { invoke_cli(cli, :map, inputs, given, global) }
+
+  context 'when targeting v2' do
+    shared_examples "mapping the route to the app" do
+      context 'and the domain is mapped to the space' do
+        let(:space_domains) { [domain] }
+
+        context 'and the route is mapped to the space' do
+          let(:client) { fake_client :routes => [route] }
+          let(:route) { fake(:route, :space => space, :host => host_name, :domain => domain) }
+
+          it 'binds the route to the app' do
+            mock(app).add_route(route)
+            subject
+          end
+        end
+
+        context 'and the route is not mapped to the space' do
+          let(:new_route) { fake(:route) }
+
+          before do
+            stub(client).route { new_route }
+            stub(app).add_route
+            stub(new_route).create!
+          end
+
+          it 'indicates that it is creating a route' do
+            mock(cli).print("Creating route #{host_name}.#{domain_name}")
+            subject
+          end
+
+          it "creates the route in the app's space" do
+            mock(new_route).create!
+            subject
+            expect(new_route.host).to eq host_name
+            expect(new_route.domain).to eq domain
+            expect(new_route.space).to eq space
+          end
+
+          it 'indicates that it is binding the route' do
+            mock(cli).print("Binding #{host_name}.#{domain_name} to app-name")
+            subject
+          end
+
+          it 'binds the route to the app' do
+            mock(app).add_route(new_route)
+            subject
+          end
+        end
+      end
+    end
+
+    context 'when an app is specified' do
+      let(:inputs) { { :domain => domain, :host => host_name, :app => app } }
+
+      context 'and the domain is not already mapped to the space' do
+        let(:space_domains) { [] }
+        let(:inputs) { { :app => app } }
+        let(:given) { { :domain => "some-bad-domain" }}
+
+        it 'indicates that the domain is invalid' do
+          expect { subject }.to raise_error(VMC::UserError, /Unknown domain/)
+        end
+      end
+
+      include_examples "mapping the route to the app"
+    end
+
+    context 'when an app is not specified' do
+      let(:inputs) { { :domain => domain, :host => host_name } }
+      let(:space_domains) { [domain] }
+      let(:new_route) { fake(:route) }
+
+      before { stub_ask("Which application?", anything) { app } }
+
+      it 'asks for an app' do
+        stub(client).route { new_route }
+        stub(app).add_route
+        stub(new_route).create!
+        mock_ask("Which application?", anything) { app }
+        subject
+      end
+
+      include_examples "mapping the route to the app"
+    end
+
+    context "when a host is not specified" do
+      let(:inputs) { { :domain => domain, :app => app } }
+      let(:new_route) { fake(:route) }
+
+      before do
+        stub(client).route { new_route }
+        stub(app).add_route
+        stub(new_route).create!
+      end
+
+      it "creates a route with an empty string as its host" do
+        mock(new_route).create!
+        subject
+        expect(new_route.host).to eq ""
+      end
+    end
+  end
+
+  context 'when targeting v1' do
+    let(:given) { { :domain => "foo.bar.com" } }
+    let(:app) { v1_fake :app, :name => "foo" }
+    let(:client) { v1_fake_client }
+    let(:inputs) { { :app => app } }
+
+    it "adds the domain to the app's urls" do
+      stub(app).update!
+      subject
+      expect(app.urls).to include "foo.bar.com"
+    end
+  end
+end

data/spec/vmc/cli/route/unmap_spec.rb

@@ -0,0 +1,215 @@
+require 'spec_helper'
+
+describe VMC::Route::Unmap do
+  let(:global) { { :color => false } }
+  let(:given) { {} }
+  let(:inputs) { {} }
+  let(:client) { fake_client }
+  let!(:cli) { described_class.new }
+  let(:app){ fake(:app, :space => space, :name => "app-name") }
+  let(:space) { fake(:space, :name => "space-name", :domains => space_domains) }
+  let(:domain) { fake(:domain, :name => domain_name ) }
+  let(:domain_name) { "some-domain.com" }
+  let(:host_name) { "some-host" }
+  let(:url) { "#{host_name}.#{domain_name}" }
+  let(:space_domains) { [domain] }
+
+  before do
+    stub(cli).client { client }
+    stub_output(cli)
+  end
+
+  subject { invoke_cli(cli, :unmap, inputs, given, global) }
+
+  describe 'metadata' do
+    let(:command) { Mothership.commands[:delete_service] }
+
+    describe 'command' do
+      subject { command }
+      its(:description) { should eq "Delete a service" }
+      it { expect(Mothership::Help.group(:services, :manage)).to include(subject) }
+    end
+
+    include_examples 'inputs must have descriptions'
+
+    describe 'arguments' do
+      subject { command.arguments }
+      it 'has the correct argument order' do
+        should eq([{:type => :optional, :value => nil, :name => :service }])
+      end
+    end
+  end
+
+  context 'when targeting v2' do
+    context "when an app and a url are specified" do
+      let(:given) { { :url => url } }
+      let(:inputs) { { :app => app } }
+
+      context "when the given route is mapped to the given app" do
+        let(:client) { fake_client :routes => [route] }
+        let(:app) { fake(:app, :space => space, :name => "app-name", :routes => [route]) }
+        let(:route) { fake(:route, :space => space, :host => host_name, :domain => domain) }
+
+        it "unmaps the url from the app" do
+          mock(app).remove_route(route)
+          subject
+        end
+      end
+
+      context "when the given route is NOT mapped to the given app" do
+        it "displays an error" do
+          expect { subject }.to raise_error(VMC::UserError, /Unknown route/)
+        end
+      end
+    end
+
+    context "when only an app is specified" do
+      let(:other_route) { fake(:route, :host => "abcd", :domain => domain) }
+      let(:route) { fake(:route, :host => "efgh", :domain => domain) }
+      let(:app) { fake(:app, :space => space, :routes => [route, other_route] )}
+      let(:inputs) { { :app => app } }
+
+      before { stub(app).remove_route(route) }
+
+      it "asks the user to select from the app's urls" do
+        mock_ask("Which URL?", anything) do |_, opts|
+          expect(opts[:choices]).to eq [other_route, route]
+          route
+        end
+
+        subject
+      end
+
+      it "unmaps the selected url from the app" do
+        stub_ask("Which URL?", anything) { route }
+        mock(app).remove_route(route)
+        subject
+      end
+    end
+
+    context "when an app is specified and the --all option is given" do
+      let(:other_route) { fake(:route, :host => "abcd", :domain => domain) }
+      let(:route) { fake(:route, :host => "efgh", :domain => domain) }
+      let(:app) { fake(:app, :routes => [route, other_route]) }
+      let(:inputs) { { :app => app, :all => true } }
+
+      it "unmaps all routes from the given app" do
+        mock(app).remove_route(route)
+        mock(app).remove_route(other_route)
+        subject
+      end
+    end
+
+    context "when a url is specified and the --delete option is given" do
+      let(:route) { fake(:route, :host => host_name, :domain => domain) }
+      let(:client) { fake_client :routes => [route] }
+      let(:given) { { :url => url } }
+      let(:inputs) { { :delete => true } }
+
+      it "deletes the route" do
+        mock(route).delete!
+        subject
+      end
+    end
+
+    context "when the --delete and --all options are both passed" do
+      let(:inputs) { { :delete => true, :all => true } }
+      let(:other_route) { fake(:route, :host => "abcd", :domain => domain) }
+      let(:route) { fake(:route, :host => "efgh", :domain => domain) }
+      let(:client) { fake_client :routes => [route, other_route] }
+
+      it "asks if the user really wants to unmap all urls" do
+        mock_ask("Really delete ALL URLS?", :default => false) { false }
+        subject
+      end
+
+      context "when the user responds with a yes" do
+        before { stub_ask("Really delete ALL URLS?", anything) { true } }
+
+        it "deletes all the user's routes" do
+          client.routes.each { |r| mock(r).delete! }
+          subject
+        end
+      end
+
+      context "when the user responds with a no" do
+        before { stub_ask("Really delete ALL URLS?", anything) { false } }
+
+        it "does not delete any routes" do
+          any_instance_of(route.class) do |route|
+            dont_allow(route).delete!
+          end
+          subject
+        end
+      end
+    end
+
+    context "when only a url is passed" do
+      let(:route) { fake(:route, :host => host_name, :domain => domain) }
+      let(:client) { fake_client :routes => [route] }
+      let(:given) { { :url => url } }
+
+      it "displays an error message" do
+        expect { subject }.to raise_error(VMC::UserError, /Missing either --delete or --app/)
+      end
+    end
+  end
+
+  context 'when targeting v1' do
+    let(:client) { CFoundry::V1::Client.new }
+    let(:app) { CFoundry::V1::App.new("some-app", client) }
+    let(:other_url) { "some.other.url.com" }
+
+    context "when an app and a url are specified" do
+      let(:given) { { :url => url } }
+      let(:inputs) { { :app => app } }
+
+      context "when the given url is not mapped to the app" do
+        before { app.urls = [other_url] }
+
+        it "displays an error message" do
+          expect { subject }.to raise_error(VMC::UserError, /URL.*not mapped/)
+        end
+      end
+
+      context "when the given url is mapped to the app" do
+        before { app.urls = [url, other_url] }
+
+        it "unmaps the url from the app" do
+          mock(app).update!
+          subject
+          expect(app.urls).to eq [other_url]
+        end
+      end
+    end
+
+    context "when only an app is specified" do
+      let(:inputs) { { :app => app } }
+      before { app.urls = [url, other_url] }
+
+      it "asks for the url" do
+        mock_ask("Which URL?", :choices => [url, other_url]) { url }
+        stub(app).update!
+        subject
+      end
+
+      it "unmaps the selected url from the app" do
+        stub_ask("Which URL?", anything) { url }
+        mock(app).update!
+        subject
+        expect(app.urls).to eq [other_url]
+      end
+    end
+
+    context "when an app is specified and the --all option is given" do
+      let(:inputs) { { :app => app, :all => true } }
+
+      it "unmaps all routes from the given app" do
+        app.urls = ["foo", "bar"]
+        mock(app).update!
+        subject
+        expect(app.urls).to eq []
+      end
+    end
+  end
+end