From: Josh Farwell <jfarwell@linuxfoundation.org>
Date: Wed, 17 Jun 2015 17:18:15 +0000 (-0700)
Subject: Added selinux and hyperkitty integration packages
X-Git-Url: https://gerrit.linuxfoundation.org/infra/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F62%2F162%2F4;p=puppet%2Fmodules%2Fmailman3.git

Added selinux and hyperkitty integration packages

Mailman-core requires extra packages for SELinux policy and
for connecting to Hyperkitty, including these by default.

Change-Id: I4248b1f8f11a63b085f1b780366d6feff52b63c5
Signed-off-by: Josh Farwell <jfarwell@linuxfoundation.org>
---

diff --git a/manifests/core/install.pp b/manifests/core/install.pp
index bc49dbb..638f6dc 100644
--- a/manifests/core/install.pp
+++ b/manifests/core/install.pp
@@ -51,6 +51,16 @@ class mailman3::core::install (
       require => Class['::mailman3::repo']
     }
 
+    package { 'mailman3-hyperkitty':
+      ensure  => present,
+      require => Class['::mailman3::repo']
+    }
+
+    package { 'mailman3-selinux':
+      ensure  => present,
+      require => Class['::mailman3::repo']
+    }
+
   }
 
 }
diff --git a/spec/classes/core__install_spec.rb b/spec/classes/core__install_spec.rb
index 4bf0735..c4fa73e 100644
--- a/spec/classes/core__install_spec.rb
+++ b/spec/classes/core__install_spec.rb
@@ -20,6 +20,8 @@ describe 'mailman3::core::install', :type => :class do
   # With good parameters, we should include repo install packages by default
   context 'with known good parameters' do
     it { is_expected.to contain_package('mailman3') }
+    it { is_expected.to contain_package('mailman3-hyperkitty') }
+    it { is_expected.to contain_package('mailman3-selinux') }
     it { is_expected.to contain_class('mailman3::repo') }
     it { is_expected.to contain_yumrepo('fedorapeople-mailman3') }
   end
@@ -35,6 +37,8 @@ describe 'mailman3::core::install', :type => :class do
       }
 
     it { is_expected.to_not contain_package('mailman3') }
+    it { is_expected.to_not contain_package('mailman3-hyperkitty') }
+    it { is_expected.to_not contain_package('mailman3-selinux') }
     it { is_expected.to_not contain_class('mailman3::repo') }
     it { is_expected.to_not contain_yumrepo('fedorapeople-mailman3') }