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

test branch puppet-dev at googlegroups.com
Wed Jul 14 10:30:24 UTC 2010


The following commit has been merged in the upstream branch:
commit b19a0442ed31cedd3846b9e26c25b5ea4b7ff22f
Author: Jesse Wolfe <jes5199 at gmail.com>
Date:   Thu Mar 25 17:17:16 2010 -0700

    Some tests were leaking state when the test failed

diff --git a/test/language/parser.rb b/test/language/parser.rb
index 13f4d17..ca8ee75 100755
--- a/test/language/parser.rb
+++ b/test/language/parser.rb
@@ -499,6 +499,8 @@ file { "/tmp/yayness":
                 check.call(res, "multiresource")
             end
         end
+
+    ensure
         if Puppet.features.rails?
             Puppet[:storeconfigs] = false
             Puppet::Resource::Catalog.cache_class =  catalog_cache_class
@@ -535,6 +537,8 @@ file { "/tmp/yayness":
             assert_instance_of(AST::Collection, coll)
             assert_equal(form, coll.form)
         end
+
+    ensure
         if Puppet.features.rails?
             Puppet[:storeconfigs] = false
             Puppet::Resource::Catalog.cache_class =  catalog_cache_class
@@ -643,7 +647,7 @@ file { "/tmp/yayness":
             parser.parse %{define one::two { }}
         }
         assert(parser.definition("one::two"), "Could not find one::two with no namespace")
-        
+
         # Now try using the definition
         assert_nothing_raised("Could not parse fully-qualified definition usage") {
             parser.parse %{one::two { yayness: }}
@@ -1180,7 +1184,7 @@ file { "/tmp/yayness":
             result = parser.newclass "Yayness"
         end
         assert_equal(result, parser.find_hostclass("", "yayNess"))
-        
+
         assert_nothing_raised do
             result = parser.newdefine "FunTest"
         end
diff --git a/test/language/scope.rb b/test/language/scope.rb
index 671af71..8819de6 100755
--- a/test/language/scope.rb
+++ b/test/language/scope.rb
@@ -395,6 +395,7 @@ Host <<||>>"
                 assert(flat.find{|o| o.name == name }, "Did not find #{name}")
             end
         }
+    ensure
         Puppet[:storeconfigs] = false
         Puppet::Resource::Catalog.cache_class =  catalog_cache_class
         Puppet::Node::Facts.cache_class = facts_cache_class

-- 
Puppet packaging for Debian



More information about the Pkg-puppet-devel mailing list