From 0a306f288e84bed857e4d82c695dcea0c6ceb37a Mon Sep 17 00:00:00 2001 From: Bruce Williams Date: Sat, 13 Mar 2010 01:05:20 -0800 Subject: Some fixes after live testing, add support for bare cloned repos --- spec/unit/puppet/provider/vcsrepo/git_spec.rb | 122 +++++++++++--------------- 1 file changed, 49 insertions(+), 73 deletions(-) (limited to 'spec/unit/puppet/provider/vcsrepo/git_spec.rb') diff --git a/spec/unit/puppet/provider/vcsrepo/git_spec.rb b/spec/unit/puppet/provider/vcsrepo/git_spec.rb index f8a7170..5cf6c86 100644 --- a/spec/unit/puppet/provider/vcsrepo/git_spec.rb +++ b/spec/unit/puppet/provider/vcsrepo/git_spec.rb @@ -11,27 +11,55 @@ describe provider_class do end describe 'when creating' do - context "when a source is given" do - context "and when a revision is given" do - it "should execute 'git clone' and 'git reset'" do - @resource.expects(:value).with(:path).returns(@path).at_least_once - @resource.expects(:value).with(:source).returns('git://example.com/repo.git').at_least_once - @provider.expects(:git).with('clone', 'git://example.com/repo.git', @path) - @resource.expects(:value).with(:revision).returns('abcdef').at_least_once - Dir.expects(:chdir).with(@path).yields - @provider.expects('git').with('reset', '--hard', 'abcdef') - @provider.create - end + context "and when a source is given" do + before do + @resource.expects(:value).with(:source).returns('git://example.com/repo.git').at_least_once end - context "and when a revision is not given" do - it "should just execute 'git clone'" do - @resource.expects(:value).with(:path).returns(@path).at_least_once - @resource.expects(:value).with(:source).returns('git://example.com/repo.git').at_least_once - @resource.expects(:value).with(:revision).returns(nil).at_least_once - @provider.expects(:git).with('clone', 'git://example.com/repo.git', @path) - @provider.create - end + context "and when ensure = present" do + before do + @resource.expects(:value).with(:ensure).returns(:present).at_least_once + end + context "and when a revision is given" do + it "should execute 'git clone' and 'git reset'" do + @resource.expects(:value).with(:path).returns(@path).at_least_once + @provider.expects(:git).with('clone', 'git://example.com/repo.git', @path) + @resource.expects(:value).with(:revision).returns('abcdef').at_least_once + Dir.expects(:chdir).with(@path).yields + @provider.expects('git').with('reset', '--hard', 'abcdef') + @provider.create + end + end + context "and when a revision is not given" do + it "should just execute 'git clone'" do + @resource.expects(:value).with(:path).returns(@path).at_least_once + @resource.expects(:value).with(:revision).returns(nil).at_least_once + @provider.expects(:git).with('clone', 'git://example.com/repo.git', @path) + @provider.create + end + end end + context "and when ensure = bare" do + before do + @resource.expects(:value).with(:ensure).returns(:bare).at_least_once + end + context "and when a revision is given" do + it "should just execute 'git clone --bare'" do + @resource.expects(:value).with(:path).returns(@path).at_least_once + @resource.expects(:value).with(:revision).returns(nil).at_least_once + @provider.expects(:git).with('clone', '--bare', 'git://example.com/repo.git', @path) + @provider.create + end + end + context "and when a revision is not given" do + it "should just execute 'git clone --bare'" do + @resource.expects(:value).with(:path).returns(@path).at_least_once + @resource.expects(:value).with(:revision).returns(nil).at_least_once + @provider.expects(:git).with('clone', '--bare', 'git://example.com/repo.git', @path) + @provider.create + end + end + end + end context "when a source is not given" do before do @@ -39,7 +67,7 @@ describe provider_class do @resource.expects(:value).with(:source).returns(nil) end context "when ensure = present" do - before { @resource.expects(:value).with(:ensure).returns('present').at_least_once } + before { @resource.expects(:value).with(:ensure).returns(:present).at_least_once } context "when the path does not exist" do it "should execute 'git init'" do Dir.expects(:mkdir).with(@path) @@ -68,7 +96,7 @@ describe provider_class do end end context "when ensure = bare" do - before { @resource.expects(:value).with(:ensure).returns('bare').at_least_once } + before { @resource.expects(:value).with(:ensure).returns(:bare).at_least_once } context "when the path does not exist" do it "should execute 'git init --bare'" do Dir.expects(:chdir).with(@path).yields @@ -107,58 +135,6 @@ describe provider_class do end end - describe "when checking existence" do - context "when ensure = present" do - context "when a working copy exists" do - it "should be true" do - @resource.expects(:value).with(:ensure).returns('present').at_least_once - @provider.expects(:working_copy_exists?).returns(true) - @provider.should be_exists - end - end - context "when a bare repo exists" do - it "should be " do - @resource.expects(:value).with(:ensure).returns('present').at_least_once - @provider.expects(:working_copy_exists?).returns(false) - @provider.should_not be_exists - end - end - end - context "when ensure = bare" do - context "when a working copy exists" do - it "should be false" do - @resource.expects(:value).with(:ensure).returns('bare').at_least_once - @provider.expects(:bare_exists?).returns(false) - @provider.should_not be_exists - end - end - context "when a bare repo exists" do - it "should be true" do - @resource.expects(:value).with(:ensure).returns('bare').at_least_once - @provider.expects(:bare_exists?).returns(true) - @provider.should be_exists - end - end - end - context "when ensure = absent" do - before { @resource.expects(:value).with(:ensure).returns('absent') } - context "when the path exists" do - it "should be true" do - @resource.expects(:value).with(:path).returns(@path) - File.expects(:directory?).with(@path).returns(true) - @provider.should be_exists - end - end - context "when the path does not exist" do - it "should be false" do - @resource.expects(:value).with(:path).returns(@path) - File.expects(:directory?).with(@path).returns(false) - @provider.should_not be_exists - end - end - end - end - describe "when checking the revision property" do context "when given a non-SHA ref as the resource revision" do context "when its SHA is not different than the curent SHA" do -- cgit v1.2.3