]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commitdiff
use prefix instead of include in acceptance tests
authortphoney <tp@puppet.com>
Tue, 5 Mar 2019 16:50:54 +0000 (16:50 +0000)
committertphoney <tp@puppet.com>
Tue, 5 Mar 2019 16:50:54 +0000 (16:50 +0000)
spec/acceptance/firewall_attributes_exceptions_spec.rb
spec/acceptance/firewall_iptmodules_spec.rb
spec/acceptance/firewall_tee_spec.rb
spec/acceptance/firewall_time_spec.rb

index a2956b20e93af13b4c59a830866bcfcdc846ef16..f85e031965767417a010eb15416f5b2c81d9a785 100644 (file)
@@ -405,7 +405,7 @@ describe 'firewall basics', docker: true do
 
   # iptables version 1.3.5 is not suppored by the ip6tables provider
   # iptables version 1.4.7 fails for multiple hl entries
-  describe 'testing ipv6', unless: (os[:family] == 'redhat' && ['5', '6'].include?(os[:release][0])) || (os[:family] == 'sles') do
+  describe 'testing ipv6', unless: (os[:family] == 'redhat' && os[:release].start_with?('5', '6')) || (os[:family] == 'sles') do
     describe 'hop_limit' do
       context 'when 5' do
         pp42 = <<-PUPPETCODE
@@ -1139,7 +1139,7 @@ describe 'firewall basics', docker: true do
           end
         end
 
-        context 'when multiple addrtype', unless: (os[:family] == 'redhat' && ['5', '6'].include?(os[:release][0])) do
+        context 'when multiple addrtype', unless: (os[:family] == 'redhat' && os[:release].start_with?('5', '6')) do
           pp105 = <<-PUPPETCODE
                 class { '::firewall': }
                 firewall { '620 - test':
index 6cea07360d3461225f8dc89090a7bbea2c4eca84..52baf2365a91f550a4664df8ac5a04b3aa42ee2b 100644 (file)
@@ -66,7 +66,7 @@ describe 'firewall iptmodules' do
     end
   end
 
-  describe 'ip6tables ipt_modules tests', unless: (os[:family] == 'redhat' && ['5', '6'].include?(os[:release][0])) || (os[:family] == 'sles') do
+  describe 'ip6tables ipt_modules tests', unless: (os[:family] == 'redhat' && os[:release].start_with?('5', '6')) || (os[:family] == 'sles') do
     context 'when all the modules with multiple args' do
       pp3 = <<-PUPPETCODE
               class { '::firewall': }
index 2fca0808802f6e562f56ac3adfc8f5cdb48f5058..29b00b43245fbafc06088ebd2658ecc9409f4dc1 100644 (file)
@@ -1,6 +1,6 @@
 require 'spec_helper_acceptance'
 
-describe 'firewall tee', unless: (os[:family] == 'redhat' && ['5', '6'].include?(os[:release][0])) || (os[:family] == 'sles') do
+describe 'firewall tee', unless: (os[:family] == 'redhat' && os[:release].start_with?('5', '6')) || (os[:family] == 'sles') do
   before :all do
     iptables_flush_all_tables
     ip6tables_flush_all_tables
index d1b6fb1f21758a30c99df3889bd00ed4433c78ab..89fcef773d618df3be86594dd0efae0a4de05534 100644 (file)
@@ -1,6 +1,6 @@
 require 'spec_helper_acceptance'
 
-describe 'firewall time', unless: (os[:family] == 'redhat' && ['5', '6'].include?(os[:release][0])) || (os[:family] == 'sles') do
+describe 'firewall time', unless: (os[:family] == 'redhat' && os[:release].start_with?('5', '6')) || (os[:family] == 'sles') do
   before :all do
     iptables_flush_all_tables
     ip6tables_flush_all_tables