(GH-C&T-7) Remove code specific to unsupported OSs

diff --git a/provision.yaml b/provision.yaml
index 43201c3dc594ae54cba367a43e45a570effdb3f4..f42a6473134c93022d4e34a5eab13c34cda207bb 100644 (file)
--- a/provision.yaml
+++ b/provision.yaml
@@ -16,7 +16,6 @@ travis_deb:
 travis_ub_6:
   provisioner: docker
   images:
-  - litmusimage/ubuntu:14.04
   - litmusimage/ubuntu:16.04
   - litmusimage/ubuntu:18.04
   - litmusimage/ubuntu:20.04

diff --git a/spec/acceptance/firewall_attributes_exceptions_spec.rb b/spec/acceptance/firewall_attributes_exceptions_spec.rb
index cf5f1449191fd0ae98c34ebdf23824396807f27c..08af13ca9b16b78bf7ee4df31e3f5801e3eb7480 100644 (file)
--- a/spec/acceptance/firewall_attributes_exceptions_spec.rb
+++ b/spec/acceptance/firewall_attributes_exceptions_spec.rb
@@ -1375,7 +1375,7 @@ describe 'firewall basics', docker: true do
   describe 'condition', condition_parameter_test: false do
     context 'is set' do
       pp = <<-PUPPETCODE
-        if $facts['os']['name'] == 'Ubuntu' and versioncmp($facts['os']['release']['full'], '14.04') > 0 {
+        if $facts['os']['name'] == 'Ubuntu' {
           firewall { '010 isblue ipv4':
             ensure    => 'present',
             condition => '! isblue',
@@ -1389,7 +1389,7 @@ describe 'firewall basics', docker: true do
       it 'applies' do
         apply_manifest(pp)
       end
-      if fetch_os_name == 'ubuntu' && os[:release].to_i > 14
+      if fetch_os_name == 'ubuntu'
         it 'contains the rule' do
           run_shell('iptables-save') do |r|
             expect(r.stdout).to match(%r{-A INPUT -i enp0s8 -p icmp -m condition ! --condition "isblue"  -m comment --comment "010 isblue ipv4" -j DROP})

diff --git a/spec/unit/classes/firewall_linux_spec.rb b/spec/unit/classes/firewall_linux_spec.rb
index 42391040aaf342ba6c2f00ffe8871c7d8cb3a2df..755de355261664f66c25719da7f52c657a2dccbc 100644 (file)
--- a/spec/unit/classes/firewall_linux_spec.rb
+++ b/spec/unit/classes/firewall_linux_spec.rb
@@ -28,7 +28,7 @@ describe 'firewall::linux', type: :class do
 
   ['Debian', 'Ubuntu'].each do |os|
     context "Debian Like: operatingsystem => #{os}" do
-      releases = ((os == 'Debian') ? ['8'] : ['14.04'])
+      releases = ((os == 'Debian') ? ['10'] : ['20.04'])
       releases.each do |osrel|
         let(:facts) do
           {

diff --git a/spec/unit/facter/iptables_persistent_version_spec.rb b/spec/unit/facter/iptables_persistent_version_spec.rb
index 04ff63f1cf44f3d358c853e58ae2052958c147ee..82af6ce305d6e728c947df5d058457487e26c1b0 100644 (file)
--- a/spec/unit/facter/iptables_persistent_version_spec.rb
+++ b/spec/unit/facter/iptables_persistent_version_spec.rb
@@ -6,12 +6,12 @@ describe 'Facter::Util::Fact iptables_persistent_version' do
   context 'when iptables-persistent applicable' do
     before(:each) { Facter.clear }
 
-    let(:dpkg_cmd) { "dpkg-query -Wf '${Version}' iptables-persistent 2>/dev/null" }
+    let(:dpkg_cmd) { "dpkg-query -Wf '${Version}' netfilter-persistent 2>/dev/null" }
 
     {
       'Ubuntu' => '0.5.3ubuntu2',
     }.each do |os, ver|
-      os_release = '14.04'
+      os_release = '20.04'
 
       describe "#{os} package installed" do
         before(:each) do
@@ -27,7 +27,7 @@ describe 'Facter::Util::Fact iptables_persistent_version' do
     describe 'Ubuntu package not installed' do
       before(:each) do
         allow(Facter.fact(:operatingsystem)).to receive(:value).and_return('Ubuntu')
-        allow(Facter.fact(:operatingsystemrelease)).to receive(:value).and_return('14.04')
+        allow(Facter.fact(:operatingsystemrelease)).to receive(:value).and_return('20.04')
         allow(Facter::Util::Resolution).to receive(:exec).with(dpkg_cmd)
                                                          .and_return(nil)
       end