pbox 1.17.2

data/spec/rhc/commands/env_spec.rb

@@ -0,0 +1,493 @@
+require 'spec_helper'
+require 'rest_spec_helper'
+require 'rhc/commands/env'
+require 'rhc/config'
+
+describe RHC::Commands::Env do
+
+  def exit_with_code_and_message(code, message=nil)
+    expect{ run }.to exit_with_code(code)
+    run_output.should match(message) if message
+  end
+
+  def exit_with_code_and_without_message(code, message=nil)
+    expect{ run }.to exit_with_code(code)
+    run_output.should_not match(message) if message
+  end
+
+  def succeed_with_message(message="done")
+    exit_with_code_and_message(0, message)
+  end
+
+  def succeed_without_message(message="done")
+    exit_with_code_and_without_message(0, message)
+  end
+
+  let!(:rest_client) { MockRestClient.new }
+
+  before(:each) do
+    user_config
+    @rest_domain = rest_client.add_domain("mock_domain_0")
+    @rest_app = @rest_domain.add_application("mock_app_0", "ruby-1.8.7")
+  end
+
+  describe 'env help' do
+    let(:arguments) { ['env', '--help'] }
+    context 'help is run' do
+      it "should display help" do
+        expect { run }.to exit_with_code(0)
+      end
+      it('should output usage') { run_output.should match("Usage: pbox env <action>$") }
+    end
+  end
+
+  describe 'env set --help' do
+    [['env', 'set', '--help'],
+     ['env', 'add', '--help'],
+     ['set-env', '--help'],
+     ['env-set', '--help']
+    ].each_with_index do |args, i|
+      context "help is run run with arguments #{i}" do
+        let(:arguments) { args }
+        it "should display help" do
+          expect { run }.to exit_with_code(0)
+        end
+        it('should output usage') { run_output.should match("Usage: pbox env-set <VARIABLE=VALUE>") }
+      end
+    end
+  end
+
+  describe 'env unset --help' do
+    [['env', 'unset', '--help'],
+     ['env', 'remove', '--help'],
+     ['unset-env', '--help'],
+     ['env-unset', '--help']
+    ].each_with_index do |args, i|
+      context "help is run run with arguments #{i}" do
+        let(:arguments) { args }
+        it "should display help" do
+          expect { run }.to exit_with_code(0)
+        end
+        it('should output usage') { run_output.should match("Usage: pbox env-unset <VARIABLE>") }
+      end
+    end
+  end
+
+  describe 'env list --help' do
+    [['env', 'list', '--help'],
+     ['list-env', '--help'],
+     ['env-list', '--help']
+    ].each_with_index do |args, i|
+      context "help is run run with arguments #{i}" do
+        let(:arguments) { args }
+        it "should display help" do
+          expect { run }.to exit_with_code(0)
+        end
+        it('should output usage') { run_output.should match("Usage: pbox env-list <app> [--namespace NAME]") }
+      end
+    end
+  end
+
+  describe 'env show --help' do
+    [['env', 'show', '--help'],
+     ['show-env', '--help'],
+     ['env-show', '--help']
+    ].each_with_index do |args, i|
+      context "help is run run with arguments #{i}" do
+        let(:arguments) { args }
+        it "should display help" do
+          expect { run }.to exit_with_code(0)
+        end
+        it('should output usage') { run_output.should match("Usage: pbox env-show <VARIABLE>") }
+      end
+    end
+  end
+
+  describe 'set env' do
+
+    [['env', 'set', 'TEST_ENV_VAR=1', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['set-env', 'TEST_ENV_VAR=1', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', '-e', 'TEST_ENV_VAR=1', '--app', 'mock_app_0', '--noprompt', '--confirm' ],
+     ['env', 'set', '--env', 'TEST_ENV_VAR=1', '--app', 'mock_app_0', '--noprompt', '--confirm' ],
+     ['env', 'set', 'FOO_BAR=1', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', 'F1=1', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', 'F_=1', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', '_FOO=1', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', 'FOO=BAR=BAZ', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', 'FOO==', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', 'FOO=Test 1 2 3', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+    ].each_with_index do |args, i|
+      context "when run with single env var #{i}" do
+        let(:arguments) { args }
+        it { succeed_with_message /Setting environment variable\(s\) \.\.\./ }
+        it { succeed_with_message /done/ }
+        it { succeed_without_message /TEST_ENV_VAR=1/ }
+      end
+    end
+
+
+    [['env', 'set', 'TEST_ENV_VAR1=1', 'TEST_ENV_VAR2=2', 'TEST_ENV_VAR3=3', '--app', 'mock_app_0', '--noprompt', '--confirm' ],
+     ['set-env', 'TEST_ENV_VAR1=1', 'TEST_ENV_VAR2=2', 'TEST_ENV_VAR3=3', '--app', 'mock_app_0', '--noprompt', '--confirm' ],
+     ['set-env', '-e', 'TEST_ENV_VAR1=1', '-e', 'TEST_ENV_VAR2=2', '-e', 'TEST_ENV_VAR3=3', '--app', 'mock_app_0', '--noprompt', '--confirm' ],
+     ['set-env', '--env', 'TEST_ENV_VAR1=1', '--env', 'TEST_ENV_VAR2=2', '--env', 'TEST_ENV_VAR3=3', '--app', 'mock_app_0', '--noprompt', '--confirm' ]
+    ].each_with_index do |args, i|
+      context "when run with multiple env vars #{i}" do
+        let(:arguments) { args }
+        it { succeed_with_message /Setting environment variable\(s\) \.\.\./ }
+        it { succeed_with_message /done/ }
+        it { succeed_without_message /TEST_ENV_VAR1=1/ }
+        it { succeed_without_message /TEST_ENV_VAR2=2/ }
+        it { succeed_without_message /TEST_ENV_VAR3=3/ }
+      end
+    end
+
+    [['env', 'set', 'TEST_ENV_VAR', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['set-env', 'TEST_ENV_VAR', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', '-e', 'TEST_ENV_VAR', '--app', 'mock_app_0', '--noprompt', '--confirm' ],
+     ['env', 'set', '--env', 'TEST_ENV_VAR', '--app', 'mock_app_0', '--noprompt', '--confirm' ],
+    ].each_with_index do |args, i|
+      context "when run with no env var provided #{i}" do
+        let(:arguments) { args }
+        it "should raise env var not provided exception" do
+          expect{ run }.to exit_with_code(159)
+          run_output.should match(/Environment variable\(s\) not provided\./)
+          run_output.should match(/Please provide at least one environment variable using the syntax VARIABLE=VALUE\./)
+        end
+      end
+    end
+
+    context 'when run with multiple env vars from file' do
+      let(:arguments) {['env', 'set', File.expand_path('../../assets/env_vars.txt', __FILE__), '--app', 'mock_app_0', '--noprompt', '--confirm' ]}
+        it { succeed_with_message /FOO=123/ }
+        it { succeed_with_message /BAR=456/ }
+        it { succeed_with_message /MY_PROTONBOX_ENV_VAR/ }
+        it { succeed_with_message /MY_EMPTY_ENV_VAR/ }
+        it { succeed_without_message /ZEE/ }
+        it { succeed_without_message /LOL/ }
+        it { succeed_without_message /MUST NOT BE INCLUDED/ }
+    end
+
+    context 'when run with multiple env vars from multiple files' do
+      let(:arguments) {['env', 'set', '-e', File.expand_path('../../assets/env_vars.txt', __FILE__), '-e', File.expand_path('../../assets/env_vars_2.txt', __FILE__), '--app', 'mock_app_0', '--noprompt', '--confirm' ]}
+        it { succeed_with_message /FOO=123/ }
+        it { succeed_with_message /BAR=456/ }
+        it { succeed_with_message /MY_PROTONBOX_ENV_VAR/ }
+        it { succeed_with_message /MY_EMPTY_ENV_VAR/ }
+        it { succeed_with_message /AND=123/ }
+        it { succeed_without_message /ZEE/ }
+        it { succeed_without_message /LOL/ }
+        it { succeed_without_message /MUST NOT BE INCLUDED/ }
+    end
+
+    context 'when run with empty file' do
+      let(:arguments) {['env', 'set', File.expand_path('../../assets/empty.txt', __FILE__), '--app', 'mock_app_0', '--noprompt', '--confirm' ]}
+        it "should raise env var not provided exception" do
+          expect{ run }.to exit_with_code(159)
+          run_output.should match(/Environment variable\(s\) not found in the provided file\(s\)\./)
+          run_output.should match(/Please provide at least one environment variable using the syntax VARIABLE=VALUE\./)
+        end
+    end
+
+    context 'when run with --noprompt and without --confirm' do
+      let(:arguments) { ['env', 'set', 'TEST_ENV_VAR=1', '--app', 'mock_app_0', '--noprompt' ] }
+      it("should not ask for confirmation") { expect{ run }.to exit_with_code(0) }
+      it("should output confirmation") { run_output.should_not match("This action requires the --confirm option") }
+    end
+
+    context 'when run with --noprompt and without --confirm from file' do
+      let(:arguments) { ['env', 'set', File.expand_path('../../assets/env_vars.txt', __FILE__), '--app', 'mock_app_0', '--noprompt' ] }
+      it "should ask for confirmation" do
+        expect{ run }.to exit_with_code(1)
+      end
+      it("should output confirmation") { run_output.should match("This action requires the --confirm option") }
+    end
+
+    context 'when run against an unsupported server' do
+      before {
+        @rest_app.links.delete 'SET_UNSET_ENVIRONMENT_VARIABLES'
+        @rest_app.links.delete 'LIST_ENVIRONMENT_VARIABLES'
+      }
+      let(:arguments) { ['env', 'set', 'TEST_ENV_VAR=1', '--app', 'mock_app_0', '--noprompt', '--confirm' ] }
+      it "should raise env var not found exception" do
+        expect{ run }.to exit_with_code(158)
+        run_output.should match(/Server does not support environment variables/)
+      end
+    end
+
+    [['env', 'set', 'FOO.Z=BAR', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', '2FOO=BAR', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', 'FO$O=BAR', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', 'FO%O=BAR', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', 'FO*O=BAR', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', 'FOO.=BAR', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', '=BAR', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', '==', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+    ].each_with_index do |args, i|
+      context "when run with invalid env var names #{i}" do
+        let(:arguments) { args }
+        it "should raise env var not provided exception" do
+          expect{ run }.to exit_with_code(159)
+          run_output.should match(/Environment variable\(s\) not provided\./)
+          run_output.should match(/Please provide at least one environment variable using the syntax VARIABLE=VALUE\./)
+        end
+      end
+    end
+
+  end
+
+  describe 'unset env' do
+
+    [['env', 'unset', 'TEST_ENV_VAR', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['unset-env', 'TEST_ENV_VAR', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'unset', '-e', 'TEST_ENV_VAR', '--app', 'mock_app_0', '--noprompt', '--confirm' ],
+     ['env', 'unset', '--env', 'TEST_ENV_VAR', '--app', 'mock_app_0', '--noprompt', '--confirm' ]
+    ].each_with_index do |args, i|
+      context "when run with single env var #{i}" do
+        let(:arguments) { args }
+        it { succeed_with_message /TEST_ENV_VAR/ }
+        it { succeed_with_message /Removing environment variable\(s\) \.\.\./ }
+        it { succeed_with_message /removed/ }
+      end
+    end
+
+    [['env', 'unset', 'TEST_ENV_VAR1', 'TEST_ENV_VAR2', 'TEST_ENV_VAR3', '--app', 'mock_app_0', '--noprompt', '--confirm' ],
+     ['unset-env', 'TEST_ENV_VAR1', 'TEST_ENV_VAR2', 'TEST_ENV_VAR3', '--app', 'mock_app_0', '--noprompt', '--confirm' ]
+    ].each_with_index do |args, i|
+      context "when run with multiple env vars #{i}" do
+        let(:arguments) { args }
+        it { succeed_with_message /TEST_ENV_VAR1/ }
+        it { succeed_with_message /TEST_ENV_VAR2/ }
+        it { succeed_with_message /TEST_ENV_VAR3/ }
+        it { succeed_with_message /Removing environment variable\(s\) \.\.\./ }
+        it { succeed_with_message /removed/ }
+      end
+    end
+
+    context 'when run with --noprompt and without --confirm' do
+      let(:arguments) { ['env', 'unset', 'TEST_ENV_VAR', '--app', 'mock_app_0', '--noprompt' ] }
+      it "should ask for confirmation" do
+        expect{ run }.to exit_with_code(1)
+      end
+      it("should output confirmation") { run_output.should match("This action requires the --confirm option") }
+    end
+  end
+
+  describe 'list env' do
+    context 'when list with default format' do
+      before(:each) do
+        @rest_app.set_environment_variables(
+          [RHC::Rest::EnvironmentVariable.new({:name => 'FOO', :value => '123'}),
+           RHC::Rest::EnvironmentVariable.new({:name => 'BAR', :value => '456'})])
+      end
+      let(:arguments) { ['env', 'list', '--app', 'mock_app_0'] }
+      it { succeed_with_message /FOO=123/ }
+      it { succeed_with_message /BAR=456/ }
+      it "should contain the environment variables" do
+        @rest_app.environment_variables.length.should == 2
+      end
+    end
+
+    context 'when list with default format and empty env vars' do
+      let(:arguments) { ['env', 'list', '--app', 'mock_app_0'] }
+      it "should exit with no message" do
+        expect{ run }.to exit_with_code(0)
+      end
+    end
+
+    context 'when list with quotes format' do
+      before(:each) do
+        @rest_app.set_environment_variables(
+          [RHC::Rest::EnvironmentVariable.new({:name => 'FOO', :value => '123'}),
+           RHC::Rest::EnvironmentVariable.new({:name => 'BAR', :value => '456'})])
+      end
+      let(:arguments) { ['env', 'list', '--app', 'mock_app_0', '--quotes'] }
+      it { succeed_with_message /FOO="123"/ }
+      it { succeed_with_message /BAR="456"/ }
+      it "should contain the environment variables" do
+        @rest_app.environment_variables.length.should == 2
+      end
+    end
+
+    context 'when list with quotes format and empty env vars' do
+      let(:arguments) { ['env', 'list', '--app', 'mock_app_0', '--quotes'] }
+      it "should exit with no message" do
+        expect{ run }.to exit_with_code(0)
+      end
+    end
+
+    context 'when list with table format' do
+      before(:each) do
+        @rest_app.set_environment_variables(
+          [RHC::Rest::EnvironmentVariable.new({:name => 'FOO', :value => '123'}),
+           RHC::Rest::EnvironmentVariable.new({:name => 'BAR', :value => '456'})])
+      end
+      let(:arguments) { ['env', 'list', '--app', 'mock_app_0', '--table'] }
+      it { succeed_with_message /Name\s+Value/ }
+      it { succeed_with_message /FOO\s+123/ }
+      it { succeed_with_message /BAR\s+456/ }
+      it "should contain the right number of env vars" do
+        @rest_app.environment_variables.length.should == 2
+      end
+    end
+
+    context 'when list with table and quotes format' do
+      before(:each) do
+        @rest_app.set_environment_variables(
+          [RHC::Rest::EnvironmentVariable.new({:name => 'FOO', :value => '123'}),
+           RHC::Rest::EnvironmentVariable.new({:name => 'BAR', :value => '456'})])
+      end
+      let(:arguments) { ['env', 'list', '--app', 'mock_app_0', '--table', '--quotes'] }
+      it { succeed_with_message /Name\s+Value/ }
+      it { succeed_with_message /FOO\s+"123"/ }
+      it { succeed_with_message /BAR\s+"456"/ }
+      it "should contain the right number of env vars" do
+        @rest_app.environment_variables.length.should == 2
+      end
+    end
+
+    context 'when list with table format and empty env vars' do
+      let(:arguments) { ['env', 'list', '--app', 'mock_app_0', '--table'] }
+      it "should exit with no message" do
+        expect{ run }.to exit_with_code(0)
+      end
+    end
+  end
+
+  describe 'show env' do
+    context 'when show with default format' do
+      before(:each) do
+        @rest_app.set_environment_variables(
+          [RHC::Rest::EnvironmentVariable.new({:name => 'FOO', :value => '123'}),
+           RHC::Rest::EnvironmentVariable.new({:name => 'BAR', :value => '456'})])
+      end
+      let(:arguments) { ['env', 'show', 'FOO', '--app', 'mock_app_0'] }
+      it { succeed_with_message /FOO=123/ }
+      it "should not contain env vars not specified to show" do
+        run_output.should_not match(/BAR=456/)
+      end
+      it "should contain the right number of env vars" do
+        @rest_app.environment_variables.length.should == 2
+      end
+    end
+
+    context 'when show with default format and not found env var' do
+      let(:arguments) { ['env', 'show', 'FOO', '--app', 'mock_app_0'] }
+      it "should raise env var not found exception" do
+        expect{ run }.to exit_with_code(157)
+        run_output.should match(/Environment variable\(s\) FOO can't be found in application mock_app_0/)
+      end
+    end
+
+    context 'when show with default format and not found env var' do
+      before(:each) do
+        @rest_app.set_environment_variables(
+          [RHC::Rest::EnvironmentVariable.new({:name => 'FOO', :value => '123'}),
+           RHC::Rest::EnvironmentVariable.new({:name => 'BAR', :value => '456'})])
+      end
+      let(:arguments) { ['env', 'show', 'ZEE', '--app', 'mock_app_0'] }
+      it "should contain the right number of env vars" do
+        @rest_app.environment_variables.length.should == 2
+      end
+      it "should not contain env vars not specified to show" do
+        run_output.should_not match(/FOO=123/)
+        run_output.should_not match(/BAR=456/)
+      end
+      it "should raise env var not found exception" do
+        expect{ run }.to exit_with_code(157)
+        run_output.should match(/Environment variable\(s\) ZEE can't be found in application mock_app_0/)
+      end
+    end
+
+    context 'when show with quotes format' do
+      before(:each) do
+        @rest_app.set_environment_variables(
+          [RHC::Rest::EnvironmentVariable.new({:name => 'FOO', :value => '123'}),
+           RHC::Rest::EnvironmentVariable.new({:name => 'BAR', :value => '456'})])
+      end
+      let(:arguments) { ['env', 'show', 'FOO', '--app', 'mock_app_0', '--quotes'] }
+      it { succeed_with_message /FOO="123"/ }
+      it "should not contain env vars not specified to show" do
+        run_output.should_not match(/BAR="456"/)
+      end
+      it "should contain the right number of env vars" do
+        @rest_app.environment_variables.length.should == 2
+      end
+    end
+
+    context 'when show with quotes format and not found env var' do
+      before(:each) do
+        @rest_app.set_environment_variables(
+          [RHC::Rest::EnvironmentVariable.new({:name => 'FOO', :value => '123'}),
+           RHC::Rest::EnvironmentVariable.new({:name => 'BAR', :value => '456'})])
+      end
+      let(:arguments) { ['env', 'show', 'ZEE', '--app', 'mock_app_0', '--quotes'] }
+      it "should contain the right number of env vars" do
+        @rest_app.environment_variables.length.should == 2
+      end
+      it "should not contain env vars not specified to show" do
+        run_output.should_not match(/FOO=123/)
+        run_output.should_not match(/BAR=456/)
+      end
+      it "should raise env var not found exception" do
+        expect{ run }.to exit_with_code(157)
+        run_output.should match(/Environment variable\(s\) ZEE can't be found in application mock_app_0/)
+      end
+    end
+
+    context 'when show with table format' do
+      before(:each) do
+        @rest_app.set_environment_variables(
+          [RHC::Rest::EnvironmentVariable.new({:name => 'FOO', :value => '123'}),
+           RHC::Rest::EnvironmentVariable.new({:name => 'BAR', :value => '456'})])
+      end
+      let(:arguments) { ['env', 'show', 'FOO', '--app', 'mock_app_0', '--table'] }
+      it { succeed_with_message /Name\s+Value/ }
+      it { succeed_with_message /FOO\s+123/ }
+      it "should not contain env vars not specified to show" do
+        run_output.should_not match(/BAR/)
+      end
+      it "should contain the right number of env vars" do
+        @rest_app.environment_variables.length.should == 2
+      end
+    end
+
+    context 'when show with table and quotes format' do
+      before(:each) do
+        @rest_app.set_environment_variables(
+          [RHC::Rest::EnvironmentVariable.new({:name => 'FOO', :value => '123'}),
+           RHC::Rest::EnvironmentVariable.new({:name => 'BAR', :value => '456'})])
+      end
+      let(:arguments) { ['env', 'show', 'FOO', '--app', 'mock_app_0', '--table', '--quotes'] }
+      it { succeed_with_message /Name\s+Value/ }
+      it { succeed_with_message /FOO\s+"123"/ }
+      it "should not contain env vars not specified to show" do
+        run_output.should_not match(/BAR/)
+      end
+      it "should contain the right number of env vars" do
+        @rest_app.environment_variables.length.should == 2
+      end
+    end
+
+    context 'when show with table format and not found env var' do
+      before(:each) do
+        @rest_app.set_environment_variables(
+          [RHC::Rest::EnvironmentVariable.new({:name => 'FOO', :value => '123'}),
+           RHC::Rest::EnvironmentVariable.new({:name => 'BAR', :value => '456'})])
+      end
+      let(:arguments) { ['env', 'show', 'ZEE', '--app', 'mock_app_0', '--table'] }
+      it "should contain the right number of env vars" do
+        @rest_app.environment_variables.length.should == 2
+      end
+      it "should not contain env vars not specified to show" do
+        run_output.should_not match(/FOO/)
+        run_output.should_not match(/BAR/)
+      end
+      it "should raise env var not found exception" do
+        expect{ run }.to exit_with_code(157)
+        run_output.should match(/Environment variable\(s\) ZEE can't be found in application mock_app_0/)
+      end
+    end
+
+  end
+
+end

data/spec/rhc/commands/git_clone_spec.rb

@@ -0,0 +1,80 @@
+require 'spec_helper'
+require 'rest_spec_helper'
+require 'rhc/commands/git_clone'
+require 'fileutils'
+
+describe RHC::Commands::GitClone do
+  before(:each) do
+    FakeFS.activate!
+    FakeFS::FileSystem.clear
+    user_config
+    @instance = RHC::Commands::GitClone.new
+    RHC::Commands::GitClone.stub(:new) do
+      @instance.stub(:git_config_get) { "" }
+      @instance.stub(:git_config_set) { "" }
+      Kernel.stub(:sleep) { }
+      @instance.stub(:host_exists?) do |host|
+        host.match("dnserror") ? false : true
+      end
+      @instance
+    end
+  end
+  let!(:rest_client){ MockRestClient.new }
+  before(:each) do
+    @domain = rest_client.add_domain("mockdomain")
+    @app = @domain.add_application("app1", "mock_unique_standalone_cart")
+  end
+
+  after(:each) do
+    FakeFS.deactivate!
+  end
+
+  describe 'git-clone' do
+    let(:arguments) { ['app', 'git-clone', 'app1'] }
+
+    context "stubbing git_clone_repo" do
+      context "reports success successfully" do
+        before do
+          @instance.stub(:git_clone_repo) do |git_url, repo_dir|
+            Dir::mkdir(repo_dir)
+            say "Cloned"
+            true
+          end
+        end
+
+        it { expect { run }.to exit_with_code(0) }
+        it { run_output.should match("Cloned") }
+      end
+
+      context "testing git_clone_deploy_hooks" do
+        before do
+          @instance.stub(:git_clone_repo) do |git_url, repo_dir|
+            FileUtils.mkdir_p "#{repo_dir}/.git/hooks"
+            FileUtils.mkdir_p "#{repo_dir}/.protonbox/git_hooks"
+            FileUtils.touch "#{repo_dir}/.protonbox/git_hooks/pre_commit"
+            @instance.git_clone_deploy_hooks(repo_dir)
+            say "Copied" if File.exists?("#{repo_dir}/.git/hooks/pre_commit")
+            true
+          end
+
+          # Get around the FakeFS bug (defunkt/fakefs#177) by
+          # stubbing the #cp call to inject a expected fs entry
+          FileUtils.stub(:cp) do |hook, dir|
+            FakeFS::FileSystem.add(
+              File.join(dir, File.basename(hook)),
+              FakeFS::FileSystem.find(hook))
+          end
+        end
+        it { expect { run }.to exit_with_code(0) }
+        it { run_output.should match("Copied") }
+      end
+
+      context "reports failure" do
+        before{ @instance.stub(:git_clone_repo).and_raise(RHC::GitException) }
+
+        it { expect { run }.to exit_with_code(216) }
+        it { run_output.should match("Git returned an error") }
+      end
+    end
+  end
+end

data/spec/rhc/commands/logout_spec.rb

@@ -0,0 +1,86 @@
+require 'spec_helper'
+require 'rest_spec_helper'
+require 'rhc'
+require 'rhc/commands/logout'
+
+describe RHC::Commands::Logout do
+
+  describe '#run' do
+    let(:arguments) { ['logout'] }
+    let(:username) { 'foo' }
+    let(:password) { nil }
+    let(:supports_auth) { false }
+    let(:server) { mock_uri }
+    let!(:token_store) { RHC::Auth::TokenStore.new(Dir.mktmpdir) }
+    before{ user_config }
+    before do
+      stub_api(false, supports_auth)
+      challenge{ stub_user }
+      RHC::Auth::TokenStore.stub(:new).and_return(token_store)
+    end
+
+    context "when calling from the alias" do
+      let(:arguments){ ['account', 'logout', '-h'] }
+      it("should print usage"){ run_output.should match "Usage: pbox logout" }
+    end
+
+    it("should clear the token cache"){ expect{ run }.to call(:clear).on(token_store) }
+    it("should exit with success"){ expect{ run }.to exit_with_code(0) }
+    it("should display a message"){ run_output.should match("All local sessions removed.") }
+
+    context "when --all is requested" do
+      let(:arguments) { ['account', 'logout', '--all'] }
+
+      context "if the server does not implement authorizations" do
+        it("should display a message"){ run_output.should match(/Deleting all authorizations associated with your account.*not supported/) }
+        it("should exit with success"){ expect{ run }.to exit_with_code(0) }
+      end
+
+      context "if the server implements authorizations" do
+        let(:supports_auth) { true }
+        before{ stub_delete_authorizations }
+
+        it("should display a message"){ run_output.should match(/Deleting all authorizations associated with your account.*done/) }
+        it("should exit with success"){ expect{ run }.to exit_with_code(0) }
+      end
+    end
+
+    context "when --token is provided" do
+      let(:arguments) { ['account', 'logout', '--token', 'foo'] }
+      def user_auth; { :token => 'foo' }; end
+
+      context "if the server does not implement authorizations" do
+        it("should display a message"){ run_output.should match(/Ending session on server.*not supported/) }
+        it("should exit with success"){ expect{ run }.to exit_with_code(0) }
+      end
+
+      context "if the server implements authorizations" do
+        let(:supports_auth) { true }
+
+        context "if the server returns successfully" do
+          before{ stub_delete_authorization('foo') }
+
+          it("should display a message"){ run_output.should match(/Ending session on server.*deleted/) }
+          it("should exit with success"){ expect{ run }.to exit_with_code(0) }
+          it("should clear the token cache"){ expect{ run }.to call(:clear).on(token_store) }
+        end
+
+        context "if the server rejects the token" do
+          before{ stub_request(:delete, mock_href('broker/rest/user/authorizations/foo', false)).to_return(:status => 401, :body => {}.to_json) }
+
+          it("should display a message"){ run_output.should match(/Ending session on server.*already closed/) }
+          it("should exit with success"){ expect{ run }.to exit_with_code(0) }
+          it("should clear the token cache"){ expect{ run }.to call(:clear).on(token_store) }
+        end
+
+        context "if the server returns an unexpected error" do
+          before{ stub_request(:delete, mock_href('broker/rest/user/authorizations/foo', false)).to_return(:status => 500, :body => {}.to_json) }
+
+          it("should display a message"){ run_output.should match(/Ending session on server.*The server did not respond/) }
+          it("should exit with success"){ expect{ run }.to exit_with_code(0) }
+          it("should clear the token cache"){ expect{ run }.to call(:clear).on(token_store) }
+        end
+      end
+    end
+  end
+end