cf 0.1.5 → 0.6.0.rc1

data/spec/cf/cli/service/bind_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe CF::Service::Bind do
+  describe 'metadata' do
+    let(:command) { Mothership.commands[:bind_service] }
+
+    describe 'command' do
+      subject { command }
+      its(:description) { should eq "Bind a service to an application" }
+      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 },
+          { :type => :optional, :value => nil, :name => :app }
+        ])
+      end
+    end
+  end
+end

data/spec/cf/cli/service/delete_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe CF::Service::Delete do
+  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
+end

data/spec/cf/cli/service/rename_spec.rb

@@ -0,0 +1,105 @@
+require 'spec_helper'
+require "cf/cli/service/rename"
+
+describe CF::Service::Rename do
+  let(:global) { { :color => false, :quiet => true } }
+  let(:inputs) { {} }
+  let(:given) { {} }
+  let(:client) { fake_client }
+  let(:service) {}
+  let(:new_name) { "some-new-name" }
+
+  before do
+    any_instance_of(CF::CLI) do |cli|
+      stub(cli).client { client }
+      stub(cli).precondition { nil }
+    end
+  end
+
+  subject { Mothership.new.invoke(:rename_service, inputs, given, global) }
+
+  describe 'metadata' do
+    let(:command) { Mothership.commands[:rename_service] }
+
+    describe 'command' do
+      subject { command }
+      its(:description) { should eq "Rename 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 },
+          { :type => :optional, :value => nil, :name => :name }
+        ])
+      end
+    end
+  end
+
+  context 'when there are no services' do
+    context 'and a service is given' do
+      let(:given) { { :service => "some-service" } }
+      it { expect { subject }.to raise_error(CF::UserError, "Unknown service 'some-service'.") }
+    end
+
+    context 'and a service is not given' do
+      it { expect { subject }.to raise_error(CF::UserError, "No services.") }
+    end
+  end
+
+  context 'when there are services' do
+    let(:client) { fake_client(:service_instances => services) }
+    let(:services) { fake_list(:service_instance, 2) }
+    let(:renamed_service) { services.first }
+
+    context 'when the defaults are used' do
+      it 'asks for the service and new name and renames' do
+        mock_ask("Rename which service?", anything) { renamed_service }
+        mock_ask("New name") { new_name }
+        mock(renamed_service).name=(new_name)
+        mock(renamed_service).update!
+        subject
+      end
+    end
+
+    context 'when no name is provided, but a service is' do
+      let(:given) { { :service => renamed_service.name } }
+
+      it 'asks for the new name and renames' do
+        dont_allow_ask("Rename which service?", anything)
+        mock_ask("New name") { new_name }
+        mock(renamed_service).name=(new_name)
+        mock(renamed_service).update!
+        subject
+      end
+    end
+
+    context 'when a service is provided and a name' do
+      let(:inputs) { { :service => renamed_service, :name => new_name } }
+
+      it 'renames the service' do
+        mock(renamed_service).update!
+        subject
+      end
+
+      it 'displays the progress' do
+        mock_with_progress("Renaming to #{new_name}")
+        mock(renamed_service).update!
+
+        subject
+      end
+
+      context 'and the name already exists' do
+        it 'fails' do
+          mock(renamed_service).update! { raise CFoundry::ServiceInstanceNameTaken.new("Taken", 200) }
+          expect { subject }.to raise_error(CFoundry::ServiceInstanceNameTaken)
+        end
+      end
+    end
+  end
+end
+

data/spec/cf/cli/service/service_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+describe CF::Service::Service do
+  describe 'metadata' do
+    let(:command) { Mothership.commands[:service] }
+
+    describe 'command' do
+      subject { command }
+      its(:description) { should eq "Show service information" }
+      it { expect(Mothership::Help.group(:services)).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 => :required, :value=>nil, :name=>:service}])
+      end
+    end
+  end
+end
+

data/spec/cf/cli/service/unbind_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe CF::Service::Unbind do
+  describe 'metadata' do
+    let(:command) { Mothership.commands[:unbind_service] }
+
+    describe 'command' do
+      subject { command }
+      its(:description) { should eq "Unbind a service from an application" }
+      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 },
+          { :type => :optional, :value => nil, :name => :app }
+        ])
+      end
+    end
+  end
+end

data/spec/cf/cli/space/create_spec.rb

@@ -0,0 +1,93 @@
+require 'spec_helper'
+require "cf/cli/space/create"
+
+command CF::Space::Create do
+  describe 'metadata' do
+    let(:command) { Mothership.commands[:create_space] }
+
+    describe 'command' do
+      subject { command }
+      its(:description) { should eq "Create a space in an organization" }
+      it { expect(Mothership::Help.group(:spaces)).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 => :name },
+          { :type => :optional, :value => nil, :name => :organization }
+        ])
+      end
+    end
+  end
+
+  describe "running the command" do
+    let(:client) do
+      fake_client(:current_organization => organization)
+    end
+
+    let(:organization) { fake(:organization) }
+
+    let(:new_space) { fake :space, :name => new_name }
+    let(:new_name) { "some-new-name" }
+
+    before do
+      stub(client).space { new_space }
+      stub(new_space).create!
+      stub(new_space).add_manager
+      stub(new_space).add_developer
+      stub(new_space).add_auditor
+    end
+
+    context "when --target is given" do
+      subject { cf %W[create-space #{new_space.name} --target] }
+
+      it "switches them to the new space" do
+        mock_invoke :target, :organization => organization,
+          :space => new_space
+        subject
+      end
+    end
+
+    context "when --target is NOT given" do
+      subject { cf %W[create-space #{new_space.name}] }
+
+      it "tells the user how they can switch to the new space" do
+        subject
+        expect(output).to say("Space created! Use switch-space #{new_space.name} to target it.")
+      end
+    end
+
+    context "when we don't specify an organization" do
+      subject { cf %W[create-space #{new_space.name}] }
+
+      context "when we have a default organization" do
+        it "uses that organization to create a space" do
+          subject
+
+          stdout.rewind
+          expect(stdout.readline).to include "Creating space"
+        end
+      end
+
+      context "when we don't have a default organization" do
+        let(:organization) { nil }
+
+        it "shows the help for the command" do
+          subject
+
+          stdout.rewind
+          expect(stdout.readline).to include "Create a space in an organization"
+        end
+
+        it "does not try to create the space" do
+          new_space.create! { raise "should not call this method" } # rr not behaving
+          subject
+        end
+      end
+    end
+  end
+end

data/spec/cf/cli/space/rename_spec.rb

@@ -0,0 +1,102 @@
+require 'spec_helper'
+require "cf/cli/space/rename"
+
+describe CF::Space::Rename do
+  let(:spaces) { fake_list(:space, 3) }
+  let(:organization) { fake(:organization, :spaces => spaces) }
+  let(:client) { fake_client(:current_organization => organization, :spaces => spaces) }
+  let(:new_name) { "some-new-name" }
+
+  before do
+    any_instance_of described_class do |cli|
+      stub(cli).client { client }
+      stub(cli).precondition { nil }
+    end
+  end
+
+  describe 'metadata' do
+    let(:command) { Mothership.commands[:rename_space] }
+
+    describe 'command' do
+      subject { command }
+      its(:description) { should eq "Rename a space" }
+      it { expect(Mothership::Help.group(:spaces)).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 => :space },
+          { :type => :optional, :value => nil, :name => :name }
+        ])
+      end
+    end
+  end
+
+  context 'when there are no spaces' do
+    let(:spaces) { [] }
+
+    context 'and a space is given' do
+      subject { cf %W[rename-space --space some-invalid-space --no-force --no-quiet] }
+      it 'prints out an error message' do
+        subject
+        expect(stderr.string).to include "Unknown space 'some-invalid-space'."
+      end
+    end
+
+    context 'and a space is not given' do
+      subject { cf %W[rename-space --no-force] }
+      it 'prints out an error message' do
+        subject
+        expect(stderr.string).to include "No spaces."
+      end
+    end
+  end
+
+  context 'when there are spaces' do
+    let(:renamed_space) { spaces.first }
+
+    context 'when the defaults are used' do
+      subject { cf %W[rename-space --no-force --no-quiet] }
+
+      it 'asks for the space and new name and renames' do
+        mock_ask("Rename which space?", anything) { renamed_space }
+        mock_ask("New name") { new_name }
+        mock(renamed_space).name=(new_name)
+        mock(renamed_space).update!
+        subject
+      end
+    end
+
+    context 'when no name is provided, but a space is' do
+      subject { cf %W[rename-space --space #{renamed_space.name} --no-force] }
+
+      it 'asks for the new name and renames' do
+        dont_allow_ask("Rename which space?", anything)
+        mock_ask("New name") { new_name }
+        mock(renamed_space).name=(new_name)
+        mock(renamed_space).update!
+        subject
+      end
+    end
+
+    context 'when a space is provided and a name' do
+      subject { cf %W[rename-space --space #{renamed_space.name} --name #{new_name} --no-force] }
+
+      it 'renames the space' do
+        mock(renamed_space).update!
+        subject
+      end
+
+      it 'displays the progress' do
+        mock_with_progress("Renaming to #{new_name}")
+        mock(renamed_space).update!
+
+        subject
+      end
+    end
+  end
+end

data/spec/cf/cli/space/spaces_spec.rb

@@ -0,0 +1,104 @@
+require 'spec_helper'
+require 'stringio'
+
+describe CF::Space::Spaces do
+  let(:full) { false }
+  let!(:space_1) { fake(:space, :name => "bb_second", :apps => fake_list(:app, 2), :service_instances => [fake(:service_instance)]) }
+  let!(:space_2) { fake(:space, :name => "aa_first", :apps => [fake(:app)], :service_instances => fake_list(:service_instance, 3), :domains => [fake(:domain)]) }
+  let!(:space_3) { fake(:space, :name => "cc_last", :apps => fake_list(:app, 2), :service_instances => fake_list(:service_instance, 2), :domains => fake_list(:domain, 2)) }
+  let(:spaces) { [space_1, space_2, space_3]}
+  let(:organization) { fake(:organization, :spaces => spaces) }
+  let(:client) { fake_client(:spaces => spaces, :current_organization => organization) }
+
+  before do
+    any_instance_of described_class do |cli|
+      stub(cli).client { client }
+      stub(cli).precondition { nil }
+    end
+  end
+
+  describe 'metadata' do
+    let(:command) { Mothership.commands[:spaces] }
+
+    describe 'command' do
+      subject { command }
+      its(:description) { should eq "List spaces in an organization" }
+      it { expect(Mothership::Help.group(:spaces)).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 => :organization }])
+      end
+    end
+  end
+
+  subject { cf %W[spaces --#{bool_flag(:full)} --no-quiet] }
+
+  it 'should have the correct first two lines' do
+    subject
+
+    stdout.rewind
+    expect(stdout.readline).to match /Getting spaces.*OK/
+    expect(stdout.readline).to eq "\n"
+  end
+
+  context 'when there are no spaces' do
+    let(:spaces) { [] }
+
+    context 'and the full flag is given' do
+      let(:full) { true }
+
+      it 'displays yaml-style output with all space details' do
+        any_instance_of CF::Space::Spaces do |spaces|
+          dont_allow(spaces).invoke
+        end
+        subject
+      end
+    end
+
+    context 'and the full flag is not given (default is false)' do
+      it 'should show only the progress' do
+        subject
+
+        stdout.rewind
+        expect(stdout.readline).to match /Getting spaces.*OK/
+        expect(stdout).to be_eof
+      end
+    end
+  end
+
+  context 'when there are spaces' do
+    context 'and the full flag is given' do
+      let(:full) { true }
+
+      it 'displays yaml-style output with all space details' do
+        any_instance_of CF::Space::Spaces do |spaces|
+          mock(spaces).invoke(:space, :space => space_2, :full => true).ordered
+          mock(spaces).invoke(:space, :space => space_1, :full => true).ordered
+          mock(spaces).invoke(:space, :space => space_3, :full => true).ordered
+        end
+        subject
+      end
+    end
+
+    context 'and the full flag is not given (default is false)' do
+      it 'displays tabular output with names, spaces and domains' do
+        subject
+
+        stdout.rewind
+        stdout.readline
+        stdout.readline
+
+        expect(stdout.readline).to match /name\s+apps\s+services/
+        spaces.sort_by(&:name).each do |space|
+          expect(stdout.readline).to match /#{space.name}\s+#{name_list(space.apps)}\s+#{name_list(space.service_instances)}/
+        end
+        expect(stdout).to be_eof
+      end
+    end
+  end
+end