Skip to content

Commit

Permalink
only run epel acceptance test on EL7
Browse files Browse the repository at this point in the history
Newer versions of EL have nodejs packages in appstream, not EPEL.
  • Loading branch information
kenyon committed Sep 17, 2023
1 parent 6e856d5 commit 8b7f74b
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions spec/acceptance/class_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
end
end

context 'repo_class => epel', if: ((fact('os.family') == 'RedHat') && (fact('os.release.major') != '8')) do
context 'repo_class => epel', if: ((fact('os.family') == 'RedHat') && (fact('os.release.major') == '7')) do
include_examples 'cleanup'

it_behaves_like 'an idempotent resource' do
Expand All @@ -44,7 +44,6 @@ class { 'nodejs':
it { is_expected.to be_installed }

it 'comes from the expected source' do
pending("This won't work until we have CentOS 7.4 because of dependency")
pkg_output = shell(pkg_cmd)
expect(pkg_output.stdout).to match 'epel'
end
Expand Down

0 comments on commit 8b7f74b

Please sign in to comment.