Fixing the first part of #787. Not all collections were

being evaluated on the first pass because they were being
deleted from the collections list during evaluation, which caused
some to get skipped.  This commit fixes that problem, which helps
in the trivial cases where the collections are in the same scope.
I expect it's still broken for more complicated usages.

1 files changed, 25 insertions, 0 deletions

diff --git a/spec/unit/parser/compile.rb b/spec/unit/parser/compile.rb
index 5f239636b..3be7d1637 100755
--- a/spec/unit/parser/compile.rb
+++ b/spec/unit/parser/compile.rb
@@ -71,6 +71,31 @@ describe Puppet::Parser::Compile, " when evaluating classes" do
     end
 end
 
+describe Puppet::Parser::Compile, " when evaluating collections" do
+    before do
+        @node = stub 'node', :name => 'mynode'
+        @parser = stub 'parser', :version => "1.0"
+        @scope = stub 'scope', :source => mock("source")
+        @compile = Puppet::Parser::Compile.new(@node, @parser)
+    end
+
+    it "should evaluate each collection" do
+        2.times { |i|
+            coll = mock 'coll%s' % i
+            @compile.add_collection(coll)
+            
+            # This is the hard part -- we have to emulate the fact that
+            # collections delete themselves if they are done evaluating.
+            coll.expects(:evaluate).with do
+                @compile.delete_collection(coll)
+            end
+        }
+
+        @compile.class.publicize_methods(:evaluate_collections) { @compile.evaluate_collections }
+    end
+end
+
+
 describe Puppet::Parser::Compile, " when evaluating found classes" do
     before do
         @node = stub 'node', :name => 'mynode'