feat: use semicolon as delimiter when super.users config is passed as array

manifests/broker/config.pp

@@ -31,7 +31,7 @@
     owner   => $user_name,
     group   => $group_name,
     mode    => $config_mode,
-    content => template('kafka/properties.erb'),
+    content => epp('kafka/properties.epp', { 'doctag' => $doctag, 'config' => $config }),
     notify  => $config_notify,
     require => File[$config_dir],
   }

manifests/consumer/config.pp

@@ -28,7 +28,7 @@
     owner   => $user_name,
     group   => $group_name,
     mode    => $config_mode,
-    content => template('kafka/properties.erb'),
+    content => epp('kafka/properties.epp', { 'doctag' => $doctag, 'config' => $config }),
     notify  => $config_notify,
     require => File[$config_dir],
   }

manifests/producer/config.pp

@@ -25,7 +25,7 @@
     owner   => $user_name,
     group   => $group_name,
     mode    => $config_mode,
-    content => template('kafka/properties.erb'),
+    content => epp('kafka/properties.epp', { 'doctag' => $doctag, 'config' => $config }),
     notify  => $config_notify,
     require => File[$config_dir],
   }

spec/classes/broker_spec.rb

@@ -49,7 +49,40 @@
 
       describe 'kafka::broker::config' do
         context 'defaults' do
-          it { is_expected.to contain_file('/opt/kafka/config/server.properties') }
+          it {
+            is_expected.to contain_file('/opt/kafka/config/server.properties').
+              with_content(%r{zookeeper.connect=localhost:2181})
+          }
+        end
+
+        context 'array values' do
+          let(:params) do
+            {
+              'config' => {
+                'listeners' => ['PLAINTEXT://:9092', 'SSL://:9093'],
+              }
+            }
+          end
+
+          it {
+            is_expected.to contain_file('/opt/kafka/config/server.properties').
+              with_content(%r{^listeners=PLAINTEXT://:9092,SSL://:9093$})
+          }
+        end
+
+        context 'special delimiter for super.users' do
+          let(:params) do
+            {
+              'config' => {
+                'super.users' => ['User:Alice', 'User:Bob'],
+              }
+            }
+          end
+
+          it {
+            is_expected.to contain_file('/opt/kafka/config/server.properties').
+              with_content(%r{^super.users=User:Alice;User:Bob$})
+          }
         end
 
         context 'with manage_log4j => true' do

templates/properties.epp

@@ -0,0 +1,14 @@
+<%- |
+String $doctag,
+Hash $config
+| -%>
+#
+# Note: This file is managed by Puppet.
+#
+# See: http://kafka.apache.org/documentation.html#<%= $doctag %> for more details
+#
+<% $config.keys.sort.each |$k| { -%>
+<%   $v = $config[$k] -%>
+<%   if $k == 'super.users' { $delimiter = ';' } else { $delimiter = ',' } -%>
+<%=    $k %>=<%= if $v =~ Array {  $v.join($delimiter)  } else { $v } %>
+<% } -%>