startapp 0.1.6

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: app 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: app 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: app 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: app 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: app 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_OPENSHIFT_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_OPENSHIFT_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,102 @@
+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 'when run without git installed' do
+      before do
+        @instance.stub(:has_git?) { false }
+      end
+      it "should print out git warning" do
+        run_output.should match("You do not have git installed")
+      end
+    end
+
+    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") }
+
+        context 'when app has an initial git url' do
+          before do
+            @app2 = @domain.add_application("app2", "mock_unique_standalone_cart", nil, "default", "git://test")
+            @instance.stub(:git_remote_add) do |remote_name, remote_url|
+              say "Added remote #{remote_name} pointing to #{remote_url}"
+              true
+            end
+          end
+          let(:arguments) { ['git-clone', 'app2'] }
+          it { run_output.should match("Added remote startapp pointing to git://test") }
+        end
+
+      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}/.openshift/git_hooks"
+            FileUtils.touch "#{repo_dir}/.openshift/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