Skip to content

Commit

Permalink
test(pkgs_spec): simplify using match (trigger: update for suse)
Browse files Browse the repository at this point in the history
* https://travis-ci.org/github/myii/salt-formula/jobs/692337807#L2366-L2381
  - `opensuse` now has `3000.3` available
* Simplification is useful anyway, even without this (could still use `case`)
  • Loading branch information
myii committed Jun 1, 2020
1 parent 335af02 commit b2b1fee
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 76 deletions.
24 changes: 3 additions & 21 deletions test/integration/v201902-py2/controls/pkgs_spec.rb
Original file line number Diff line number Diff line change
@@ -1,35 +1,17 @@
# frozen_string_literal: true

# Prepare platform "finger"
platform_finger = "#{platform[:name]}-#{platform[:release].split('.')[0]}"

version =
case platform[:family]
when 'debian'
'2019.2.5+ds-1'
when 'redhat'
case platform_finger
when 'centos-7'
'2019.2.5-1.el7'
when 'centos-6'
'2019.2.5-1.el6'
when 'amazon-2'
'2019.2.5-1.amzn2'
when 'amazon-2018'
'2019.2.5-1.amzn1'
end
end

control 'salt packages' do
title 'should be installed'

version = '2019.2.5'

%w[
salt-master
salt-minion
].each do |p|
describe package(p) do
it { should be_installed }
its('version') { should eq version }
its('version') { should match(/^#{version}/) }
end
end
end
22 changes: 3 additions & 19 deletions test/integration/v201902-py3/controls/pkgs_spec.rb
Original file line number Diff line number Diff line change
@@ -1,33 +1,17 @@
# frozen_string_literal: true

# Prepare platform "finger"
platform_finger = "#{platform[:name]}-#{platform[:release].split('.')[0]}"

version =
case platform[:family]
when 'debian'
'2019.2.5+ds-1'
when 'redhat'
case platform_finger
when 'centos-8'
'2019.2.5-1.el8'
when 'centos-7'
'2019.2.5-1.el7'
when 'amazon-2'
'2019.2.5-1.amzn2'
end
end

control 'salt packages' do
title 'should be installed'

version = '2019.2.5'

%w[
salt-master
salt-minion
].each do |p|
describe package(p) do
it { should be_installed }
its('version') { should eq version }
its('version') { should match(/^#{version}/) }
end
end
end
10 changes: 3 additions & 7 deletions test/integration/v3000-py2/controls/pkgs_spec.rb
Original file line number Diff line number Diff line change
@@ -1,21 +1,17 @@
# frozen_string_literal: true

version =
case platform[:family]
when 'debian'
'3000.3+ds-1'
end

control 'salt packages' do
title 'should be installed'

version = '3000.3'

%w[
salt-master
salt-minion
].each do |p|
describe package(p) do
it { should be_installed }
its('version') { should eq version }
its('version') { should match(/^#{version}/) }
end
end
end
32 changes: 3 additions & 29 deletions test/integration/v3000-py3/controls/pkgs_spec.rb
Original file line number Diff line number Diff line change
@@ -1,43 +1,17 @@
# frozen_string_literal: true

# Prepare platform "finger"
platform_finger = "#{platform[:name]}-#{platform[:release].split('.')[0]}"

version =
case platform[:family]
when 'debian'
'3000.3+ds-1'
when 'redhat'
case platform_finger
when 'centos-8'
'3000.3-1.el8'
when 'centos-7'
'3000.3-1.el7'
when 'amazon-2'
'3000.3-1.amzn2'
end
when 'fedora'
'3000.3-1.fc31'
when 'suse'
# Issue in the upstream repo, should be `3000.3`
case platform[:release]
when /^15.2/
'3000.2-lp152.7.1'
when /^15.1/
'3000.2-lp151.2.1'
end
end

control 'salt packages' do
title 'should be installed'

version = '3000.3'

%w[
salt-master
salt-minion
].each do |p|
describe package(p) do
it { should be_installed }
its('version') { should eq version }
its('version') { should match(/^#{version}/) }
end
end
end

0 comments on commit b2b1fee

Please sign in to comment.