add schema_file param to port389::instance define

This param controls SchemaFile entrie(s) in the setup.inf file.

manifests/instance.pp

@@ -7,6 +7,7 @@
   $root_dn_pwd                = $::port389::root_dn_pwd,
   $server_port                = $::port389::server_port,
   $suffix                     = port389_domain2dn($::port389::admin_domain),
+  $schema_file                = undef,
 ) {
   # follow the same server identifier validation rules as setup-ds-admin.pl
   validate_re($title, '^[\w#%:@-]*$', "The ServerIdentifier '${title}' contains invalid characters.  It must contain only alphanumeric characters and the following: #%:@_-")
@@ -49,13 +50,13 @@
       'InstallLdifFile'  => '',
       'RootDN'           => $root_dn,
       'RootDNPwd'        => $root_dn_pwd,
+      'SchemaFile'       => $schema_file,
       'ServerIdentifier' => $title,
       'ServerPort'       => $server_port,
       'SlapdConfigForMC' => 'yes',
       'Suffix'           => $suffix,
       'UseExistingMC'    => '0',
       'ds_bename'        => 'userRoot',
-      #'SchemaFile'       => [],
       #'bak_dir' => '/var/lib/dirsrv/slapd-ldap1/bak',
       #'bindir' => '/usr/bin',
       #'cert_dir' => '/etc/dirsrv/slapd-ldap1',

spec/defines/port389_instance_spec.rb

@@ -78,6 +78,44 @@
         :logoutput => true,
       })
     end
+
+    context 'schema_file =>' do
+      context '/dne/foo.ldif' do
+        let(:params) {{ :schema_file => '/dne/foo.ldif' }}
+
+        it do
+          should contain_file('setup_ldap1.inf').with({
+            :ensure  => 'file',
+            :path    => '/var/lib/dirsrv/setup/setup_ldap1.inf',
+            :owner   => 'nobody',
+            :group   => 'nobody',
+            :mode    => '0600',
+            :backup  => false,
+            :content => %r{SchemaFile=/dne/foo.ldif},
+          })
+        end
+      end
+
+      context '[ /dne/foo.ldif, /dne/bar.ldif ]' do
+        files = %w{ /dne/foo.ldif /dne/bar.ldif }
+        let(:params) {{ :schema_file => files }}
+
+        files.each do |f|
+          it do
+            should contain_file('setup_ldap1.inf').with({
+              :ensure  => 'file',
+              :path    => '/var/lib/dirsrv/setup/setup_ldap1.inf',
+              :owner   => 'nobody',
+              :group   => 'nobody',
+              :mode    => '0600',
+              :backup  => false,
+              :content => /SchemaFile=#{files}/
+            })
+          end
+        end
+      end
+
+    end # schema_file =>
   end
 
 end

templates/inf.erb

@@ -2,7 +2,9 @@
 [<%= section %>]
   <%- mysection = @data[section] -%>
   <%- mysection.keys.sort.each do |key| -%>
-    <%- next if mysection[key].nil? -%>
+    <%- # at least puppet 3.4.2 leaks :undef-%>
+    <%- # https://tickets.puppetlabs.com/browse/PUP-1467 -%>
+    <%- next if mysection[key].nil? || mysection[key] == :undef -%>
     <%- if mysection[key].is_a?(Array) -%>
       <%- mysection[key].each do |v| -%>
 <%= key %>=<%= v %>