diff options
| author | Luke Kanies <luke@madstop.com> | 2007-10-04 16:55:15 -0500 |
|---|---|---|
| committer | Luke Kanies <luke@madstop.com> | 2007-10-04 16:55:15 -0500 |
| commit | a93db8728ddc9a4b26dec738ff39863666ea229e (patch) | |
| tree | edcfef3751e41451b51961e1cf17b7d9a8fdcdc5 /lib | |
| parent | 9984a3520dd061acec98495ddd5a21b6126376f4 (diff) | |
| download | puppet-a93db8728ddc9a4b26dec738ff39863666ea229e.tar.gz puppet-a93db8728ddc9a4b26dec738ff39863666ea229e.tar.xz puppet-a93db8728ddc9a4b26dec738ff39863666ea229e.zip | |
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.
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/puppet/indirector/ldap/node.rb | 7 |
1 files changed, 5 insertions, 2 deletions
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| |