Fixing the interpreter to nodesearch across all listed names, just like is done in the manifests. Also fixing a comment in type.rb

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

1 files changed, 32 insertions, 0 deletions

diff --git a/test/language/interpreter.rb b/test/language/interpreter.rb
index c22bb73ba..51ee1e5f4 100755
--- a/test/language/interpreter.rb
+++ b/test/language/interpreter.rb
@@ -317,4 +317,36 @@ class nothernode {}
             end
         end
     end
+
+    # Make sure nodesearch uses all names, not just one.
+    def test_nodesearch_multiple_names
+        bucket = {}
+        Puppet::Parser::Interpreter.send(:define_method, "nodesearch_multifake") do |node|
+            if bucket[node]
+                return *bucket[node]
+            else
+                return nil, nil
+            end
+        end
+        manifest = tempfile()
+        File.open(manifest, "w") do |f| f.puts "" end
+        interp = nil
+        assert_nothing_raised {
+            interp = Puppet::Parser::Interpreter.new(
+                :Manifest => manifest,
+                :NodeSources => [:multifake]
+            )
+        }
+
+        bucket["name.domain.com"] = [:parent, [:classes]]
+
+        ret = nil
+
+        assert_nothing_raised do
+            assert_equal bucket["name.domain.com"],
+                interp.nodesearch("name", "name.domain.com")
+        end
+
+
+    end
 end