vmc 0.5.0.beta.7 → 0.5.0.beta.10

data/spec/spec_helper.rb

@@ -2,15 +2,25 @@ SPEC_ROOT = File.dirname(__FILE__).freeze
 
 require "rspec"
 require "cfoundry"
-require "cfoundry/spec_helper"
+require "cfoundry/test_support"
 require "vmc"
+require "vmc/test_support"
 
 Dir[File.expand_path('../support/**/*.rb', __FILE__)].each do |file|
   require file
 end
 
 RSpec.configure do |c|
+  c.include Fake::FakeMethods
   c.mock_with :rr
+
+  c.include VMC::TestSupport::FakeHomeDir
+  c.include VMC::TestSupport::CommandHelper
+  c.include VMC::TestSupport::InteractHelper
+
+  c.before do
+    VMC::CLI.send(:class_variable_set, :@@client, nil)
+  end
 end
 
 class String
@@ -25,17 +35,6 @@ class String
   end
 end
 
-def with_output_to(output = StringIO.new)
-  old_out = $stdout
-  old_err = $stderr
-  $stdout = output
-  $stderr = output
-  yield output
-ensure
-  $stdout = old_out
-  $stderr = old_err
-end
-
 def name_list(xs)
   if xs.empty?
     "none"
@@ -43,3 +42,16 @@ def name_list(xs)
     xs.collect(&:name).join(", ")
   end
 end
+
+def invoke_cli(cli, *args)
+  stub.proxy(cli).invoke.with_any_args
+  stub(cli.class).new { cli }
+  cli.invoke(*args)
+end
+
+def stub_output(cli)
+  stub(cli).print
+  stub(cli).puts
+  stub(Interact::Progress::Dots).start!
+  stub(Interact::Progress::Dots).stop!
+end

data/spec/vmc/cli/app/instances_spec.rb

@@ -5,7 +5,6 @@ describe VMC::App::Stats do
   let(:global) { { :color => false } }
   let(:inputs) { {:app => apps[0]} }
   let(:given) { {} }
-  let(:output) { StringIO.new }
   let(:client) { fake_client(:apps => apps) }
   let(:apps) { [fake(:app, :name => "basic_app")] }
   let(:time) { Time.local(2012,11,1,2,30)}
@@ -25,7 +24,7 @@ describe VMC::App::Stats do
   end
 
   subject do
-    with_output_to output do
+    capture_output do
       Mothership.new.invoke(:instances, inputs, given, global)
     end
   end
@@ -51,16 +50,12 @@ describe VMC::App::Stats do
 
   it 'prints out the instances in the correct order' do
     subject
-
-    output.rewind
-    expect(output.string).to match /.*instance \#1.*instance \#2.*instance \#12.*/m
+    expect(stdout.string).to match /.*instance \#1.*instance \#2.*instance \#12.*/m
   end
 
   it 'prints out one of the instances correctly' do
     subject
-
-    output.rewind
-    expect(output.string).to include <<-OUT.strip_heredoc
+    expect(stdout.string).to include <<-OUT.strip_heredoc
       instance #2: started
         started: #{time.strftime("%F %r")}
         debugger: port bar at foo

data/spec/vmc/cli/app/push/create_spec.rb

@@ -46,7 +46,8 @@ describe VMC::App::Create do
         :framework => framework,
         :runtime => runtime,
         :memory => "1G",
-        :command => "ruby main.rb"
+        :command => "ruby main.rb",
+        :buildpack => "git://example.com"
       }
     end
 
@@ -59,6 +60,7 @@ describe VMC::App::Create do
       its([:command]) { should eq "ruby main.rb" }
       its([:runtime]) { should eq runtime }
       its([:memory]) { should eq 1024 }
+      its([:buildpack]) { should eq "git://example.com" }
     end
 
     context 'when certain inputs are not given' do
@@ -279,7 +281,8 @@ describe VMC::App::Create do
         :framework => framework,
         :runtime => runtime,
         :production => false,
-        :memory => 1024
+        :memory => 1024,
+        :buildpack => "git://example.com"
       }
     end
 
@@ -288,15 +291,15 @@ describe VMC::App::Create do
     subject { create.create_app(attributes) }
 
     it 'creates an app based on the resulting inputs' do
-      attributes.each do |key, val|
-        mock(app).__send__(:"#{key}=", val)
-      end
-
       mock(create).filter(:create_app, app) { app }
 
       mock(app).create!
 
       subject
+
+      attributes.each do |key, val|
+        expect(app.send(key)).to eq val
+      end
     end
   end
 
@@ -337,7 +340,7 @@ describe VMC::App::Create do
         mock_ask('URL', anything) { url_choices.first }
 
         mock(create).invoke(:map, :app => app, :url => url_choices.first) do
-          raise CFoundry::RouteHostTaken.new(nil, nil, "foo", 1234)
+          raise CFoundry::RouteHostTaken.new("foo", 1234)
         end
       end
 

data/spec/vmc/cli/app/push_spec.rb

@@ -120,7 +120,7 @@ describe VMC::App::Push do
 
       it 'outputs the changed instances' do
         mock(push).line("Changes:")
-        mock(push).line("instances: 1 -> 2")
+        mock(push).line("total_instances: 1 -> 2")
         stub(app).update!
         subject
       end

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

@@ -95,7 +95,7 @@ describe VMC::App::Rename do
 
       context 'and the name already exists' do
         it 'fails' do
-          mock(renamed_app).update! { raise CFoundry::AppNameTaken.new(nil, nil, "Bad Name", 404) }
+          mock(renamed_app).update! { raise CFoundry::AppNameTaken.new("Bad Name", 404) }
           expect { subject }.to raise_error(CFoundry::AppNameTaken)
         end
       end

data/spec/vmc/cli/app/scale_spec.rb

@@ -0,0 +1,81 @@
+require "spec_helper"
+require "webmock/rspec"
+
+describe VMC::App::Scale do
+  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
+
+  subject { invoke_cli(cli, :scale, inputs, given, global) }
+
+  context "when the --disk flag is given" do
+    let(:before_value) { 512 }
+    let(:app) { fake :app, :disk_quota => before_value }
+    let(:inputs) { { :app => app, :disk => "1G" } }
+
+    it "changes the application's disk quota" do
+      mock(app).update!
+      expect { subject }.to change(app, :disk_quota).from(before_value).to(1024)
+    end
+  end
+
+  context "when the --memory flag is given" do
+    let(:before_value) { 512 }
+    let(:app) { fake :app, :memory => before_value }
+    let(:inputs) { { :app => app, :memory => "1G" } }
+
+    it "changes the application's memory" do
+      mock(app).update!
+      expect { subject }.to change(app, :memory).from(before_value).to(1024)
+    end
+
+    context "if --restart is true" do
+      it "restarts the application" do
+        stub(app).update!
+        stub(app).started? { true }
+        mock(cli).invoke :restart, :app => app
+        subject
+      end
+    end
+  end
+
+  context "when the --instances flag is given" do
+    let(:before_value) { 3 }
+    let(:app) { fake :app, :total_instances => before_value }
+
+    let(:inputs) { { :app => app, :instances => 5 } }
+
+    it "changes the application's number of instances" do
+      mock(app).update!
+      expect { subject }.to change(app, :total_instances).from(before_value).to(5)
+    end
+  end
+
+  context "when the --plan flag is given" do
+    context "when the plan name begins with a 'p'" do
+      let(:app) { fake :app, :production => false }
+      let(:inputs) { { :app => app, :plan => "P100" } }
+
+      it "changes the application's 'production' flag to true" do
+        mock(app).update!
+        expect { subject }.to change(app, :production).from(false).to(true)
+      end
+    end
+
+    context "when the plan name does not begin with a 'p'" do
+      let(:app) { fake :app, :production => true }
+      let(:inputs) { { :app => app, :plan => "D100" } }
+
+      it "changes the application's 'production' flag to false" do
+        mock(app).update!
+        expect { subject }.to change(app, :production).from(true).to(false)
+      end
+    end
+  end
+end

data/spec/vmc/cli/app/stats_spec.rb

@@ -5,7 +5,6 @@ describe VMC::App::Stats do
   let(:global) { { :color => false } }
   let(:inputs) { {:app => apps[0]} }
   let(:given) { {} }
-  let(:output) { StringIO.new }
   let(:client) { fake_client(:apps => apps) }
   let(:apps) { [fake(:app, :name => "basic_app")] }
 
@@ -33,9 +32,7 @@ describe VMC::App::Stats do
   end
 
   subject do
-    with_output_to output do
-      Mothership.new.invoke(:stats, inputs, given, global)
-    end
+    capture_output { Mothership.new.invoke(:stats, inputs, given, global) }
   end
 
   describe 'metadata' do
@@ -59,8 +56,7 @@ describe VMC::App::Stats do
 
   it 'prints out the stats' do
     subject
-
-    output.rewind
-    expect(output.readlines.last).to match /.*0\s+0\.0% of\s+cores\s+29\.9M of 288M\s+14\.9M of 256M.*/
+    stdout.rewind
+    expect(stdout.readlines.last).to match /.*0\s+0\.0% of\s+cores\s+29\.9M of 288M\s+14\.9M of 256M.*/
   end
 end

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

@@ -0,0 +1,140 @@
+require 'spec_helper'
+
+describe VMC::Domain::Map do
+  let(:global) { { :color => false } }
+  let(:given) { {} }
+  let(:client) { fake_client :current_organization => organization, :current_space => space }
+  let!(:cli) { described_class.new }
+
+  before do
+    stub(cli).client { client }
+    stub_output(cli)
+  end
+
+  let(:organization) { fake(:organization) }
+  let(:space) { fake(:space) }
+  let(:domain) { fake(:domain, :name => domain_name) }
+  let(:domain_name) { "some.domain.com" }
+
+  subject { invoke_cli(cli, :map_domain, inputs, given, global) }
+
+  shared_examples "binding a domain to a space" do
+    it "adds the domain to the space's organization" do
+      mock(space.organization).add_domain(domain)
+      stub(space).add_domain(domain)
+      subject
+    end
+
+    it 'adds the domain to the space' do
+      stub(space.organization).add_domain(domain)
+      mock(space).add_domain(domain)
+      subject
+    end
+  end
+
+  shared_examples "binding a domain to an organization" do
+    it 'does NOT add the domain to a space' do
+      any_instance_of(space.class) do |space|
+        dont_allow(space).add_domain(domain)
+      end
+    end
+
+    it 'adds the domain to the organization' do
+      mock(organization).add_domain(domain)
+      subject
+    end
+  end
+
+  shared_examples "mapping a domain to a space" do
+    context "when the domain does NOT exist" do
+      before do
+        stub(client).domain { domain }
+        stub(domain).create!
+        stub(space.organization).add_domain(domain)
+        stub(space).add_domain(domain)
+      end
+
+      it 'creates the domain' do
+        mock(domain).create!
+        subject
+        expect(domain.name).to eq domain_name
+        expect(domain.owning_organization).to eq organization
+      end
+
+      include_examples "binding a domain to a space"
+    end
+
+    context "when the domain already exists" do
+      let(:client) {
+        fake_client :domains => [domain],
+                    :current_organization => organization,
+                    :current_space => space
+      }
+
+      include_examples "binding a domain to a space"
+    end
+  end
+
+  context 'when a domain and a space are passed' do
+    let(:inputs) { { :space => space, :name => domain_name } }
+
+    include_examples "mapping a domain to a space"
+  end
+
+  context 'when a domain and an organization are passed' do
+    let(:inputs) { { :organization => organization, :name => domain_name } }
+
+    context "and the domain does NOT exist" do
+      before do
+        stub(client).domain { domain }
+        stub(domain).create!
+        stub(organization).add_domain(domain)
+      end
+
+      include_examples "binding a domain to an organization"
+
+      it 'adds the domain to the organization' do
+        mock(organization).add_domain(domain)
+        subject
+      end
+
+      context "and the --shared option is passed" do
+        let(:inputs) { { :organization => organization, :name => domain_name, :shared => true } }
+
+        it 'adds the domain to the organization' do
+          mock(domain).create!
+          subject
+          expect(domain.name).to eq domain_name
+        end
+
+        it "does not add the domain to a specific organization" do
+          stub(domain).create!
+          subject
+          expect(domain.owning_organization).to be_nil
+        end
+      end
+    end
+
+    context "and the domain already exists" do
+      let(:client) {
+        fake_client :domains => [domain],
+                    :current_organization => organization,
+                    :current_space => space
+      }
+
+      include_examples "binding a domain to an organization"
+    end
+  end
+
+  context 'when a domain, organization, and space is passed' do
+    let(:inputs) { { :name => domain_name, :organization => organization, :space => space } }
+
+    include_examples "mapping a domain to a space"
+  end
+
+  context 'when only a domain is passed' do
+    let(:inputs) { { :name => domain_name } }
+
+    include_examples "mapping a domain to a space"
+  end
+end

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

@@ -0,0 +1,73 @@
+require 'spec_helper'
+
+describe VMC::Domain::Unmap do
+  let(:global) { { :color => false } }
+  let(:given) { {} }
+  let(:client) { fake_client :current_organization => organization, :current_space => space }
+  let!(:cli) { described_class.new }
+
+  before do
+    stub(cli).client { client }
+    stub_output(cli)
+  end
+
+  let(:organization) { fake(:organization) }
+  let(:space) { fake(:space) }
+  let(:domain) { fake(:domain, :name => domain_name) }
+  let(:domain_name) { "some.domain.com" }
+
+  subject { invoke_cli(cli, :unmap_domain, inputs, given, global) }
+
+  context "when the --delete flag is given" do
+    let(:inputs) { { :domain => domain, :delete => true } }
+
+    it "asks for a confirmation" do
+      mock_ask("Really delete #{domain_name}?", :default => false) { false }
+      stub(domain).delete!
+      subject
+    end
+
+    context "and the user answers 'no' to the confirmation" do
+      it "does NOT delete the domain" do
+        stub_ask("Really delete #{domain_name}?", anything) { false }
+        dont_allow(domain).delete!
+        subject
+      end
+    end
+
+    context "and the user answers 'yes' to the confirmation" do
+      it "deletes the domain" do
+        stub_ask("Really delete #{domain_name}?", anything) { true }
+        mock(domain).delete!
+        subject
+      end
+    end
+  end
+
+  context "when a space is given" do
+    let(:inputs) { { :domain => domain, :space => space } }
+
+    it "unmaps the domain from the space" do
+      mock(space).remove_domain(domain)
+      subject
+    end
+  end
+
+  context "when an organization is given" do
+    let(:inputs) { { :domain => domain, :organization => organization } }
+
+    it "unmaps the domain from the organization" do
+      mock(organization).remove_domain(domain)
+      subject
+    end
+  end
+
+  context "when only the domain is given" do
+    let(:inputs) { { :domain => domain } }
+
+    it "unmaps the domain from the current space" do
+      mock(client.current_space).remove_domain(domain)
+      subject
+    end
+  end
+end