Add missing tests for RHEL 8

puppetlabs · Oct 26, 2020 · 985d58c · 985d58c
1 parent 3877dd9
commit 985d58c
Show file tree

Hide file tree

Showing 2 changed files with 42 additions and 0 deletions.
diff --git a/spec/classes/mod/alias_spec.rb b/spec/classes/mod/alias_spec.rb
@@ -53,6 +53,22 @@
       it { is_expected.to contain_apache__mod('alias') }
       it { is_expected.to contain_file('alias.conf').with(content: %r{Alias \/icons\/ "\/usr\/share\/httpd\/icons\/"}) }
     end
+    context 'on a RedHat 8-based OS', :compile do
+      let :facts do
+        {
+          id: 'root',
+          kernel: 'Linux',
+          osfamily: 'RedHat',
+          operatingsystem: 'RedHat',
+          operatingsystemrelease: '8',
+          path: '/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin',
+          is_pe: false,
+        }
+      end
+
+      it { is_expected.to contain_apache__mod('alias') }
+      it { is_expected.to contain_file('alias.conf').with(content: %r{Alias \/icons\/ "\/usr\/share\/httpd\/icons\/"}) }
+    end
     context 'with icons options', :compile do
       let :pre_condition do
         'class { apache: default_mods => false }'

diff --git a/spec/classes/mod/status_spec.rb b/spec/classes/mod/status_spec.rb
@@ -200,6 +200,32 @@ def status_conf_spec_require(requires, extended_status, status_path)
 
         it { is_expected.to contain_file('status.conf').with_path('/etc/httpd/conf.modules.d/status.conf') }
       end
+
+      context "on a RedHat 8 OS with default params and #{req_key} requires" do
+        let :facts do
+          {
+            osfamily: 'RedHat',
+            operatingsystemrelease: '8',
+            operatingsystem: 'RedHat',
+            id: 'root',
+            kernel: 'Linux',
+            path: '/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin',
+            is_pe: false,
+          }
+        end
+
+        let :params do
+          {
+            requires: req_value,
+          }
+        end
+
+        it { is_expected.to contain_apache__mod('status') }
+
+        status_conf_spec_require(req_value, 'On', '/server-status')
+
+        it { is_expected.to contain_file('status.conf').with_path('/etc/httpd/conf.modules.d/status.conf') }
+      end
     end
 
     context "with custom parameters $allow_from => ['10.10.10.10','11.11.11.11'], $extended_status => 'Off', $status_path => '/custom-status'" do