add missing $ldapserver param to pureftpd::config::ldap

Author: Joshua Hoblitt

manifests/config/ldap.pp

@@ -13,6 +13,7 @@
 #
 
 class pureftpd::config::ldap (
+  $ldapserver          = undef,
   $ldapport            = undef,
   $ldapbinddn          = undef,
   $ldapbindpw          = undef,
@@ -30,6 +31,7 @@
 
   # the complete list of options is defined in pure-ftpd/src/log_ldap_p.h
   $conf_options = [
+    'LDAPServer',
     'LDAPPort',
     'LDAPBindDN',
     'LDAPBindPW',

spec/classes/config_ldap_spec.rb

@@ -11,6 +11,16 @@
     end
   end
 
+  describe 'with ldapserver' do
+    let(:params) {{ :ldapserver => 'ldap.example.com' }}
+    it do
+      should contain_class('pureftpd::config::ldap') 
+      should contain_file('/etc/pure-ftpd/pureftpd-ldap.conf') \
+        .with_ensure('file') \
+        .with_content(/^LDAPServer          ldap.example.com/)
+    end
+  end
+
   describe 'with ldapport' do
     let(:params) {{ :ldapport => '389' }}
     it do
@@ -153,26 +163,28 @@
 
   describe 'with everything' do
     let(:params) {{
-      :ldapauthmethod => 'PASSWORD',
-      :ldapport => '389',
-      :ldapbinddn => 'cn=Manager,dc=c9x,dc=org',
-      :ldapbindpw => 'r00tPaSsw0rD',
-      :ldapbasedn => 'cn=Users,dc=c9x,dc=org',
-      :ldapfilter => '(&(objectClass=posixAccount)(uid=\L))',
-      :ldaphomedir => 'homeDirectory',
-      :ldapversion => '3',
-      :ldapdefaultuid => '100',
+      :ldapserver          => 'ldap.example.com',
+      :ldapauthmethod      => 'PASSWORD',
+      :ldapport            => '389',
+      :ldapbinddn          => 'cn=Manager,dc=c9x,dc=org',
+      :ldapbindpw          => 'r00tPaSsw0rD',
+      :ldapbasedn          => 'cn=Users,dc=c9x,dc=org',
+      :ldapfilter          => '(&(objectClass=posixAccount)(uid=\L))',
+      :ldaphomedir         => 'homeDirectory',
+      :ldapversion         => '3',
+      :ldapdefaultuid      => '100',
       :ldapforcedefaultuid => 'False',
-      :ldapdefaultgid => '100',
-      :ldapdefaultgid => '100',
+      :ldapdefaultgid      => '100',
+      :ldapdefaultgid      => '100',
       :ldapforcedefaultgid => 'False',
-      :ldapusetls => 'False',
-      :ldapauthmethod => 'PASSWORD',
+      :ldapusetls          => 'False',
+      :ldapauthmethod      => 'PASSWORD',
     }}
     it do
       should contain_class('pureftpd::config::ldap') 
       should contain_file('/etc/pure-ftpd/pureftpd-ldap.conf') \
         .with_ensure('file') \
+        .with_content(/^LDAPServer          ldap.example.com/) \
         .with_content(/^LDAPPort            389/) \
         .with_content(/^LDAPBindDN          cn=Manager,dc=c9x,dc=org/) \
         .with_content(/^LDAPBindPW          r00tPaSsw0rD/) \