Skip to content

Commit

Permalink
pend a test to catch if notify in C8 is ever fixed.
Browse files Browse the repository at this point in the history
  • Loading branch information
traylenator committed Mar 23, 2022
1 parent 16232ef commit e50723f
Showing 1 changed file with 30 additions and 1 deletion.
31 changes: 30 additions & 1 deletion spec/acceptance/class_spec.rb
Original file line number Diff line number Diff line change
@@ -1,7 +1,36 @@
require 'spec_helper_acceptance'

describe 'squid class' do
context 'configure http_access' do
context 'configure http_access with default service type' do
it 'works idempotently with no errors' do
pending('the default Type=notify in squid.service working again on CentOS 8') if fact('os.family') == 'RedHat' && fact('os.release.major') != '7'
pp = <<-EOS
# The default Type=notify is problematic in github CI.
if $facts['os']['family'] == 'RedHat' and $facts['os']['release']['major'] != '7' {
systemd::dropin_file{'simple.conf':
ensure => absent,
unit => 'squid.service',
before => Service['squid'],
}
}
class { 'squid':}
squid::http_port{'3128':}
squid::acl{'our_networks':
type => src,
entries => ['all'],
}
squid::http_access{'our_networks':
action => 'allow',
comment => 'Our networks hosts are allowed',
}
EOS
# Run it twice and test for idempotency
apply_manifest(pp, catch_failures: true)
apply_manifest(pp, catch_changes: true)
end
end

context 'configure http_access with simple service type' do
it 'works idempotently with no errors' do
pp = <<-EOS
# The default Type=notify is problematic in github CI.
Expand Down

0 comments on commit e50723f

Please sign in to comment.