From 3e13bd59689a27a393c847bdbed3ac38765d79e9 Mon Sep 17 00:00:00 2001
From: Luke Kanies <luke@madstop.com>
Date: Tue, 20 May 2008 10:13:33 -0500
Subject: Intermediate commit so I can move on to other things.

---
 lib/puppet/provider/group/ldap.rb |  2 +-
 lib/puppet/provider/user/ldap.rb  |  2 +-
 spec/unit/provider/user/ldap.rb   | 10 ++++++++++
 3 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/lib/puppet/provider/group/ldap.rb b/lib/puppet/provider/group/ldap.rb
index 632358ff1..5af400a4e 100644
--- a/lib/puppet/provider/group/ldap.rb
+++ b/lib/puppet/provider/group/ldap.rb
@@ -12,7 +12,7 @@ Puppet::Type.type(:group).provide :ldap, :parent => Puppet::Provider::Ldap do
         as it iterates across all existing groups to pick the appropriate next
         one."
 
-    confine :true => Puppet.features.ldap?
+    confine :true => Puppet.features.ldap?, :false => (Puppet[:ldapuser] == "")
 
     # We're mapping 'members' here because we want to make it
     # easy for the ldap user provider to manage groups.  This
diff --git a/lib/puppet/provider/user/ldap.rb b/lib/puppet/provider/user/ldap.rb
index ba91a871e..d670ad435 100644
--- a/lib/puppet/provider/user/ldap.rb
+++ b/lib/puppet/provider/user/ldap.rb
@@ -12,7 +12,7 @@ Puppet::Type.type(:user).provide :ldap, :parent => Puppet::Provider::Ldap do
         as it iterates across all existing users to pick the appropriate next
         one."
 
-    confine :true => Puppet.features.ldap?
+    confine :feature => :ldap, :false => (Puppet[:ldapuser] == "")
 
     manages(:posixAccount, :person).at("ou=People").named_by(:uid).and.maps :name => :uid,
         :password => :userPassword,
diff --git a/spec/unit/provider/user/ldap.rb b/spec/unit/provider/user/ldap.rb
index c4731cbbb..eb13d8bbd 100755
--- a/spec/unit/provider/user/ldap.rb
+++ b/spec/unit/provider/user/ldap.rb
@@ -24,6 +24,16 @@ describe provider_class do
         provider_class.manager.rdn.should == :uid
     end
 
+    it "should be unsuitable if ldap is unavailable" do
+        Puppet.features.expects(:ldap?).returns false
+        provider_class.should_not be_suitable
+    end
+
+    it "should be suitable if ldap is available" do
+        Puppet.features.expects(:ldap?).returns true
+        provider_class.should be_suitable
+    end
+
     {:name => "uid",
         :password => "userPassword",
         :comment => "cn",
-- 
cgit