From 176b1ec7a4620cae24996609ada33f21cc2eef84 Mon Sep 17 00:00:00 2001 From: Bruce Williams Date: Fri, 12 Mar 2010 16:54:25 -0800 Subject: [PATCH] Basic tests for git provider, stub for svn provider --- spec/spec.opts | 6 ++ spec/spec_helper.rb | 18 +++++ spec/unit/puppet/provider/vcsrepo/git_spec.rb | 65 +++++++++++++++++++ spec/unit/puppet/provider/vcsrepo/svn_spec.rb | 16 +++++ 4 files changed, 105 insertions(+) create mode 100644 spec/spec.opts create mode 100644 spec/spec_helper.rb create mode 100644 spec/unit/puppet/provider/vcsrepo/git_spec.rb create mode 100644 spec/unit/puppet/provider/vcsrepo/svn_spec.rb diff --git a/spec/spec.opts b/spec/spec.opts new file mode 100644 index 0000000..91cd642 --- /dev/null +++ b/spec/spec.opts @@ -0,0 +1,6 @@ +--format +s +--colour +--loadby +mtime +--backtrace diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb new file mode 100644 index 0000000..a4aeeae --- /dev/null +++ b/spec/spec_helper.rb @@ -0,0 +1,18 @@ +require 'pathname' +dir = Pathname.new(__FILE__).parent +$LOAD_PATH.unshift(dir, dir + 'lib', dir + '../lib') + +require 'mocha' +require 'puppet' +gem 'rspec', '=1.2.9' +require 'spec/autorun' + +Spec::Runner.configure do |config| + config.mock_with :mocha +end + +# We need this because the RAL uses 'should' as a method. This +# allows us the same behaviour but with a different method name. +class Object + alias :must :should +end diff --git a/spec/unit/puppet/provider/vcsrepo/git_spec.rb b/spec/unit/puppet/provider/vcsrepo/git_spec.rb new file mode 100644 index 0000000..d859004 --- /dev/null +++ b/spec/unit/puppet/provider/vcsrepo/git_spec.rb @@ -0,0 +1,65 @@ +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(:git) + +describe provider_class do + + before :each do + @resource = stub("resource") + @provider = provider_class.new(@resource) + @path = '/tmp/vcsrepo' + end + + context '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 + 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 + end + end + context "when a source is not given" do + it "should execute 'git init'" 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 + end + end + + context '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) + @provider.destroy + end + end + + context "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? + end + end + + context "when checking 'revision' property" + +end diff --git a/spec/unit/puppet/provider/vcsrepo/svn_spec.rb b/spec/unit/puppet/provider/vcsrepo/svn_spec.rb new file mode 100644 index 0000000..657913b --- /dev/null +++ b/spec/unit/puppet/provider/vcsrepo/svn_spec.rb @@ -0,0 +1,16 @@ +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(:svn) + +describe provider_class do + + before do + @resource = stub("resource") + @provider = provider_class.new(@resource) + end + + describe 'when creating' + describe 'when updating' + describe 'when destroying' + +end