Skip to content

Commit

Permalink
Remove duplicate tests to speed up runs
Browse files Browse the repository at this point in the history
  • Loading branch information
garethr committed Oct 9, 2015
1 parent 2adf7a9 commit a5ebc02
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 24 deletions.
24 changes: 0 additions & 24 deletions spec/acceptance/docker_full_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -15,30 +15,6 @@
end

describe 'docker class' do
context 'without any parameters' do
let(:pp) {"
class { 'docker': }
"}

it 'should run successfully' do
apply_manifest(pp, :catch_failures => true)
end

it 'should run idempotently' do
apply_manifest(pp, :catch_changes => true) unless fact('selinux') == 'true'
end

it 'should be start a docker process' do
shell('ps -aux | grep docker') do |r|
expect(r.stdout).to match(/\/usr\/bin\/docker/)
end
end

it 'should install a working docker client' do
shell('docker ps', :acceptable_exit_codes => [0])
end
end

context 'passing a TCP address to bind to' do
before(:all) do
@pp =<<-EOS
Expand Down
6 changes: 6 additions & 0 deletions spec/acceptance/docker_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,12 @@ class { 'docker':
apply_manifest(pp, :catch_changes=>true)
end

it 'should be start a docker process' do
shell('ps -aux | grep docker') do |r|
expect(r.stdout).to match(/\/usr\/bin\/docker/)
end
end

describe package(package_name) do
it { is_expected.to be_installed }
end
Expand Down

0 comments on commit a5ebc02

Please sign in to comment.