Fixing ldap nodes -- they were always returning true because i was returning an empty array for missing nodes.

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

1 files changed, 17 insertions, 1 deletions

diff --git a/test/language/interpreter.rb b/test/language/interpreter.rb
index e195e28b3..755a07aa8 100755
--- a/test/language/interpreter.rb
+++ b/test/language/interpreter.rb
@@ -151,9 +151,17 @@ class TestInterpreter < Test::Unit::TestCase
 
         interp = mkinterp :NodeSources => [:ldap, :code]
 
-        # Make sure we can find 'culain' in ldap
+        # Make sure we get nil and nil back when we search for something missing
         parent, classes = nil
         assert_nothing_raised do
+            parent, classes = interp.ldapsearch("nosuchhost")
+        end
+
+        assert_nil(parent, "Got a parent for a non-existent host")
+        assert_nil(classes, "Got classes for a non-existent host")
+
+        # Make sure we can find 'culain' in ldap
+        assert_nothing_raised do
             parent, classes = interp.ldapsearch("culain")
         end
 
@@ -173,6 +181,14 @@ class TestInterpreter < Test::Unit::TestCase
         # culain uses basenode, so create that
         basenode = interp.newnode([:basenode])[0]
 
+        # Make sure we get nothing for nonexistent hosts
+        none = nil
+        assert_nothing_raised do
+            none = interp.nodesearch_ldap("nosuchhost")
+        end
+
+        assert_nil(none, "Got a node for a non-existent host")
+
         # Make sure we can find 'culain' in ldap
         culain = nil
         assert_nothing_raised do