Skip to content

Commit

Permalink
Merge pull request #2029 from sanfrancrisko/maint/IAC-785_IAC-787_tes…
Browse files Browse the repository at this point in the history
…t_backout

(maint) Restrict test exec platforms for accept tests affected by IAC-587, IAC-785 & IAC-787
  • Loading branch information
DavidS committed May 12, 2020
2 parents cacfb7c + 404ae13 commit c0d75cf
Show file tree
Hide file tree
Showing 3 changed files with 26 additions and 22 deletions.
40 changes: 20 additions & 20 deletions spec/acceptance/apache_parameters_spec.rb
Expand Up @@ -92,28 +92,28 @@ class { 'apache':
end
end

if os[:family] == 'debian'
describe 'conf_enabled => /etc/apache2/conf-enabled' do
pp = <<-MANIFEST
class { 'apache':
purge_configs => false,
conf_enabled => "/etc/apache2/conf-enabled"
}
MANIFEST
it 'applies cleanly' do
run_shell('touch /etc/apache2/conf-enabled/test.conf')
apply_manifest(pp, catch_failures: true)
end
# IAC-785: The Shibboleth mod does not seem to be configured correctly on Debian 10 systems. We should reenable
# this test on Debian 10 systems once the issue has been RCA'd and resolved.
describe 'conf_enabled => /etc/apache2/conf-enabled', if: os[:family] == 'debian' && os[:release].to_i < 10 do
pp = <<-MANIFEST
class { 'apache':
purge_configs => false,
conf_enabled => "/etc/apache2/conf-enabled"
}
MANIFEST
it 'applies cleanly' do
run_shell('touch /etc/apache2/conf-enabled/test.conf')
apply_manifest(pp, catch_failures: true)
end

# Ensure the created file didn't disappear.
describe file('/etc/apache2/conf-enabled/test.conf') do
it { is_expected.to be_file }
end
# Ensure the created file didn't disappear.
describe file('/etc/apache2/conf-enabled/test.conf') do
it { is_expected.to be_file }
end

# Ensure the default file didn't disappear.
describe file('/etc/apache2/conf-enabled/security.conf') do
it { is_expected.to be_file }
end
# Ensure the default file didn't disappear.
describe file('/etc/apache2/conf-enabled/security.conf') do
it { is_expected.to be_file }
end
end

Expand Down
4 changes: 3 additions & 1 deletion spec/acceptance/itk_spec.rb
Expand Up @@ -14,7 +14,9 @@
variant = :prefork
end

describe 'apache::mod::itk class', if: service_name do
# IAC-787: The http-itk mod package is not available in any of the standard RHEL/CentOS 8.x repos. Disable this test
# on those platforms until we can find a suitable source for this package.
describe 'apache::mod::itk class', if: service_name, unless: os[:family] == 'redhat' && os[:release].to_i >= 8 do
describe 'running puppet code' do
let(:pp) do
case variant
Expand Down
4 changes: 3 additions & 1 deletion spec/acceptance/vhost_spec.rb
Expand Up @@ -1198,7 +1198,9 @@ class { 'apache::mod::shib': }
end
end

describe 'auth_oidc', unless: (os[:family] == 'ubuntu' && os[:release].to_f == 14.04) do
# IAC-587: These tests do not currently run successfully on certain RHEL OSs due to dependency issues with the
# mod_auth_openidc module.
describe 'auth_oidc', if: (os[:family] == 'ubuntu' && os[:release].to_i > 14 || os[:family] == 'debian') do
pp = <<-MANIFEST
class { 'apache': }
apache::vhost { 'test.server':
Expand Down

0 comments on commit c0d75cf

Please sign in to comment.