summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBruce Williams <bruce@codefluency.com>2010-03-18 00:16:43 -0700
committerBruce Williams <bruce@codefluency.com>2010-03-18 00:16:43 -0700
commitdc3e0ee4fb56c5704925253d8727d6fdd343cf5c (patch)
treefb4971773c268521721b46ef0d2ec5e9df380c40
parent736c094145ff6f86d2bfdc8cecf32da8180d95fe (diff)
CVS uses provider example group for specs
-rw-r--r--spec/support/vcsrepo_helpers.rb9
-rw-r--r--spec/unit/puppet/provider/vcsrepo/cvs_spec.rb118
2 files changed, 45 insertions, 82 deletions
diff --git a/spec/support/vcsrepo_helpers.rb b/spec/support/vcsrepo_helpers.rb
index e0b9beb..f631db0d 100644
--- a/spec/support/vcsrepo_helpers.rb
+++ b/spec/support/vcsrepo_helpers.rb
@@ -1,12 +1,11 @@
module VcsrepoHelpers
- def expects_chdir
- Dir.expects(:chdir).with(resource.value(:path)).at_least_once.yields
+ def expects_chdir(path = resource.value(:path))
+ Dir.expects(:chdir).with(path).at_least_once.yields
end
- def expects_mkdir
- Dir.expects(:mkdir).with(resource.value(:path)).at_least_once
+ def expects_mkdir(path = resource.value(:path))
+ Dir.expects(:mkdir).with(path).at_least_once
end
-
end
diff --git a/spec/unit/puppet/provider/vcsrepo/cvs_spec.rb b/spec/unit/puppet/provider/vcsrepo/cvs_spec.rb
index c0c2dbc..2860af5 100644
--- a/spec/unit/puppet/provider/vcsrepo/cvs_spec.rb
+++ b/spec/unit/puppet/provider/vcsrepo/cvs_spec.rb
@@ -1,107 +1,70 @@
require 'pathname'; Pathname.new(__FILE__).realpath.ascend { |x| begin; require (x + 'spec_helper.rb'); break; rescue LoadError; end }
-provider_class = Puppet::Type.type(:vcsrepo).provider(:cvs)
+describe_provider :vcsrepo, :cvs, :resource => {:path => '/tmp/vcsrepo'} do
-describe provider_class do
-
- before :each do
- @resource = stub("resource")
- @provider = provider_class.new(@resource)
- @path = '/tmp/vcsrepo'
- end
-
- describe 'when creating' do
- before do
- @resource.expects(:value).with(:path).returns(@path).at_least_once
- end
- context "when a source is given" do
- before do
- @source = ":pserver:anonymous@example.com:/sources/myproj"
- @resource.expects(:value).with(:source).returns(@source).at_least_once
- Dir.expects(:chdir).with(File.dirname(@path)).yields
- end
- context "and when a revision is given" do
- before do
- @tag = 'SOMETAG'
- @resource.expects(:value).with(:revision).returns(@tag).at_least_once
- @resource.expects(:value).with(:compression).returns(nil).at_least_once
- end
+ describe 'creating' do
+ context "with a source", :resource => {:source => ':ext:source@example.com:/foo/bar'} do
+ context_with :revision do
it "should execute 'cvs checkout' and 'cvs update -r'" do
- @provider.expects(:cvs).with('-d', @source, 'checkout', '-d', File.basename(@path), File.basename(@source))
- Dir.expects(:chdir).with(@path).yields
- @provider.expects(:cvs).with('update', '-r', @tag, '.')
- @provider.create
+ expects_chdir
+ expects_chdir(File.dirname(resource.value(:path)))
+ provider.expects(:cvs).with('-d', resource.value(:source), 'checkout', '-d', File.basename(resource.value(:path)), File.basename(resource.value(:source)))
+ provider.expects(:cvs).with('update', '-r', resource.value(:revision), '.')
+ provider.create
end
end
- context "and when a revision is not given" do
- before do
- @resource.expects(:value).with(:revision).returns(nil).at_least_once
- @resource.expects(:value).with(:compression).returns(nil).at_least_once
- end
+
+ context_without :revision do
it "should just execute 'cvs checkout' without a revision" do
- @provider.expects(:cvs).with('-d', @source, 'checkout', '-d', File.basename(@path), File.basename(@source))
- @provider.create
+ provider.expects(:cvs).with('-d', resource.value(:source), 'checkout', '-d', File.basename(resource.value(:path)), File.basename(resource.value(:source)))
+ provider.create
end
end
- context "when a compression level is given" do
- before do
- @resource.expects(:value).with(:revision).returns(nil).at_least_once
- @resource.expects(:value).with(:compression).returns('3').at_least_once
- end
+
+ context "with a compression", :resource => {:compression => '3'} do
it "should just execute 'cvs checkout' without a revision" do
- @provider.expects(:cvs).with('-d', @source, '-z', '3', 'checkout', '-d', File.basename(@path), File.basename(@source))
- @provider.create
+ provider.expects(:cvs).with('-d', resource.value(:source), '-z', '3', 'checkout', '-d', File.basename(resource.value(:path)), File.basename(resource.value(:source)))
+ provider.create
end
end
end
+
context "when a source is not given" do
- before do
- @resource.expects(:value).with(:source).returns(nil)
- end
it "should execute 'cvs init'" do
- @provider.expects(:cvs).with('-d', @path, 'init')
- @provider.create
+ provider.expects(:cvs).with('-d', resource.value(:path), 'init')
+ provider.create
end
end
end
- describe 'when destroying' do
+ describe '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)
- @provider.destroy
+ FileUtils.expects(:rm_rf).with(resource.value(:path))
+ provider.destroy
end
end
- describe "when checking existence" do
- before do
- @resource.expects(:value).with(:path).returns(@path)
- end
- context "when a source is provided" do
- before do
- @resource.expects(:value).with(:source).returns(":pserver:anonymous@example.com:/sources/myproj")
- end
+ describe "checking existence" do
+ context_with :source do
it "should check for the CVS directory" do
- File.expects(:directory?).with(File.join(@path, 'CVS'))
- @provider.exists?
+ File.expects(:directory?).with(File.join(resource.value(:path), 'CVS'))
+ provider.exists?
end
end
- context "when a source is not provided" do
- before do
- @resource.expects(:value).with(:source).returns(nil)
- end
+
+ context_without :source do
it "should check for the CVSROOT directory" do
- File.expects(:directory?).with(File.join(@path, 'CVSROOT'))
- @provider.exists?
+ File.expects(:directory?).with(File.join(resource.value(:path), 'CVSROOT'))
+ provider.exists?
end
end
end
- describe "when checking the revision property" do
+ describe "checking the revision property" do
before do
- @resource.expects(:value).with(:path).returns(@path).at_least_once
- @tag_file = File.join(@path, 'CVS', 'Tag')
+ @tag_file = File.join(resource.value(:path), 'CVS', 'Tag')
end
+
context "when CVS/Tag exists" do
before do
@tag = 'HEAD'
@@ -109,28 +72,29 @@ describe provider_class do
end
it "should read CVS/Tag" do
File.expects(:read).with(@tag_file).returns("T#{@tag}")
- @provider.revision.should == @tag
+ provider.revision.should == @tag
end
end
+
context "when CVS/Tag does not exist" do
before do
File.expects(:exist?).with(@tag_file).returns(false)
end
it "assumes MAIN" do
- @provider.revision.should == 'MAIN'
+ provider.revision.should == 'MAIN'
end
end
end
describe "when setting the revision property" do
before do
- @resource.expects(:value).with(:path).returns(@path).at_least_once
@tag = 'SOMETAG'
end
+
it "should use 'cvs update -r'" do
- Dir.expects(:chdir).with(@path).yields
- @provider.expects('cvs').with('update', '-r', @tag, '.')
- @provider.revision = @tag
+ expects_chdir
+ provider.expects('cvs').with('update', '-r', @tag, '.')
+ provider.revision = @tag
end
end