]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commitdiff
firewall: tests: add `debian` tests for `ensure => stopped`
authorDamon Breeden <damon.breeden@ahrefs.com>
Fri, 6 Jan 2023 18:17:09 +0000 (18:17 +0000)
committerDamon Breeden <damon.breeden@ahrefs.com>
Fri, 6 Jan 2023 18:17:09 +0000 (18:17 +0000)
also `with Debian unstable, ensure => stopped`, seems to have been overlooked

spec/unit/classes/firewall_linux_debian_spec.rb

index 690144e99c9ed55e4f57fa505051adb0e2a3b3a5..6104f6aba29550d9393a0358c164360e5b7ca8b7 100644 (file)
@@ -43,6 +43,16 @@ describe 'firewall::linux::debian', type: :class do
     }
   end
 
+  context 'with Debian 10, ensure => stopped' do
+    let(:params) { { ensure: 'stopped' } }
+
+    it {
+      is_expected.to contain_service('netfilter-persistent').with(
+        ensure: 'stopped',
+      )
+    }
+  end
+
   context 'with Debian 11' do
     let(:facts) do
       {
@@ -83,6 +93,16 @@ describe 'firewall::linux::debian', type: :class do
     }
   end
 
+  context 'with Debian 11, ensure => stopped' do
+    let(:params) { { ensure: 'stopped' } }
+
+    it {
+      is_expected.to contain_service('netfilter-persistent').with(
+        ensure: 'stopped',
+      )
+    }
+  end
+
   context 'with Debian unstable' do
     let(:facts) do
       {
@@ -105,4 +125,31 @@ describe 'firewall::linux::debian', type: :class do
       )
     }
   end
+
+  context 'with Debian unstable, enable => false' do
+    let(:facts) do
+      {
+        osfamily: 'Debian',
+        operatingsystem: 'Debian',
+        operatingsystemrelease: 'unstable',
+      }
+    end
+    let(:params) { { enable: 'false' } }
+
+    it {
+      is_expected.to contain_service('netfilter-persistent').with(
+        enable: 'false',
+      )
+    }
+  end
+
+  context 'with Debian unstable, ensure => stopped' do
+    let(:params) { { ensure: 'stopped' } }
+
+    it {
+      is_expected.to contain_service('netfilter-persistent').with(
+        ensure: 'stopped',
+      )
+    }
+  end
 end