puppet 0.25.5 → 2.6.0

data/spec/unit/provider/mount_spec.rb

@@ -0,0 +1,130 @@
+#!/usr/bin/env ruby
+
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+require 'puppet/provider/mount'
+
+describe Puppet::Provider::Mount do
+  before :each do
+    @mounter = Object.new
+    @mounter.extend(Puppet::Provider::Mount)
+
+    @name = "/"
+
+    @resource = stub 'resource'
+    @resource.stubs(:[]).with(:name).returns(@name)
+
+    @mounter.stubs(:resource).returns(@resource)
+  end
+
+  describe Puppet::Provider::Mount, " when mounting" do
+
+    it "should use the 'mountcmd' method to mount" do
+      @mounter.stubs(:options).returns(nil)
+      @mounter.expects(:mountcmd)
+
+      @mounter.mount
+    end
+
+    it "should flush before mounting if a flush method exists" do
+      @mounter.meta_def(:flush) { }
+      @mounter.expects(:flush)
+      @mounter.stubs(:mountcmd)
+      @mounter.stubs(:options).returns(nil)
+
+      @mounter.mount
+    end
+
+    it "should add the options following '-o' if they exist and are not set to :absent" do
+      @mounter.stubs(:options).returns("ro")
+      @mounter.expects(:mountcmd).with { |*ary| ary[0] == "-o" and ary[1] == "ro" }
+
+      @mounter.mount
+    end
+
+    it "should specify the filesystem name to the mount command" do
+      @mounter.stubs(:options).returns(nil)
+      @mounter.expects(:mountcmd).with { |*ary| ary[-1] == @name }
+
+      @mounter.mount
+    end
+  end
+
+  describe Puppet::Provider::Mount, " when remounting" do
+
+    it "should use '-o remount' if the resource specifies it supports remounting" do
+      @mounter.stubs(:info)
+      @resource.stubs(:[]).with(:remounts).returns(:true)
+      @mounter.expects(:mountcmd).with("-o", "remount", @name)
+      @mounter.remount
+    end
+
+    it "should unmount and mount if the resource does not specify it supports remounting" do
+      @mounter.stubs(:info)
+      @resource.stubs(:[]).with(:remounts).returns(false)
+      @mounter.expects(:unmount)
+      @mounter.expects(:mount)
+      @mounter.remount
+    end
+
+    it "should log that it is remounting" do
+      @resource.stubs(:[]).with(:remounts).returns(:true)
+      @mounter.stubs(:mountcmd)
+      @mounter.expects(:info).with("Remounting")
+      @mounter.remount
+    end
+  end
+
+  describe Puppet::Provider::Mount, " when unmounting" do
+
+    it "should call the :umount command with the resource name" do
+      @mounter.expects(:umount).with(@name)
+      @mounter.unmount
+    end
+  end
+
+  describe Puppet::Provider::Mount, " when determining if it is mounted" do
+
+    it "should parse the results of running the mount command with no arguments" do
+      Facter.stubs(:value).returns("whatever")
+      @mounter.expects(:mountcmd).returns("")
+
+      @mounter.mounted?
+    end
+
+    it "should match ' on /private/var/automount<name>' if the operating system is Darwin" do
+      Facter.stubs(:value).with("operatingsystem").returns("Darwin")
+      @mounter.expects(:mountcmd).returns("/dev/whatever on /private/var/automount/\ndevfs on /dev")
+
+      @mounter.should be_mounted
+    end
+
+    it "should match ' on <name>' if the operating system is Darwin" do
+      Facter.stubs(:value).with("operatingsystem").returns("Darwin")
+      @mounter.expects(:mountcmd).returns("/dev/disk03 on / (local, journaled)\ndevfs on /dev")
+
+      @mounter.should be_mounted
+    end
+
+    it "should match '^<name> on' if the operating system is Solaris" do
+      Facter.stubs(:value).with("operatingsystem").returns("Solaris")
+      @mounter.expects(:mountcmd).returns("/ on /dev/dsk/whatever\n/var on /dev/dsk/other")
+
+      @mounter.should be_mounted
+    end
+
+    it "should match ' on <name>' if the operating system is not Darwin or Solaris" do
+      Facter.stubs(:value).with("operatingsystem").returns("Debian")
+      @mounter.expects(:mountcmd).returns("/dev/dsk/whatever on / and stuff\n/dev/other/disk on /var and stuff")
+
+      @mounter.should be_mounted
+    end
+
+    it "should not be considered mounted if it did not match the mount output" do
+      Facter.stubs(:value).with("operatingsystem").returns("Debian")
+      @mounter.expects(:mountcmd).returns("/dev/dsk/whatever on /something/else and stuff\n/dev/other/disk on /var and stuff")
+
+      @mounter.should_not be_mounted
+    end
+  end
+end

data/spec/unit/provider/naginator_spec.rb

@@ -0,0 +1,58 @@
+#!/usr/bin/env ruby
+
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+require 'puppet/provider/naginator'
+
+describe Puppet::Provider::Naginator do
+  before do
+    @resource_type = stub 'resource_type', :name => :nagios_test
+    @class = Class.new(Puppet::Provider::Naginator)
+
+    @class.stubs(:resource_type).returns @resource_type
+  end
+
+  it "should be able to look up the associated Nagios type" do
+    nagios_type = mock "nagios_type"
+    nagios_type.stubs :attr_accessor
+    Nagios::Base.expects(:type).with(:test).returns nagios_type
+
+    @class.nagios_type.should equal(nagios_type)
+  end
+
+  it "should use the Nagios type to determine whether an attribute is valid" do
+    nagios_type = mock "nagios_type"
+    nagios_type.stubs :attr_accessor
+    Nagios::Base.expects(:type).with(:test).returns nagios_type
+
+    nagios_type.expects(:parameters).returns [:foo, :bar]
+
+    @class.valid_attr?(:test, :foo).should be_true
+  end
+
+  it "should use Naginator to parse configuration snippets" do
+    parser = mock 'parser'
+    parser.expects(:parse).with("my text").returns "my instances"
+    Nagios::Parser.expects(:new).returns(parser)
+
+    @class.parse("my text").should == "my instances"
+  end
+
+  it "should join Nagios::Base records with '\\n' when asked to convert them to text" do
+    @class.expects(:header).returns "myheader\n"
+
+    @class.to_file([:one, :two]).should == "myheader\none\ntwo"
+  end
+
+  it "should be able to prefetch instance from configuration files" do
+    @class.should respond_to(:prefetch)
+  end
+
+  it "should be able to generate a list of instances" do
+    @class.should respond_to(:instances)
+  end
+
+  it "should never skip records" do
+    @class.should_not be_skip_record("foo")
+  end
+end

data/spec/unit/provider/package/aix_spec.rb

@@ -0,0 +1,66 @@
+#!/usr/bin/env ruby
+
+require File.dirname(__FILE__) + '/../../../spec_helper'
+
+provider_class = Puppet::Type.type(:package).provider(:aix)
+
+describe provider_class do
+  before(:each) do
+    # Create a mock resource
+    @resource = stub 'resource'
+
+    # A catch all; no parameters set
+    @resource.stubs(:[]).returns(nil)
+
+    # But set name and source
+    @resource.stubs(:[]).with(:name).returns "mypackage"
+    @resource.stubs(:[]).with(:source).returns "mysource"
+    @resource.stubs(:[]).with(:ensure).returns :installed
+
+    @provider = provider_class.new
+    @provider.resource = @resource
+  end
+
+  [:install, :uninstall, :latest, :query, :update].each do |method|
+    it "should have a #{method} method" do
+      @provider.should respond_to(method)
+    end
+  end
+
+  it "should uninstall a package" do
+    @provider.expects(:installp).with('-gu', 'mypackage')
+    @provider.uninstall
+  end
+
+  describe "when installing" do
+    it "should install a package" do
+      @resource.stubs(:should).with(:ensure).returns(:installed)
+      @provider.expects(:installp).with('-acgwXY', '-d', 'mysource', 'mypackage')
+      @provider.install
+    end
+
+    it "should install a specific package version" do
+      @resource.stubs(:should).with(:ensure).returns("1.2.3.4")
+      @provider.expects(:installp).with('-acgwXY', '-d', 'mysource', 'mypackage 1.2.3.4')
+      @provider.install
+    end
+  end
+
+  describe "when finding the latest version" do
+    it "should return the current version when no later version is present" do
+      @provider.stubs(:latest_info).returns(nil)
+      @provider.stubs(:properties).returns( { :ensure => "1.2.3.4" } )
+      @provider.latest.should == "1.2.3.4"
+    end
+
+    it "should return the latest version of a package" do
+      @provider.stubs(:latest_info).returns( { :version => "1.2.3.5" } )
+      @provider.latest.should == "1.2.3.5"
+    end
+  end
+
+  it "update should install a package" do
+    @provider.expects(:install).with(false)
+    @provider.update
+  end
+end

data/spec/unit/provider/package/apt_spec.rb

@@ -0,0 +1,145 @@
+#!/usr/bin/env ruby
+
+Dir.chdir(File.dirname(__FILE__)) { (s = lambda { |f| File.exist?(f) ? require(f) : Dir.chdir("..") { s.call(f) } }).call("spec/spec_helper.rb") }
+
+provider = Puppet::Type.type(:package).provider(:apt)
+
+describe provider do
+  before do
+    @resource = stub 'resource', :[] => "asdf"
+    @provider = provider.new(@resource)
+
+    @fakeresult = "install ok installed asdf 1.0\n"
+  end
+
+  it "should be versionable" do
+    provider.should be_versionable
+  end
+
+  it "should use :install to update" do
+    @provider.expects(:install)
+    @provider.update
+  end
+
+  it "should use 'apt-get remove' to uninstall" do
+    @provider.expects(:aptget).with("-y", "-q", :remove, "asdf")
+
+    @provider.uninstall
+  end
+
+  it "should use 'apt-get purge' and 'dpkg purge' to purge" do
+    @provider.expects(:aptget).with("-y", "-q", :remove, "--purge", "asdf")
+    @provider.expects(:dpkg).with("--purge", "asdf")
+
+    @provider.purge
+  end
+
+  it "should use 'apt-cache policy' to determine the latest version of a package" do
+    @provider.expects(:aptcache).with(:policy, "asdf").returns "asdf:
+Installed: 1:1.0
+Candidate: 1:1.1
+Version table:
+1:1.0
+  650 http://ftp.osuosl.org testing/main Packages
+*** 1:1.1
+  100 /var/lib/dpkg/status"
+
+    @provider.latest.should == "1:1.1"
+  end
+
+  it "should print and error and return nil if no policy is found" do
+    @provider.expects(:aptcache).with(:policy, "asdf").returns "asdf:"
+
+    @provider.expects(:err)
+    @provider.latest.should be_nil
+  end
+
+  it "should be able to preseed" do
+    @provider.should respond_to(:run_preseed)
+  end
+
+  it "should preseed with the provided responsefile when preseeding is called for" do
+    @resource.expects(:[]).with(:responsefile).returns "/my/file"
+    FileTest.expects(:exist?).with("/my/file").returns true
+
+    @provider.expects(:info)
+    @provider.expects(:preseed).with("/my/file")
+
+    @provider.run_preseed
+  end
+
+  it "should not preseed if no responsefile is provided" do
+    @resource.expects(:[]).with(:responsefile).returns nil
+
+    @provider.expects(:info)
+    @provider.expects(:preseed).never
+
+    @provider.run_preseed
+  end
+
+  it "should fail if a cdrom is listed in the sources list and :allowcdrom is not specified"
+
+  describe "when installing" do
+    it "should preseed if a responsefile is provided" do
+      @resource.expects(:[]).with(:responsefile).returns "/my/file"
+      @provider.expects(:run_preseed)
+
+      @provider.stubs(:aptget)
+      @provider.install
+    end
+
+    it "should check for a cdrom" do
+      @provider.expects(:checkforcdrom)
+
+      @provider.stubs(:aptget)
+      @provider.install
+    end
+
+    it "should use 'apt-get install' with the package name if no version is asked for" do
+      @resource.expects(:[]).with(:ensure).returns :installed
+      @provider.expects(:aptget).with { |*command| command[-1] == "asdf" and command[-2] == :install }
+
+      @provider.install
+    end
+
+    it "should specify the package version if one is asked for" do
+      @resource.expects(:[]).with(:ensure).returns "1.0"
+      @provider.expects(:aptget).with { |*command| command[-1] == "asdf=1.0" }
+
+      @provider.install
+    end
+
+    it "should use --force-yes if a package version is specified" do
+      @resource.expects(:[]).with(:ensure).returns "1.0"
+      @provider.expects(:aptget).with { |*command| command.include?("--force-yes") }
+
+      @provider.install
+    end
+
+    it "should do a quiet install" do
+      @provider.expects(:aptget).with { |*command| command.include?("-q") }
+
+      @provider.install
+    end
+
+    it "should default to 'yes' for all questions" do
+      @provider.expects(:aptget).with { |*command| command.include?("-y") }
+
+      @provider.install
+    end
+
+    it "should keep config files if asked" do
+      @resource.expects(:[]).with(:configfiles).returns :keep
+      @provider.expects(:aptget).with { |*command| command.include?("DPkg::Options::=--force-confold") }
+
+      @provider.install
+    end
+
+    it "should replace config files if asked" do
+      @resource.expects(:[]).with(:configfiles).returns :replace
+      @provider.expects(:aptget).with { |*command| command.include?("DPkg::Options::=--force-confnew") }
+
+      @provider.install
+    end
+  end
+end

data/spec/unit/provider/package/dpkg_spec.rb

@@ -0,0 +1,225 @@
+#!/usr/bin/env ruby
+
+Dir.chdir(File.dirname(__FILE__)) { (s = lambda { |f| File.exist?(f) ? require(f) : Dir.chdir("..") { s.call(f) } }).call("spec/spec_helper.rb") }
+
+provider = Puppet::Type.type(:package).provider(:dpkg)
+
+describe provider do
+  before do
+    @resource = stub 'resource', :[] => "asdf"
+    @provider = provider.new(@resource)
+    @provider.expects(:execute).never # forbid "manual" executions
+
+    @fakeresult = "install ok installed asdf 1.0\n"
+  end
+
+  it "should have documentation" do
+    provider.doc.should be_instance_of(String)
+  end
+
+  describe "when listing all instances" do
+    before do
+      provider.stubs(:command).with(:dpkgquery).returns "myquery"
+    end
+
+    it "should use dpkg-query" do
+      provider.expects(:command).with(:dpkgquery).returns "myquery"
+      provider.expects(:execpipe).with("myquery -W --showformat '${Status} ${Package} ${Version}\\n'").returns @fakeresult
+
+      provider.instances
+    end
+
+    it "should create and return an instance with each parsed line from dpkg-query" do
+      pipe = mock 'pipe'
+      pipe.expects(:each).yields @fakeresult
+      provider.expects(:execpipe).yields pipe
+
+      asdf = mock 'pkg1'
+      provider.expects(:new).with(:ensure => "1.0", :error => "ok", :desired => "install", :name => "asdf", :status => "installed", :provider => :dpkg).returns asdf
+
+      provider.instances.should == [asdf]
+    end
+
+    it "should warn on and ignore any lines it does not understand" do
+      pipe = mock 'pipe'
+      pipe.expects(:each).yields "foobar"
+      provider.expects(:execpipe).yields pipe
+
+      Puppet.expects(:warning)
+      provider.expects(:new).never
+
+      provider.instances.should == []
+    end
+  end
+
+  describe "when querying the current state" do
+    it "should use dpkg-query" do
+      @provider.expects(:dpkgquery).with("-W", "--showformat",'${Status} ${Package} ${Version}\\n', "asdf").returns @fakeresult
+
+      @provider.query
+    end
+
+    it "should consider the package purged if dpkg-query fails" do
+      @provider.expects(:dpkgquery).raises Puppet::ExecutionFailure.new("eh")
+
+      @provider.query[:ensure].should == :purged
+    end
+
+    it "should return a hash of the found status with the desired state, error state, status, name, and 'ensure'" do
+      @provider.expects(:dpkgquery).returns @fakeresult
+
+      @provider.query.should == {:ensure => "1.0", :error => "ok", :desired => "install", :name => "asdf", :status => "installed", :provider => :dpkg}
+    end
+
+    it "should consider the package absent if the dpkg-query result cannot be interpreted" do
+      @provider.expects(:dpkgquery).returns "somebaddata"
+
+      @provider.query[:ensure].should == :absent
+    end
+
+    it "should fail if an error is discovered" do
+      @provider.expects(:dpkgquery).returns @fakeresult.sub("ok", "error")
+
+      lambda { @provider.query }.should raise_error(Puppet::Error)
+    end
+
+    it "should consider the package purged if it is marked 'not-installed'" do
+      @provider.expects(:dpkgquery).returns @fakeresult.sub("installed", "not-installed")
+
+      @provider.query[:ensure].should == :purged
+    end
+
+    it "should consider the package absent if it is marked 'config-files'" do
+      @provider.expects(:dpkgquery).returns @fakeresult.sub("installed", "config-files")
+      @provider.query[:ensure].should == :absent
+    end
+
+    it "should consider the package absent if it is marked 'half-installed'" do
+      @provider.expects(:dpkgquery).returns @fakeresult.sub("installed", "half-installed")
+      @provider.query[:ensure].should == :absent
+    end
+
+    it "should consider the package absent if it is marked 'unpacked'" do
+      @provider.expects(:dpkgquery).returns @fakeresult.sub("installed", "unpacked")
+      @provider.query[:ensure].should == :absent
+    end
+
+    it "should consider the package absent if it is marked 'half-configured'" do
+      @provider.expects(:dpkgquery).returns @fakeresult.sub("installed", "half-configured")
+      @provider.query[:ensure].should == :absent
+    end
+
+    it "should consider the package held if its state is 'hold'" do
+      @provider.expects(:dpkgquery).returns @fakeresult.sub("install", "hold")
+      @provider.query[:ensure].should == :held
+    end
+  end
+
+  it "should be able to install" do
+    @provider.should respond_to(:install)
+  end
+
+  describe "when installing" do
+    before do
+      @resource.stubs(:[]).with(:source).returns "mypkg"
+    end
+
+    it "should fail to install if no source is specified in the resource" do
+      @resource.expects(:[]).with(:source).returns nil
+
+      lambda { @provider.install }.should raise_error(ArgumentError)
+    end
+
+    it "should use 'dpkg -i' to install the package" do
+      @resource.expects(:[]).with(:source).returns "mypackagefile"
+      @provider.expects(:unhold)
+      @provider.expects(:dpkg).with { |*command| command[-1] == "mypackagefile"  and command[-2] == "-i" }
+
+      @provider.install
+    end
+
+    it "should keep old config files if told to do so" do
+      @resource.expects(:[]).with(:configfiles).returns :keep
+      @provider.expects(:unhold)
+      @provider.expects(:dpkg).with { |*command| command[0] == "--force-confold" }
+
+      @provider.install
+    end
+
+    it "should replace old config files if told to do so" do
+      @resource.expects(:[]).with(:configfiles).returns :replace
+      @provider.expects(:unhold)
+      @provider.expects(:dpkg).with { |*command| command[0] == "--force-confnew" }
+
+      @provider.install
+    end
+
+    it "should ensure any hold is removed" do
+      @provider.expects(:unhold).once
+      @provider.expects(:dpkg)
+      @provider.install
+    end
+  end
+
+  describe "when holding or unholding" do
+    before do
+      @tempfile = stub 'tempfile', :print => nil, :close => nil, :flush => nil, :path => "/other/file"
+      @tempfile.stubs(:write)
+      Tempfile.stubs(:new).returns @tempfile
+    end
+
+    it "should install first if holding" do
+      @provider.stubs(:execute)
+      @provider.expects(:install).once
+      @provider.hold
+    end
+
+    it "should execute dpkg --set-selections when holding" do
+      @provider.stubs(:install)
+      @provider.expects(:execute).with([:dpkg, '--set-selections'], {:stdinfile => @tempfile.path}).once
+      @provider.hold
+    end
+
+    it "should execute dpkg --set-selections when unholding" do
+      @provider.stubs(:install)
+      @provider.expects(:execute).with([:dpkg, '--set-selections'], {:stdinfile => @tempfile.path}).once
+      @provider.hold
+    end
+  end
+
+  it "should use :install to update" do
+    @provider.expects(:install)
+    @provider.update
+  end
+
+  describe "when determining latest available version" do
+    it "should return the version found by dpkg-deb" do
+      @resource.expects(:[]).with(:source).returns "myfile"
+      @provider.expects(:dpkg_deb).with { |*command| command[-1] == "myfile" }.returns "asdf\t1.0"
+      @provider.latest.should == "1.0"
+    end
+
+    it "should warn if the package file contains a different package" do
+      @provider.expects(:dpkg_deb).returns("foo\tversion")
+      @provider.expects(:warning)
+      @provider.latest
+    end
+
+    it "should cope with names containing ++" do
+      @resource = stub 'resource', :[] => "asdf++"
+      @provider = provider.new(@resource)
+      @provider.expects(:dpkg_deb).returns "asdf++\t1.0"
+      @provider.latest.should == "1.0"
+    end
+  end
+
+  it "should use 'dpkg -r' to uninstall" do
+    @provider.expects(:dpkg).with("-r", "asdf")
+    @provider.uninstall
+  end
+
+  it "should use 'dpkg --purge' to purge" do
+    @provider.expects(:dpkg).with("--purge", "asdf")
+    @provider.purge
+  end
+end