Merge branch 'release/1.2.1'

Author: ju5t

README.md

@@ -408,7 +408,7 @@ bundle exec rake spec_prep
 BEAKER_destroy=onpass bundle exec rake beaker:debian7
 BEAKER_destroy=onpass bundle exec rake beaker:debian8
 BEAKER_destroy=onpass bundle exec rake beaker:centos6
-BEAKER_destroy=onpass bundle exec rake beaker:centos7
+BEAKER_destroy=onpass BEAKER_PUPPET_COLLECTION=puppet5 bundle exec rake beaker:centos7
 BEAKER_destroy=onpass BEAKER_PUPPET_COLLECTION=puppet5 bundle exec rake beaker:centos8
 BEAKER_destroy=onpass bundle exec rake beaker:ubuntu1404
 BEAKER_destroy=onpass bundle exec rake beaker:ubuntu1604

manifests/install.pp

@@ -8,6 +8,11 @@
     }
   }
 
+  package { 'iptables':
+    ensure => installed,
+    before => Exec['csf-install'],
+  }
+
   package { 'csf-perl':
     ensure => installed,
     name   => 'perl',

metadata.json

@@ -1,6 +1,6 @@
 {
   "name": "sensson-csf",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "author": "sensson",
   "license": "Apache-2.0",
   "summary": "A module to manage ConfigServer Firewall and the Login Failure Daemon",

spec/classes/csf_install_spec.rb

@@ -23,6 +23,8 @@
           it { is_expected.to contain_exec('csf-install').with('notify' => 'Service[csf]') }
           it { is_expected.to contain_exec('csf-install').with('require' => 'Package[csf-perl]') }
 
+          it { is_expected.to contain_package('iptables').with('ensure' => 'installed') }
+
           if facts[:operatingsystem] == 'CentOS' && facts[:operatingsystemmajrelease].to_i < 7
             it { is_expected.to contain_package('iptables-ipv6').with('ensure' => 'installed') }
             it { is_expected.to contain_package('iptables-ipv6').with('before' => 'Exec[csf-install]') }