rhc 1.9.6 → 1.10.7

data/spec/rhc/commands/alias_spec.rb

@@ -295,4 +295,32 @@ describe RHC::Commands::Alias do
     end
   end
 
+  describe 'aliases' do
+    before do 
+      rest_client.stub(:api_version_negotiated).and_return(1.4)
+    end
+    context 'app with existing certificate' do
+      let(:arguments) { ['aliases', 'mock_app_0'] }
+      it { expect { run }.to exit_with_code(0) }
+      it { run_output.should =~ /Has Certificate?/m }
+      it { run_output.should =~ /Certificate Added/m }
+      it { run_output.should =~ /www.foo.bar/m }
+    end
+    context 'app without certificates' do
+      let(:arguments) { ['aliases', 'mock_app_1'] }
+      it { expect { run }.to exit_with_code(0) }
+      it { run_output.should =~ /No aliases associated with the application mock_app_1/m }
+    end
+    context 'simple list is server does not support ssl certs' do
+      let(:arguments) { ['aliases', 'mock_app_0'] }
+      before do 
+        rest_client.stub(:api_version_negotiated).and_return(1.3)
+      end
+      it { expect { run }.to exit_with_code(0) }
+      it { run_output.should =~ /no/m }
+      it { run_output.should =~ /-/m }
+      it { run_output.should =~ /www.foo.bar/m }
+    end
+  end
+
 end

data/spec/rhc/commands/app_spec.rb

@@ -19,8 +19,10 @@ describe RHC::Commands::App do
       @instance.stub(:git_config_set) { "" }
       Kernel.stub(:sleep) { }
       @instance.stub(:git_clone_repo) do |git_url, repo_dir|
+        $terminal.instance_variable_get(:@output).puts "Cloning into..."
         raise RHC::GitException, "Error in git clone" if repo_dir == "giterrorapp"
         Dir::mkdir(repo_dir)
+        File.expand_path(repo_dir)
       end
       @instance.stub(:host_exists?) do |host|
         host.match("dnserror") ? false : true
@@ -41,6 +43,8 @@ describe RHC::Commands::App do
     context 'app' do
       let(:arguments) { ['app'] }
       it { run_output.should match('Usage:') }
+      it { run_output.should match('List of Actions') }
+      it { run_output.should_not match('Options') }
     end
   end
 
@@ -159,7 +163,7 @@ describe RHC::Commands::App do
       before{ rest_client.domains.clear }
       let(:arguments) { ['app', 'create', 'app1', 'mock_standalone_cart-1'] }
       # skips login stage and insecure check because of mock rest client, doesn't check keys
-      it { run_output(['mydomain', 'y', 'mykey']).should match(/This wizard.*Checking your namespace.*Your domain name 'mydomain' has been successfully created.*Creating application.*Your public SSH key.*Uploading key 'mykey' .*Downloading the application.*Success/m) }
+      it { run_output(['mydomain', 'y', 'mykey']).should match(/This wizard.*Checking your namespace.*Your domain name 'mydomain' has been successfully created.*Creating application.*Your public SSH key.*Uploading key 'mykey'.*Your application 'app1' is now available.*Cloned to/m) }
     end
 
     context 'when run without a cart' do
@@ -207,8 +211,8 @@ describe RHC::Commands::App do
       let(:arguments) { ['app', 'create', 'app1', 'mock_standalone_cart-1', '--from', 'git://url', '--noprompt', '-p',  'password'] }
       it { expect { run }.to exit_with_code(0) }
       it { run_output.should match("Success") }
+      it { run_output.should match("Git remote: git:fake.foo/git/app1.git\n") }
       it { run_output.should match("Source Code: git://url\n") }
-      it { run_output.should match("Initial Git URL: git://url\n") }
       after{ rest_client.domains.first.applications.first.initial_git_url.should == 'git://url' }
     end
 
@@ -229,9 +233,7 @@ describe RHC::Commands::App do
   end
 
   describe 'cart matching behavior' do
-    before(:each) do
-      domain = rest_client.add_domain("mockdomain")
-    end
+    before{ rest_client.add_domain("mockdomain") }
 
     context 'multiple web matches' do
       let(:arguments) { ['app', 'create', 'app1', 'mock_standalone_cart', '--trace', '--noprompt'] }
@@ -246,8 +248,8 @@ describe RHC::Commands::App do
       it('picks the non web cart') { run_output.should match('Using unique_mock_cart-1') }
     end
     context 'when I pick very ambiguous carts' do
-      let(:arguments) { ['app', 'create', 'app1', 'mock', '--noprompt'] }
-      it('shows only web carts') { run_output.should_not match('unique_mock_cart-1') }
+      let(:arguments) { ['app', 'create', 'app1', 'mock', 'embcart-', '--noprompt'] }
+      it('shows only web carts') { run_output.should match("There are multiple cartridges matching 'mock'") }
     end
     context 'when I pick only embedded carts' do
       let(:arguments) { ['app', 'create', 'app1', 'mock_cart', '--trace', '--noprompt'] }
@@ -259,7 +261,7 @@ describe RHC::Commands::App do
     end
     context 'when I pick multiple standalone carts' do
       let(:arguments) { ['app', 'create', 'app1', 'unique_standalone', 'mock_standalone_cart', '--trace', '--noprompt'] }
-      it { expect { run }.to raise_error(RHC::MultipleCartridgesException, /You must select only a single web cart/) }
+      it { expect { run }.to raise_error(RHC::MultipleCartridgesException, /There are multiple cartridges matching 'mock_standalone_cart'/) }
     end
     context 'when I pick a custom URL cart' do
       let(:arguments) { ['app', 'create', 'app1', 'http://foo.com', '--trace', '--noprompt'] }
@@ -267,9 +269,9 @@ describe RHC::Commands::App do
       it('lists the cart using the short_name') { run_output.should match(%r(Cartridges:\s+http://foo.com$)) }
     end    
     context 'when I pick a custom URL cart and a web cart' do
-      let(:arguments) { ['app', 'create', 'app1', 'http://foo.com', 'unique_standalone', '--trace', '--noprompt'] }
+      let(:arguments) { ['app', 'create', 'app1', 'http://foo.com', 'embcart-1', '--trace', '--noprompt'] }
       it('tells me about custom carts') { run_output.should match("The cartridge 'http://foo.com' will be downloaded") }
-      it('lists the carts using the short_name') { run_output.should match(%r(Cartridges:\s+http://foo.com, mock_unique_standalone_cart-1$)) }
+      it('lists the carts using the short_name') { run_output.should match(%r(Cartridges:\s+http://foo.com, embcart-1$)) }
     end    
   end
 
@@ -494,6 +496,37 @@ describe RHC::Commands::App do
         end
       end
     end
+
+    context "against a 1.5 server" do
+      let!(:rest_client){ nil }
+      let(:username){ mock_user }
+      let(:password){ 'password' }
+      let(:server){ mock_uri }
+      let(:arguments){ ['delete-app', 'foo', '--confirm', '--trace'] }
+      before do 
+        stub_api(true)
+        stub_one_domain('test')
+        stub_one_application('test', 'foo')
+      end
+      before do 
+        stub_api_request(:delete, "broker/rest/domains/test/applications/foo").
+          to_return({
+            :body   => {
+              :type => nil,
+              :data => nil,
+              :messages => [
+                {:exit_code => 0, :field => nil, :severity => 'info', :text => 'Removed foo'},
+                {:exit_code => 0, :field => nil, :severity => 'result', :text => 'Job URL changed'},
+              ]
+            }.to_json,
+            :status => 200
+          })
+      end
+
+      it("should display info returned by the server"){ run_output.should match "Removed foo" }
+      it("should display results returned by the server"){ run_output.should match "Job URL changed" }
+      it('should exit successfully'){ expect{ run }.to exit_with_code(0) }
+    end
   end
 
   describe 'app show' do
@@ -564,66 +597,52 @@ describe RHC::Commands::App do
   end
 
   describe 'app show --gears' do
-    let(:arguments) { ['app', 'show', 'app1', '--gears', '--noprompt'] }
+    let(:arguments) { ['app', 'show', 'app1', '--gears'] }
 
     context 'when run' do
       before(:each) do
         @domain = rest_client.add_domain("mockdomain")
         @domain.add_application("app1", "mock_type")
       end
-      it { run_output.should match("fakegearid started fake_geargroup_cart-0.1 small fakegearid@fakesshurl.com") }
+      it { run_output.should match("fakegearid0 started mock_type  small fakegearid0@fakesshurl.com") }
+      it { expect{ run }.to exit_with_code(0) }
     end
   end
 
-  describe 'app ssh' do
-    let(:arguments) { ['app', 'ssh', 'app1'] }
+  describe 'app show --gears quota' do
+    let(:arguments) { ['app', 'show', 'app1', '--gears', 'quota'] }
 
     context 'when run' do
-      before(:each) do
+      before do
         @domain = rest_client.add_domain("mockdomain")
-        @domain.add_application("app1", "mock_type")
-        Kernel.should_receive(:system).with("ssh fakeuuidfortestsapp1@127.0.0.1").and_return(0)
+        @domain.add_application("app1", "mock_type", true)
+        expect_multi_ssh('echo "$(du -s 2>/dev/null | cut -f 1)"', 'fakegearid0@fakesshurl.com' => '1734934', 'fakegearid1@fakesshurl.com' => '1934234')
       end
-      it { run_output.should match("Connecting to fakeuuidfortestsapp") }
-      it { expect { run }.to exit_with_code(0) }
+      it { run_output.should match(/Gear.*Cartridges.*Used.*fakegearid0.*1\.7 MB.*1 GB.*fakegearid1.*1\.9 MB/m) }
+      it { expect{ run }.to exit_with_code(0) }
     end
   end
 
-  describe 'app ssh no system ssh' do
-    let(:arguments) { ['app', 'ssh', 'app1'] }
+  describe 'app show --gears ssh' do
+    let(:arguments) { ['app', 'show', 'app1', '--gears', 'ssh'] }
 
     context 'when run' do
-      before(:each) do
+      before do
         @domain = rest_client.add_domain("mockdomain")
-        @domain.add_application("app1", "mock_type")
-        @instance.should_receive(:has_ssh?).and_return(false)
+        @domain.add_application("app1", "mock_type", true)
       end
-      it { run_output.should match("Please use the --ssh option to specify the path to your SSH executable, or install SSH.") }
-      it { expect { run }.to exit_with_code(1) }
+      it { run_output.should == "fakegearid0@fakesshurl.com\nfakegearid1@fakesshurl.com\n\n" }
+      it { expect{ run }.to exit_with_code(0) }
     end
   end
 
-  describe 'app ssh can use system exec' do
-    let(:arguments) { ['app', 'ssh', 'app1', '--ssh', 'path_to_ssh'] }
+  describe 'app show --gears badcommand' do
+    let(:arguments) { ['app', 'show', 'app1', '--gears', 'badcommand'] }
 
     context 'when run' do
-      before(:each) do
-        @domain = rest_client.add_domain("mockdomain")
-        @domain.add_application("app1", "mock_type")
-        @instance.should_not_receive(:has_ssh?)
-        Kernel.should_receive(:system).with("path_to_ssh fakeuuidfortestsapp1@127.0.0.1").and_return(1)
-      end
-      it { run_output.should match("Connecting to fakeuuidfortestsapp") }
-      it { expect { run }.to exit_with_code(1) }
-    end
-  end
-
-  describe 'ssh tests' do
-    let(:arguments) { ['app', 'ssh', 'app1', '-s /bin/blah'] }
-
-    context 'has_ssh?' do
-      before{ @instance.stub(:ssh_version){ raise "Fake Exception" } }
-      its(:has_ssh?) { should be_false }
+      before{ rest_client.add_domain("mockdomain").add_application("app1", "mock_type", true) }
+      it { run_output.should match(/The operation badcommand is not supported/m) }
+      it { expect{ run }.to exit_with_code(1) }
     end
   end
 

data/spec/rhc/commands/authorization_spec.rb

@@ -52,6 +52,22 @@ describe RHC::Commands::Authorization do
     end
   end
 
+  describe '#list' do
+    let(:arguments) { ['authorization', 'list'] }
+    context "with authorizations" do
+      with_authorization
+      before{ stub_authorizations }
+      it('should display the note')           { run_output.should =~ /an_authorization/ }
+      it('should display the token')          { run_output.should =~ /Token:\s+a_token_value/ }
+      it('should display the expiration')     { run_output.should =~ /Expires In:\s+1 minute/ }
+      it('should display the creation date')  { run_output.should =~ /Created:\s+#{RHC::Helpers.date('2013-02-21T01:00:01Z')}/ }
+      it('should display the scopes')         { run_output.should =~ /Scopes:\s+session read/ }
+      it{ expect{ run }.to exit_with_code(0) }
+    end
+
+    expect_an_unsupported_message
+  end
+
   describe "#delete" do
     let(:arguments) { ['authorization', 'delete', 'foo', 'bar'] }
 

data/spec/rhc/commands/cartridge_spec.rb

@@ -59,6 +59,7 @@ describe RHC::Commands::Cartridge do
         it{ run_output.should match "Mock2 description\n\n" }
         it{ run_output.should match "Tagged with: scheduled" }
         it{ run_output.should_not match("Tagged with: cartridge") }
+        it{ run_output.should match /Premium Cart.*\[premium_cart\-1\*\] \(web\)/ }
       end
     end
   end
@@ -208,6 +209,38 @@ describe RHC::Commands::Cartridge do
         expect { run }.to raise_error RHC::CartridgeNotFoundException
       end
     end
+
+    context "against a 1.5 server" do
+      let!(:rest_client){ nil }
+      let(:username){ mock_user }
+      let(:password){ 'password' }
+      let(:server){ mock_uri }
+      let(:arguments){ ['remove-cartridge', 'jenkins-1.4', '-a', 'foo', '--confirm', '--trace'] }
+      before do 
+        stub_api(true)
+        stub_one_domain('test')
+        stub_one_application('test', 'foo').with(:query => {:include => 'cartridges'})
+        stub_application_cartridges('test', 'foo', [{:name => 'php-5.3'}, {:name => 'jenkins-1.4'}])
+      end
+      before do 
+        stub_api_request(:delete, "broker/rest/domains/test/applications/foo/cartridges/jenkins-1.4").
+          to_return({
+            :body   => {
+              :type => nil,
+              :data => nil,
+              :messages => [
+                {:exit_code => 0, :field => nil, :severity => 'info', :text => 'Removed Jenkins'},
+                {:exit_code => 0, :field => nil, :severity => 'result', :text => 'Job URL changed'},
+              ]
+            }.to_json,
+            :status => 200
+          })
+      end
+
+      it("should display info returned by the server"){ run_output.should match "Removed Jenkins" }
+      it("should display results returned by the server"){ run_output.should match "Job URL changed" }
+      it('should exit successfully'){ expect{ run }.to exit_with_code(0) }
+    end
   end
 
   describe 'cartridge status' do
@@ -353,10 +386,10 @@ describe RHC::Commands::Cartridge do
     let(:arguments) { ['cartridge', 'scale', @cart_type || 'mock_type', '-a', 'app1', '--noprompt', '--config', 'test.conf', '-l', 'test@test.foo', '-p',  'password'] | (@extra_args || []) }
 
     let(:current_scale) { 1 }
-    before(:each) do
+    before do
       domain = rest_client.add_domain("mock_domain")
-      app = domain.add_application("app1", "mock_type", scalable)
-      app.cartridges.first.stub(:current_scale).and_return(current_scale)
+      @app = domain.add_application("app1", "mock_type", scalable)
+      @app.cartridges.first.stub(:current_scale).and_return(current_scale)
     end
 
     context 'when run with scalable app' do
@@ -371,6 +404,21 @@ describe RHC::Commands::Cartridge do
         succeed_with_message "minimum: 6"
       end
 
+      it "with an explicit value" do
+        @extra_args = ["6"]
+        succeed_with_message "minimum: 6, maximum: 6"
+      end
+
+      it "with an invalid explicit value" do
+        @extra_args = ["a6"]
+        fail_with_message "Multiplier must be a positive integer"
+      end
+
+      it "with an explicit value and a different minimum" do
+        @extra_args = ["6", "--min", "5"]
+        succeed_with_message "minimum: 5, maximum: 6"
+      end
+
       it "with a max value" do
         @extra_args = ["--max","3"]
         succeed_with_message 'maximum: 3'
@@ -385,6 +433,14 @@ describe RHC::Commands::Cartridge do
         @extra_args = ["--max","a"]
         fail_with_message "invalid argument: --max"
       end
+
+      context "when the operation times out" do
+        before{ @app.cartridges.first.should_receive(:set_scales).twice.and_raise(RHC::Rest::TimeoutException.new('Timeout', HTTPClient::ReceiveTimeoutError.new)) }
+        it "displays an error" do
+          @extra_args = ["--min","2"]
+          fail_with_message "The server has closed the connection, but your scaling operation is still in progress"
+        end
+      end
     end
 
     context 'when run with a nonscalable app' do

data/spec/rhc/commands/port_forward_spec.rb

@@ -28,7 +28,7 @@ describe RHC::Commands::PortForward do
       it "should error out and suggest restarting the application" do
         expect { run }.to exit_with_code(1)
       end
-      it { run_output.should match(/Application \S+ is stopped\..*restart/m) }
+      it { run_output.should match(/none.*The application is stopped\..*restart/m) }
     end
 
     context 'when port forwarding an app without ports to forward' do
@@ -174,11 +174,11 @@ describe RHC::Commands::PortForward do
 
     context 'when port forwarding a single gear on a scaled app' do
       let(:gear_host) { 'fakesshurl.com' }
-      let(:gear_user) { 'fakegearid' }
+      let(:gear_user) { 'fakegearid0' }
       let(:arguments) { ['port-forward', '--noprompt', '--config', 'test.conf', '-l', 'test@test.foo', '-p', 'password', '--app', 'mockapp', '--gear', @gear_id] }
 
       it 'should run successfully' do
-        @gear_id = 'fakegearid'
+        @gear_id = 'fakegearid0'
         Net::SSH.should_receive(:start).with(gear_host, gear_user).and_yield(@ssh).twice
 
         @ssh.should_receive(:exec!).with("rhc-list-ports --exclude-remote").
@@ -199,15 +199,15 @@ describe RHC::Commands::PortForward do
       end
 
       it 'should fail if the specified gear has no ssh info' do
-        @gear_id = 'fakegearid'
+        @gear_id = 'fakegearid0'
 
         # Given - gears in gear group should not have ssh info
         gg = MockRestGearGroup.new(rest_client)
         @app.stub(:gear_groups).and_return([gg])
-        gg.stub(:gears).and_return([{'state' => 'started', 'id' => 'fakegearid'}])
+        gg.stub(:gears).and_return([{'state' => 'started', 'id' => 'fakegearid0'}])
 
         expect { run }.to exit_with_code(1)
-        run_output.should include('The server does not support per gear operations')
+        run_output.should match('The server does not support operations on individual gears.')
       end
 
     end

data/spec/rhc/commands/ssh_spec.rb

@@ -0,0 +1,125 @@
+require 'spec_helper'
+require 'rest_spec_helper'
+require 'rhc/commands/ssh'
+
+describe RHC::Commands::Ssh do
+  let!(:rest_client){ MockRestClient.new }
+  let!(:config){ user_config }
+  before{ RHC::Config.stub(:home_dir).and_return('/home/mock_user') }
+  before{ Kernel.stub(:exec).and_raise(RuntimeError) }
+
+  describe 'ssh default' do
+    context 'ssh' do
+      let(:arguments) { ['ssh'] }
+      it { run_output.should match('Usage:') }
+    end
+  end
+
+  describe 'ssh without command' do
+    let(:arguments) { ['app', 'ssh', 'app1'] }
+
+    context 'when run' do
+      before(:each) do
+        @domain = rest_client.add_domain("mockdomain")
+        @domain.add_application("app1", "mock_type")
+        Kernel.should_receive(:exec).with("ssh", "fakeuuidfortestsapp1@127.0.0.1").and_return(0)
+      end
+      it { run_output.should match("Connecting to fakeuuidfortestsapp") }
+      it { expect{ run }.to exit_with_code(0) }
+    end
+  end
+
+  describe 'app ssh with command' do
+    let(:arguments) { ['app', 'ssh', 'app1', 'ls', '/tmp'] }
+
+    context 'when run' do
+      before(:each) do
+        @domain = rest_client.add_domain("mockdomain")
+        @domain.add_application("app1", "mock_type")
+        Kernel.should_receive(:exec).with("ssh", "fakeuuidfortestsapp1@127.0.0.1", "ls", "/tmp").and_return(0)
+      end
+      it { run_output.should_not match("Connecting to fakeuuidfortestsapp") }
+      it { expect { run }.to exit_with_code(0) }
+    end
+  end
+
+  describe 'app ssh with --gears' do
+    before{ rest_client.add_domain("mockdomain").add_application("app1", "mock_type", true) }
+
+    context 'with no command' do
+      let(:arguments) { ['app', 'ssh', 'app1', '--gears'] }
+      it('should display usage info') { run_output.should match("Usage:") }
+      it { expect { run }.to exit_with_code(1) }
+    end
+    context 'with a command' do
+      let(:arguments) { ['app', 'ssh', 'app1', '--gears', 'command', '--trace'] }
+      before{ expect_multi_ssh('command', 'fakegearid0@fakesshurl.com' => 'foo', 'fakegearid1@fakesshurl.com' => 'bar') }
+      it('should print the ssh output') { run_output.should == "[fakegearid0 ] foo\n[fakegearid1 ] bar\n\n" }
+      it('should return successfully') { expect{ run }.to exit_with_code(0) }
+    end
+    context 'with --raw' do
+      let(:arguments) { ['app', 'ssh', 'app1', '--gears', 'command', '--raw'] }
+      before{ expect_multi_ssh('command', 'fakegearid0@fakesshurl.com' => 'foo', 'fakegearid1@fakesshurl.com' => 'bar') }
+      it('should print the ssh output') { run_output.should == "foo\nbar\n\n" }
+    end
+    context 'with --limit' do
+      let(:arguments) { ['app', 'ssh', 'app1', '--gears', 'command', '--limit', '1'] }
+      before{ expect_multi_ssh('command', 'fakegearid0@fakesshurl.com' => 'foo', 'fakegearid1@fakesshurl.com' => 'bar') }
+      it('should print the ssh output') { run_output.should == "[fakegearid0 ] foo\n[fakegearid1 ] bar\n\n" }
+    end
+    context 'with invalid --limit value' do
+      ['0','-10'].each do |value|
+        let(:arguments) { ['app', 'ssh', 'app1', '--gears', 'command', '--limit', value] }
+        it { run_output.should match('--limit must be an integer greater than zero') }
+      end
+    end
+    context 'with multiline output and --always-prefix' do
+      let(:arguments) { ['app', 'ssh', 'app1', '--gears', 'command', '--always-prefix'] }
+      before{ expect_multi_ssh('command', 'fakegearid0@fakesshurl.com' => "foo\ntest", 'fakegearid1@fakesshurl.com' => "bar\ntest") }
+      it('should print the ssh output') { run_output.should == "[fakegearid0 ] foo\n[fakegearid0 ] test\n[fakegearid1 ] bar\n[fakegearid1 ] test\n\n" }
+    end
+    context 'with multiline output' do
+      let(:arguments) { ['app', 'ssh', 'app1', '--gears', 'command'] }
+      before{ expect_multi_ssh('command', 'fakegearid0@fakesshurl.com' => "foo\ntest", 'fakegearid1@fakesshurl.com' => "bar\ntest") }
+      it('should print the ssh output') { run_output.should == "=== fakegearid0 \nfoo\ntest\n=== fakegearid1 \nbar\ntest\n\n" }
+    end
+  end
+
+  describe 'app ssh no system ssh' do
+    let(:arguments) { ['app', 'ssh', 'app1'] }
+
+    context 'when run' do
+      before(:each) do
+        @domain = rest_client.add_domain("mockdomain")
+        @domain.add_application("app1", "mock_type")
+        RHC::Commands::Ssh.any_instance.should_receive(:has_ssh?).and_return(false)
+      end
+      it { run_output.should match("Please use the --ssh option to specify the path to your SSH executable, or install SSH.") }
+      it { expect { run }.to exit_with_code(1) }
+    end
+  end
+
+  describe 'app ssh custom ssh' do
+    let(:arguments) { ['app', 'ssh', 'app1', '--ssh', 'path_to_ssh'] }
+
+    context 'when run' do
+      before(:each) do
+        @domain = rest_client.add_domain("mockdomain")
+        @domain.add_application("app1", "mock_type")
+        RHC::Commands::Ssh.any_instance.should_not_receive(:has_ssh?)
+        Kernel.should_receive(:exec).with("path_to_ssh", "fakeuuidfortestsapp1@127.0.0.1").once.times.and_return(0)
+      end
+      it { run_output.should match("Connecting to fakeuuidfortestsapp") }
+      it { expect { run }.to exit_with_code(0) }
+    end
+  end
+
+  describe 'ssh tests' do
+    let(:arguments) { ['app', 'ssh', 'app1', '-s /bin/blah'] }
+
+    context 'has_ssh?' do
+      before{ RHC::Commands::Ssh.any_instance.stub(:ssh_version){ raise "Fake Exception" } }
+      its(:has_ssh?) { should be_false }
+    end
+  end
+end