[Pkg-puppet-devel] [SCM] Puppet packaging for Debian branch, upstream, updated. 0.25.5-639-g8f94f35

Luke Kanies luke at puppetlabs.com
Wed Jul 14 10:32:28 UTC 2010


The following commit has been merged in the upstream branch:
commit aee9c294035efd053d2de131a75867ae6bc4ddf7
Author: Luke Kanies <luke at puppetlabs.com>
Date:   Mon Apr 12 15:02:44 2010 -0700

    Fixing a warning in the aix package provider
    
    Signed-off-by: Luke Kanies <luke at puppetlabs.com>

diff --git a/lib/puppet/provider/package/aix.rb b/lib/puppet/provider/package/aix.rb
index b6f23c0..5cccdf3 100644
--- a/lib/puppet/provider/package/aix.rb
+++ b/lib/puppet/provider/package/aix.rb
@@ -23,7 +23,7 @@ Puppet::Type.type(:package).provide :aix, :parent => Puppet::Provider::Package d
 
     def self.prefetch(packages)
         if Process.euid != 0
-            raise Puppet::Error "The aix provider can only be used by root"
+            raise Puppet::Error, "The aix provider can only be used by root"
         end
 
         return unless packages.detect { |name, package| package.should(:ensure) == :latest }
diff --git a/test/ral/type/resources.rb b/test/ral/type/resources.rb
index adc75cb..2dd747e 100755
--- a/test/ral/type/resources.rb
+++ b/test/ral/type/resources.rb
@@ -59,71 +59,6 @@ class TestResources < Test::Unit::TestCase
         @type = Puppet::Type.type(:resources)
     end
 
-    def test_purge
-        # Create a purgeable type
-        mkpurgertype
-
-        purger = nil
-        assert_nothing_raised do
-            purger = @type.new :name => "purgetest", :noop => true, :loglevel => :warning
-        end
-        purger.catalog = Puppet::Resource::Catalog.new
-        assert(purger, "did not get purger manager")
-        add_purge_lister()
-
-        assert_equal($purgemembers.values.sort, @purgetype.instances.sort)
-
-        # and it should now succeed
-        assert_nothing_raised do
-            purger[:purge] = true
-        end
-        assert(purger.purge?, "purge boolean was not enabled")
-
-        # Okay, now let's try doing some purging, yo
-        managed = []
-        unmanned = []
-        3.times { managed << mk_purger(true) } # 3 managed
-        3.times { unmanned << mk_purger(false) } # 3 unmanaged
-
-        managed.each do |m|
-            assert(m.managed?, "managed resource was not considered managed")
-        end
-        unmanned.each do |u|
-            assert(! u.managed?, "unmanaged resource was considered managed")
-        end
-
-        # First make sure we get nothing back when purge is false
-        genned = nil
-        purger[:purge] = false
-        assert_nothing_raised do
-            genned = purger.generate
-        end
-        assert_equal([], genned, "Purged even when purge is false")
-
-        # Now make sure we can purge
-        purger[:purge] = true
-        assert_nothing_raised do
-            genned = purger.generate
-        end
-        assert(genned, "Did not get any generated resources")
-
-        genned.each do |res|
-            assert(res.purging, "did not mark resource for purging")
-        end
-        assert(! genned.empty?, "generated resource list was empty")
-
-        # Now make sure the generate method only finds the unmanaged resources
-        assert_equal(unmanned.collect { |r| r.title }.sort, genned.collect { |r| r.title },
-            "Did not return correct purge list")
-
-        # Now make sure our metaparams carried over
-        genned.each do |res|
-            [:noop, :loglevel].each do |param|
-                assert_equal(purger[param], res[param], "metaparam %s did not carry over" % param)
-            end
-        end
-    end
-
     # Part of #408.
     def test_check
         # First check a non-user

-- 
Puppet packaging for Debian



More information about the Pkg-puppet-devel mailing list