Merge pull request #1395 from wtfo-guru/support_absent_mailhosts

Actually use the ensure parameter for mailhost resource to provide absent support
voxpupuli · Jul 17, 2020 · f2cb723 · f2cb723
2 parents e4ba3e0 + 8262fa9
commit f2cb723
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/manifests/resource/mailhost.pp b/manifests/resource/mailhost.pp
@@ -152,6 +152,7 @@
   }
 
   concat { $config_file:
+    ensure  => $ensure,
     owner   => 'root',
     group   => $root_group,
     mode    => $nginx::global_mode,

diff --git a/spec/defines/resource_mailhost_spec.rb b/spec/defines/resource_mailhost_spec.rb
@@ -30,6 +30,13 @@
           it { is_expected.not_to contain_concat__fragment("#{title}-ssl") }
         end
 
+        describe 'absent assumption' do
+          let(:params) { default_params.merge('ensure'.to_sym => 'absent') }
+
+          it { is_expected.to contain_class('nginx') }
+          it { is_expected.to contain_concat("/etc/nginx/conf.mail.d/#{title}.conf").with('ensure' => 'absent') }
+        end
+
         describe 'mailhost template content' do
           [
             {