From a93db8728ddc9a4b26dec738ff39863666ea229e Mon Sep 17 00:00:00 2001 From: Luke Kanies Date: Thu, 4 Oct 2007 16:55:15 -0500 Subject: Adding another test to the ldap node source -- we make sure we throw an appropriate exception if a parent is specified but we cannot find it. --- lib/puppet/indirector/ldap/node.rb | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'lib/puppet') diff --git a/lib/puppet/indirector/ldap/node.rb b/lib/puppet/indirector/ldap/node.rb index 6f35b575c..2e800abbe 100644 --- a/lib/puppet/indirector/ldap/node.rb +++ b/lib/puppet/indirector/ldap/node.rb @@ -21,8 +21,11 @@ class Puppet::Indirector::Ldap::Node < Puppet::Indirector::Ldap raise ArgumentError, "Found loop in LDAP node parents; %s appears twice" % parent end parents << parent - ldapsearch(parent) do |entry| - parent_info = process(parent, entry) + + ldapsearch(parent) { |entry| parent_info = process(parent, entry) } + + unless parent_info + raise Puppet::Error.new("Could not find parent node '%s'" % parent) end information[:classes] += parent_info[:classes] parent_info[:parameters].each do |param, value| -- cgit