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 (file)
--- 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 (file)
--- 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') }