cf 0.1.5 → 0.6.0.rc1

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

@@ -0,0 +1,369 @@
+require 'spec_helper'
+require "cf/cli/app/push"
+
+describe CF::App::Push do
+  let(:global) { { :color => false, :quiet => true } }
+  let(:inputs) { {} }
+  let(:given) { {} }
+  let(:path) { "somepath" }
+  let(:client) { fake_client }
+  let(:push) { CF::App::Push.new(Mothership.commands[:push]) }
+
+  before do
+    any_instance_of(CF::CLI) do |cli|
+      stub(cli).client { client }
+      stub(cli).precondition { nil }
+    end
+  end
+
+  describe 'metadata' do
+    let(:command) { Mothership.commands[:push] }
+
+    describe 'command' do
+      subject { command }
+      its(:description) { should eq "Push an application, syncing changes if it exists" }
+      it { expect(Mothership::Help.group(:apps, :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 => :name }])
+      end
+    end
+  end
+
+  describe '#sync_app' do
+    let(:app) { fake(:app) }
+
+    before do
+      stub(app).upload
+      app.changes = {}
+    end
+
+    subject do
+      push.input = Mothership::Inputs.new(nil, push, inputs, {}, global)
+      push.sync_app(app, path)
+    end
+
+    shared_examples 'common tests for inputs' do |*args|
+      context 'when the new input is the same as the old' do
+        type, input = args
+        input ||= type
+
+        let(:inputs) { {input => old} }
+
+        it "does not update the app's #{type}" do
+          dont_allow(push).line
+          dont_allow(app).update!
+          expect { subject }.not_to change { app.send(type) }
+        end
+      end
+    end
+
+    it 'triggers the :push_app filter' do
+      mock(push).filter(:push_app, app) { app }
+      subject
+    end
+
+    it 'uploads the app' do
+      mock(app).upload(path)
+      subject
+    end
+
+    context 'when no inputs are given' do
+      let(:inputs) { {} }
+
+      it 'should not update the app' do
+        dont_allow(app).update!
+        subject
+      end
+
+      [:memory=, :framework=].each do |property|
+        it "should not set #{property} on the app" do
+          dont_allow(app).__send__(property)
+          subject
+        end
+      end
+    end
+
+    context 'when memory is given' do
+      let(:old) { 1024 }
+      let(:new) { "2G" }
+      let(:app) { fake(:app, :memory => old) }
+      let(:inputs) { { :memory => new } }
+
+      it 'updates the app memory, converting to megabytes' do
+        stub(push).line(anything)
+        mock(app).update!
+        expect { subject }.to change { app.memory }.from(old).to(2048)
+      end
+
+      it 'outputs the changed memory in human readable sizes' do
+        mock(push).line("Changes:")
+        mock(push).line("memory: 1G -> 2G")
+        stub(app).update!
+        subject
+      end
+
+      include_examples 'common tests for inputs', :memory
+    end
+
+    context 'when instances is given' do
+      let(:old) { 1 }
+      let(:new) { 2 }
+      let(:app) { fake(:app, :total_instances => old) }
+      let(:inputs) { { :instances => new } }
+
+      it 'updates the app instances' do
+        stub(push).line(anything)
+        mock(app).update!
+        expect { subject }.to change { app.total_instances }.from(old).to(new)
+      end
+
+      it 'outputs the changed instances' do
+        mock(push).line("Changes:")
+        mock(push).line("total_instances: 1 -> 2")
+        stub(app).update!
+        subject
+      end
+
+      include_examples 'common tests for inputs', :total_instances, :instances
+    end
+
+    context 'when framework is given' do
+      let(:old) { fake(:framework, :name => "Old Framework") }
+      let(:new) { fake(:framework, :name => "New Framework") }
+      let(:app) { fake(:app, :framework => old) }
+      let(:inputs) { { :framework => new } }
+
+      it 'updates the app framework' do
+        stub(push).line(anything)
+        mock(app).update!
+        expect { subject }.to change { app.framework }.from(old).to(new)
+      end
+
+      it 'outputs the changed framework using the name' do
+        mock(push).line("Changes:")
+        mock(push).line("framework: Old Framework -> New Framework")
+        stub(app).update!
+        subject
+      end
+
+      include_examples 'common tests for inputs', :framework
+    end
+
+    context 'when runtime is given' do
+      let(:old) { fake(:runtime, :name => "Old Runtime") }
+      let(:new) { fake(:runtime, :name => "New Runtime") }
+      let(:app) { fake(:app, :runtime => old) }
+      let(:inputs) { { :runtime => new } }
+
+      it 'updates the app runtime' do
+        stub(push).line(anything)
+        mock(app).update!
+        expect { subject }.to change { app.runtime }.from(old).to(new)
+      end
+
+      it 'outputs the changed runtime using the name' do
+        mock(push).line("Changes:")
+        mock(push).line("runtime: Old Runtime -> New Runtime")
+        stub(app).update!
+        subject
+      end
+
+      include_examples 'common tests for inputs', :runtime
+    end
+
+    context 'when command is given' do
+      let(:old) { "./start" }
+      let(:new) { "./start foo " }
+      let(:app) { fake(:app, :command => old) }
+      let(:inputs) { { :command => new } }
+
+      it 'updates the app command' do
+        stub(push).line(anything)
+        mock(app).update!
+        expect { subject }.to change { app.command }.from("./start").to("./start foo ")
+      end
+
+      it 'outputs the changed command in single quotes' do
+        mock(push).line("Changes:")
+        mock(push).line("command: './start' -> './start foo '")
+        stub(app).update!
+        subject
+      end
+
+      include_examples 'common tests for inputs', :command
+    end
+
+    context 'when plan is given' do
+      let(:old) { "d100" }
+      let(:new) { "p100" }
+      let(:inputs) { { :plan => new } }
+
+      include_examples 'common tests for inputs', :production, :plan
+
+      %w{p100 P100 P200}.each do |plan|
+        context "when the given plan is #{plan}" do
+          let(:inputs) { { :plan => plan } }
+
+          it 'sets production to true' do
+            stub(push).line(anything)
+            mock(app).update!
+            expect { subject }.to change { app.production }.from(false).to(true)
+          end
+
+          it 'outputs the changed plan in single quotes' do
+            mock(push).line("Changes:")
+            mock(push).line("production: false -> true")
+            stub(app).update!
+            subject
+          end
+        end
+      end
+
+      %w{d100 D100 D200 fizzbuzz}.each do |plan|
+        context "when the given plan is #{plan}" do
+          let(:app) { fake(:app, :production => true) }
+
+          let(:inputs) { { :plan => plan } }
+
+          it 'sets production to false' do
+            stub(push).line(anything)
+            mock(app).update!
+            expect { subject }.to change { app.production }.from(true).to(false)
+          end
+
+          it 'outputs the changed plan in single quotes' do
+            mock(push).line("Changes:")
+            mock(push).line("production: true -> false")
+            stub(app).update!
+            subject
+          end
+        end
+      end
+    end
+
+    context 'when restart is given' do
+      let(:inputs) { { :restart => true, :memory => 4096 } }
+
+
+      context 'when the app is already started' do
+        let(:app) { fake(:app, :state => "STARTED") }
+
+        it 'invokes the restart command' do
+          stub(push).line
+          mock(app).update!
+          mock(push).invoke(:restart, :app => app)
+          subject
+        end
+
+        context 'but there are no changes' do
+          let(:inputs) { { :restart => true } }
+
+          it 'invokes the restart command' do
+            stub(push).line
+            dont_allow(app).update!
+            mock(push).invoke(:restart, :app => app)
+            subject
+          end
+        end
+      end
+
+      context 'when the app is not already started' do
+        let(:app) { fake(:app, :state => "STOPPED") }
+
+        it 'does not invoke the restart command' do
+          stub(push).line
+          mock(app).update!
+          dont_allow(push).invoke(:restart, :app => app)
+          subject
+        end
+      end
+    end
+
+    context "when buildpack is given" do
+      let(:old) { nil }
+      let(:app) { fake(:app, :buildpack => old) }
+      let(:inputs) { { :buildpack => new } }
+
+      context "and it's an invalid URL" do
+        let(:new) { "git@github.com:foo/bar.git" }
+
+        before do
+          stub(app).update! do
+            raise CFoundry::MessageParseError.new(
+              "Request invalid due to parse error: Field: buildpack, Error: Value git@github.com:cloudfoundry/heroku-buildpack-ruby.git doesn't match regexp String /GIT_URL_REGEX/",
+              1001)
+          end
+        end
+
+        it "fails and prints a pretty message" do
+          stub(push).line(anything)
+          expect { subject }.to raise_error(
+            CF::UserError, "Buildpack must be a public git repository URI.")
+        end
+      end
+
+      context "and it's a valid URL" do
+        let(:new) { "git://github.com/foo/bar.git" }
+
+        it "updates the app's buildpack" do
+          stub(push).line(anything)
+          mock(app).update!
+          expect { subject }.to change { app.buildpack }.from(old).to(new)
+        end
+
+        it "outputs the changed buildpack with single quotes" do
+          mock(push).line("Changes:")
+          mock(push).line("buildpack: '' -> '#{new}'")
+          stub(app).update!
+          subject
+        end
+
+        include_examples 'common tests for inputs', :buildpack
+      end
+    end
+  end
+
+  describe '#setup_new_app (integration spec!!)' do
+    let(:app) { fake(:app, :guid => nil) }
+    let(:framework) { fake(:framework) }
+    let(:runtime) { fake(:runtime) }
+    let(:host) { "" }
+    let(:domain) { fake(:domain, :name => "example.com") }
+    let(:inputs) do
+      { :name => "some-app",
+        :instances => 2,
+        :framework => framework,
+        :runtime => runtime,
+        :memory => 1024,
+        :host => host,
+        :domain => domain
+      }
+    end
+    let(:global) { {:quiet => true, :color => false, :force => true} }
+
+    before do
+      stub(client).app { app }
+    end
+
+    subject do
+      push.input = Mothership::Inputs.new(Mothership.commands[:push], push, inputs, global, global)
+      push.setup_new_app(path)
+    end
+
+    it 'creates the app' do
+      mock(app).create!
+      mock(app).upload(path)
+      mock(push).filter(:create_app, app) { app }
+      mock(push).filter(:push_app, app) { app }
+      mock(push).invoke :map, :app => app, :host => host, :domain => domain
+      mock(push).invoke :start, :app => app
+      subject
+    end
+  end
+end

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

@@ -0,0 +1,104 @@
+require 'spec_helper'
+require "cf/cli/app/rename"
+
+describe CF::App::Rename do
+  let(:global) { { :color => false, :quiet => true } }
+  let(:inputs) { {} }
+  let(:given) { {} }
+  let(:client) { fake_client }
+  let(:app) {}
+  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, inputs, given, global) }
+
+  describe 'metadata' do
+    let(:command) { Mothership.commands[:rename] }
+
+    describe 'command' do
+      subject { command }
+      its(:description) { should eq "Rename an application" }
+      it { expect(Mothership::Help.group(:apps, :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 => :app },
+          { :type => :optional, :value => nil, :name => :name }
+        ])
+      end
+    end
+  end
+
+  context 'when there are no apps' do
+    context 'and an app is given' do
+      let(:given) { { :app => "some-app" } }
+      it { expect { subject }.to raise_error(CF::UserError, "Unknown app 'some-app'.") }
+    end
+
+    context 'and an app is not given' do
+      it { expect { subject }.to raise_error(CF::UserError, "No applications.") }
+    end
+  end
+
+  context 'when there are apps' do
+    let(:client) { fake_client(:apps => apps) }
+    let(:apps) { fake_list(:app, 2) }
+    let(:renamed_app) { apps.first }
+
+    context 'when the defaults are used' do
+      it 'asks for the app and new name and renames' do
+        mock_ask("Rename which application?", anything) { renamed_app }
+        mock_ask("New name") { new_name }
+        mock(renamed_app).name=(new_name)
+        mock(renamed_app).update!
+        subject
+      end
+    end
+
+    context 'when no name is provided, but a app is' do
+      let(:given) { { :app => renamed_app.name } }
+
+      it 'asks for the new name and renames' do
+        dont_allow_ask("Rename which application?", anything)
+        mock_ask("New name") { new_name }
+        mock(renamed_app).name=(new_name)
+        mock(renamed_app).update!
+        subject
+      end
+    end
+
+    context 'when an app is provided and a name' do
+      let(:inputs) { { :app => renamed_app, :name => new_name } }
+
+      it 'renames the app' do
+        mock(renamed_app).update!
+        subject
+      end
+
+      it 'displays the progress' do
+        mock_with_progress("Renaming to #{new_name}")
+        mock(renamed_app).update!
+
+        subject
+      end
+
+      context 'and the name already exists' do
+        it 'fails' do
+          mock(renamed_app).update! { raise CFoundry::AppNameTaken.new("Bad Name", 404) }
+          expect { subject }.to raise_error(CFoundry::AppNameTaken)
+        end
+      end
+    end
+  end
+end