test: add test for puppet.conf mode

theforeman · Jan 11, 2023 · 1e32bbb · 1e32bbb
1 parent c7df060
commit 1e32bbb
Showing 1 changed file with 13 additions and 1 deletion.
diff --git a/spec/classes/puppet_init_spec.rb b/spec/classes/puppet_init_spec.rb
@@ -10,18 +10,22 @@
         puppet_concat    = '/usr/local/etc/puppet/puppet.conf'
         puppet_directory = '/usr/local/etc/puppet'
         puppet_package   = "puppet#{puppet_major}"
+        puppetconf_mode  = '0644'
       when 'windows'
         puppet_concat    = 'C:/ProgramData/PuppetLabs/puppet/etc/puppet.conf'
         puppet_directory = 'C:/ProgramData/PuppetLabs/puppet/etc'
         puppet_package   = 'puppet-agent'
+        puppetconf_mode  = '0674'
       when 'Archlinux'
         puppet_concat    = '/etc/puppetlabs/puppet/puppet.conf'
         puppet_directory = '/etc/puppetlabs/puppet'
         puppet_package   = 'puppet'
+        puppetconf_mode  = '0644'
       else
         puppet_concat    = '/etc/puppetlabs/puppet/puppet.conf'
         puppet_directory = '/etc/puppetlabs/puppet'
         puppet_package   = 'puppet-agent'
+        puppetconf_mode  = '0644'
       end
 
       let :facts do
@@ -34,7 +38,7 @@
         it { should contain_class('puppet::config') }
         it { should_not contain_class('puppet::server') }
         it { should contain_file(puppet_directory).with_ensure('directory') }
-        it { should contain_concat(puppet_concat) }
+        it { should contain_concat(puppet_concat).with_mode(puppetconf_mode) }
         it { should contain_package(puppet_package)
           .with_ensure('present')
           .with_install_options(nil)
@@ -83,6 +87,14 @@
         it { should contain_puppet__config__main('ca_port').with_value(8140) }
       end
 
+      describe 'with puppetconf_mode' do
+        let :params do {
+          :puppetconf_mode => '0640',
+        } end
+
+        it { should contain_concat(puppet_concat).with_mode('0640') }
+      end
+
       # compilation is broken due to paths
       context 'on non-windows', unless: facts[:osfamily] == 'windows' do
         describe 'with package_source => Httpurl' do