Changing the realize() function to be just syntactic sugar for a collection -- it literally creates a collector object now. The benefit of this is that it is late-binding, so file order does not affect whether a resource is available.

git-svn-id: https://reductivelabs.com/svn/puppet/trunk@1810 980ebf18-57e1-0310-9a29-db15c13687c0

1 files changed, 20 insertions, 2 deletions

diff --git a/test/language/functions.rb b/test/language/functions.rb
index 21af606db..bd7143db6 100755
--- a/test/language/functions.rb
+++ b/test/language/functions.rb
@@ -321,19 +321,37 @@ class TestLangFunctions < Test::Unit::TestCase
             @scope.function_realize(ref)
         end
 
+        # Make sure it created a collection
+        assert_equal(1, @scope.collections.length,
+            "Did not set collection")
+
+        assert_nothing_raised do
+            @scope.collections.each do |coll| coll.evaluate end
+        end
+        @scope.collections.clear
+
         # Now make sure the virtual resource is no longer virtual
         assert(! virtual.virtual?, "Did not make virtual resource real")
 
         # Make sure we puke on any resource that doesn't exist
-
         none = Puppet::Parser::Resource::Reference.new(
             :type => "file", :title => "/tmp/nosuchfile",
             :scope => @scope
         )
 
-        assert_raise(Puppet::ParseError) do
+        # The function works
+        assert_nothing_raised do
             @scope.function_realize(none)
         end
+
+        # Make sure it created a collection
+        assert_equal(1, @scope.collections.length,
+            "Did not set collection")
+
+        # But the collection fails
+        assert_raise(Puppet::ParseError) do
+            @scope.collections.each do |coll| coll.evaluate end
+        end
     end
 end