Revert "Ensure config file ends with a new line"

This reverts commit e87d8f2.

Author: Simon Peeters

manifests/config.pp

@@ -81,17 +81,15 @@
   -> case $facts['os']['family'] {
     'Windows': {
       concat { "${puppet_dir}/puppet.conf":
-        mode           => '0674',
-        ensure_newline => true,
+        mode  => '0674',
       }
     }
 
     default: {
       concat { "${puppet_dir}/puppet.conf":
-        owner          => 'root',
-        group          => $puppet::params::root_group,
-        mode           => '0644',
-        ensure_newline => true,
+        owner => 'root',
+        group => $puppet::params::root_group,
+        mode  => '0644',
       }
     }
   }

manifests/config/entry.pp

@@ -30,7 +30,7 @@
   # they make sure that '1_main ' is ordered before '1_main_*'
   ensure_resource('concat::fragment', "puppet.conf_${section}", {
       target  => "${puppet::dir}/puppet.conf",
-      content => "\n[${section}]",
+      content => "\n\n[${section}]",
       order   => "${sectionorder}_${section} ",
   })
 
@@ -39,7 +39,7 @@
   if (!defined(Concat::Fragment["puppet.conf_${section}_${key}"])){
     concat::fragment{"puppet.conf_${section}_${key}":
       target  => "${puppet::dir}/puppet.conf",
-      content => "    ${key} = ${_value}",
+      content => "\n    ${key} = ${_value}",
       order   => "${sectionorder}_${section}_${key} ",
     }
   } else {

spec/defines/puppet_config_entry_spec.rb

@@ -20,7 +20,7 @@
           }
         end
         it 'should contain the section header' do
-          should contain_concat__fragment('puppet.conf_main').with_content("\n[main]")
+          should contain_concat__fragment('puppet.conf_main').with_content("\n\n[main]")
           should contain_concat__fragment('puppet.conf_main').with_order("1_main ")
         end
         it 'should contain the keyvalue pair' do
@@ -41,7 +41,7 @@
           }
         end
         it 'should contain the section header' do
-          should contain_concat__fragment('puppet.conf_main').with_content("\n[main]")
+          should contain_concat__fragment('puppet.conf_main').with_content("\n\n[main]")
           should contain_concat__fragment('puppet.conf_main').with_order("1_main ")
         end
         it 'should contain the keyvalue pair' do
@@ -63,7 +63,7 @@
           }
         end
         it 'should contain the section header' do
-          should contain_concat__fragment('puppet.conf_main').with_content("\n[main]")
+          should contain_concat__fragment('puppet.conf_main').with_content("\n\n[main]")
           should contain_concat__fragment('puppet.conf_main').with_order("1_main ")
         end
         it 'should contain the keyvalue pair' do