some tests for the enable_checkout option

2 files changed, 25 insertions, 0 deletions

diff --git a/spec/unit/provider/git_spec.rb b/spec/unit/provider/git_spec.rb
index 2bf55930db..31fb6e38ef 100644
--- a/spec/unit/provider/git_spec.rb
+++ b/spec/unit/provider/git_spec.rb
@@ -402,6 +402,21 @@ SHAS
     # @resource.should be_updated
   end
 
+  it "does not call checkout if enable_checkout is false" do
+    # will be invoked in load_current_resource
+    ::File.stub!(:exist?).with("/my/deploy/dir/.git").and_return(false)
+
+    ::File.stub!(:exist?).with("/my/deploy/dir").and_return(true)
+    ::File.stub!(:directory?).with("/my/deploy").and_return(true)
+    ::Dir.stub!(:entries).with("/my/deploy/dir").and_return(['.','..'])
+
+    @resource.enable_checkout false
+    @provider.should_receive(:clone)
+    @provider.should_not_receive(:checkout)
+    @provider.should_receive(:enable_submodules)
+    @provider.run_action(:checkout)
+  end
+
   # REGRESSION TEST: on some OSes, the entries from an empty directory will be listed as
   # ['..', '.'] but this shouldn't change the behavior
   it "does a checkout by cloning the repo and then enabling submodules when the directory entries are listed as %w{.. .}" do
diff --git a/spec/unit/resource/scm_spec.rb b/spec/unit/resource/scm_spec.rb
index 8f6593a931..2141ae3c88 100644
--- a/spec/unit/resource/scm_spec.rb
+++ b/spec/unit/resource/scm_spec.rb
@@ -118,6 +118,16 @@ describe Chef::Resource::Scm do
     @resource.enable_submodules.should be_false
   end
 
+  it "takes a boolean for #enable_checkout" do
+    @resource.enable_checkout true
+    @resource.enable_checkout.should be_true
+    lambda {@resource.enable_checkout "lolz"}.should raise_error(ArgumentError)
+  end
+
+  it "defaults to enabling checkout" do
+    @resource.enable_checkout.should be_true
+  end
+
   it "takes a string for the remote" do
     @resource.remote "opscode"
     @resource.remote.should eql("opscode")