Overhaul the spec tests to work in rspec2.
[puppet_vcsrepo.git] / spec / unit / puppet / provider / vcsrepo / cvs_spec.rb
index c0c2dbc..efa4b33 100644 (file)
-require 'pathname'; Pathname.new(__FILE__).realpath.ascend { |x| begin; require (x + 'spec_helper.rb'); break; rescue LoadError; end }
+require 'spec_helper'
 
-provider_class = Puppet::Type.type(:vcsrepo).provider(:cvs)
+describe Puppet::Type.type(:vcsrepo).provider(:cvs_provider) do
 
-describe provider_class do
+  let(:resource) { Puppet::Type.type(:vcsrepo).new({
+    :name     => 'test',
+    :ensure   => :present,
+    :provider => :cvs,
+    :revision => '2634',
+    :source   => 'lp:do',
+    :path     => '/tmp/test',
+  })}
+
+  let(:provider) { resource.provider }
 
   before :each do
-    @resource = stub("resource")
-    @provider = provider_class.new(@resource)
-    @path = '/tmp/vcsrepo'
+    Puppet::Util.stubs(:which).with('cvs').returns('/usr/bin/cvs')
   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
+  describe 'creating' do
+    context "with a source" do
+      it "should execute 'cvs checkout'" do
+        resource[:source] = ':ext:source@example.com:/foo/bar'
+        resource[:revision] = 'an-unimportant-value'
+        expects_chdir('/tmp')
+        provider.expects(:cvs).with('-d', resource.value(:source), 'checkout', '-r', 'an-unimportant-value', '-d', 'test', 'bar')
+        provider.create
       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
-        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
-        end        
+
+      it "should just execute 'cvs checkout' without a revision" do
+        resource[:source] = ':ext:source@example.com:/foo/bar'
+        resource.delete(:revision)
+        provider.expects(:cvs).with('-d', resource.value(:source), 'checkout', '-d', File.basename(resource.value(:path)), File.basename(resource.value(:source)))
+        provider.create
       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 "with a compression" 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
-        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
+          resource[:source] = ':ext:source@example.com:/foo/bar'
+          resource[:compression] = '3'
+          resource.delete(:revision)
+          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
-        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
-        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
+        resource.delete(:source)
+        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
+      provider.destroy
     end
   end
 
-  describe "when checking existence" do
-    before do
-      @resource.expects(:value).with(:path).returns(@path)
+  describe "checking existence" do
+    it "should check for the CVS directory with source" do
+      resource[:source] = ':ext:source@example.com:/foo/bar'
+      File.expects(:directory?).with(File.join(resource.value(:path), 'CVS'))
+      provider.exists?
     end
-    context "when a source is provided" do
-      before do
-        @resource.expects(:value).with(:source).returns(":pserver:anonymous@example.com:/sources/myproj")
-      end
-      it "should check for the CVS directory" do
-        File.expects(:directory?).with(File.join(@path, 'CVS'))
-        @provider.exists?
-      end
-    end
-    context "when a source is not provided" do
-      before do
-        @resource.expects(:value).with(:source).returns(nil)        
-      end
-      it "should check for the CVSROOT directory" do
-        File.expects(:directory?).with(File.join(@path, 'CVSROOT'))
-        @provider.exists?
-      end
+
+    it "should check for the CVSROOT directory without source" do
+      resource.delete(:source)
+      File.expects(:directory?).with(File.join(resource.value(:path), 'CVSROOT'))
+      provider.exists?
     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'
+        @tag = 'TAG'
         File.expects(:exist?).with(@tag_file).returns(true)
       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'        
+      it "assumes HEAD" do
+        provider.revision.should == 'HEAD'
       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
+
+    it "should use 'cvs update -dr'" do
+      expects_chdir
+      provider.expects(:cvs).with('update', '-dr', @tag, '.')
+      provider.revision = @tag
     end
   end