summaryrefslogtreecommitdiff
path: root/spec
diff options
context:
space:
mode:
Diffstat (limited to 'spec')
-rw-r--r--spec/unit/puppet/provider/vcsrepo/git_spec.rb117
-rw-r--r--spec/unit/puppet/provider/vcsrepo/svn_spec.rb36
2 files changed, 125 insertions, 28 deletions
diff --git a/spec/unit/puppet/provider/vcsrepo/git_spec.rb b/spec/unit/puppet/provider/vcsrepo/git_spec.rb
index a5ed753..f8a7170 100644
--- a/spec/unit/puppet/provider/vcsrepo/git_spec.rb
+++ b/spec/unit/puppet/provider/vcsrepo/git_spec.rb
@@ -34,12 +34,67 @@ describe provider_class do
end
end
context "when a source is not given" do
- it "should execute 'git init'" do
+ before do
@resource.expects(:value).with(:path).returns(@path).at_least_once
@resource.expects(:value).with(:source).returns(nil)
- Dir.expects(:chdir).with(@path).yields
- @provider.expects(:git).with('init')
- @provider.create
+ end
+ context "when ensure = present" do
+ 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)
+ Dir.expects(:chdir).with(@path).yields
+ @provider.expects(:bare_exists?).returns(false)
+ File.expects(:directory?).with(@path).returns(false)
+ @provider.expects(:git).with('init')
+ @provider.create
+ end
+ end
+ context "when the path is a bare repository" do
+ it "should convert it to a working copy" do
+ @provider.expects(:bare_exists?).returns(true)
+ @provider.expects(:convert_bare_to_working_copy)
+ @provider.create
+ end
+ end
+ context "when the path is not a repository" do
+ it "should raise an exception" do
+ File.expects(:directory?).with(@path).returns(true)
+ @provider.expects(:bare_exists?).returns(false)
+ proc {
+ @provider.create
+ }.should raise_error(Puppet::Error)
+ end
+ end
+ end
+ context "when ensure = bare" do
+ 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
+ File.expects(:directory?).with(@path).returns(false)
+ FileUtils.expects(:mkdir).with(@path)
+ @provider.expects(:working_copy_exists?).returns(false)
+ @provider.expects(:git).with('init', '--bare')
+ @provider.create
+ end
+ end
+ context "when the path is a working copy repository" do
+ it "should convert it to a bare repository" do
+ @provider.expects(:working_copy_exists?).returns(true)
+ @provider.expects(:convert_working_copy_to_bare)
+ @provider.create
+ end
+ end
+ context "when the path is not a repository" do
+ it "should raise an exception" do
+ File.expects(:directory?).with(@path).returns(true)
+ @provider.expects(:working_copy_exists?).returns(false)
+ proc {
+ @provider.create
+ }.should raise_error(Puppet::Error)
+ end
+ end
end
end
end
@@ -53,10 +108,54 @@ describe provider_class do
end
describe "when checking existence" do
- it "should check for the directory" do
- @resource.expects(:value).with(:path).returns(@path)
- File.expects(:directory?).with(@path)
- @provider.exists?
+ 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
@@ -106,7 +205,7 @@ describe provider_class do
end
end
end
-
+
describe "when setting the revision property" do
it "should use 'git fetch' and 'git reset'" do
@resource.expects(:value).with(:path).returns(@path).at_least_once
diff --git a/spec/unit/puppet/provider/vcsrepo/svn_spec.rb b/spec/unit/puppet/provider/vcsrepo/svn_spec.rb
index 3af06f2..fc5c37b 100644
--- a/spec/unit/puppet/provider/vcsrepo/svn_spec.rb
+++ b/spec/unit/puppet/provider/vcsrepo/svn_spec.rb
@@ -10,7 +10,7 @@ describe provider_class do
@path = '/tmp/vcsrepo'
end
- context 'when creating' do
+ describe 'when creating' do
context "when a source is given" do
context "and when a revision is given" do
it "should execute 'svn checkout' with a revision" do
@@ -53,7 +53,7 @@ describe provider_class do
end
end
- context 'when destroying' do
+ describe 'when destroying' do
it "it should remove the directory" do
@resource.expects(:value).with(:path).returns(@path).at_least_once
FileUtils.expects(:rm_rf).with(@path)
@@ -61,7 +61,7 @@ describe provider_class do
end
end
- context "when checking existence" do
+ describe "when checking existence" do
it "should check for the directory" do
@resource.expects(:value).with(:path).returns(@path)
File.expects(:directory?).with(@path)
@@ -69,23 +69,21 @@ describe provider_class do
end
end
- describe "revision property" do
- context "when checking" do
- it "should use 'svn info'" do
- @resource.expects(:value).with(:path).returns(@path)
- p fixture(:svn_info)[/^Revision:\s+(\d+)/m, 1]
- @provider.expects('svn').with('info').returns(fixture(:svn_info))
- Dir.expects(:chdir).with(@path).yields
- @provider.revision.should == '4'
- end
+ describe "when checking the revision property" do
+ it "should use 'svn info'" do
+ @resource.expects(:value).with(:path).returns(@path)
+ @provider.expects('svn').with('info').returns(fixture(:svn_info))
+ Dir.expects(:chdir).with(@path).yields
+ @provider.revision.should == '4'
end
- context "when setting" do
- it "should use 'svn update'" do
- @resource.expects(:value).with(:path).returns(@path)
- @provider.expects('svn').with('update', '-r', '30')
- Dir.expects(:chdir).with(@path).yields
- @provider.revision = '30'
- end
+ end
+
+ describe "when setting the revision property" do
+ it "should use 'svn update'" do
+ @resource.expects(:value).with(:path).returns(@path)
+ @provider.expects('svn').with('update', '-r', '30')
+ Dir.expects(:chdir).with(@path).yields
+ @provider.revision = '30'
end
end