Skip to content

Commit

Permalink
Fix pending tests
Browse files Browse the repository at this point in the history
With rspec 3, pending() started inverting the logic so that if a
pending tests succeeds then it fails (so you can block on a future
change). These tests have no body so will always succeed.

Anyway, pending -> skip reverts the logic to the usual.
  • Loading branch information
hunner committed Jan 10, 2015
1 parent 7ed3f56 commit cff25a8
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion spec/unit/classes/concat_setup_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@

context 'deprecated as a public class' do
it 'should create a warning' do
pending('rspec-puppet support for testing warning()')
skip('rspec-puppet support for testing warning()')
end
end

Expand Down
10 changes: 5 additions & 5 deletions spec/unit/defines/concat_fragment_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@
let(:params) {{ :ensure => 'invalid', :target => '/etc/motd' }}

it 'should create a warning' do
pending('rspec-puppet support for testing warning()')
skip('rspec-puppet support for testing warning()')
end
end
end # ensure =>
Expand Down Expand Up @@ -244,7 +244,7 @@
}

it 'should create a warning' do
pending('rspec-puppet support for testing warning()')
skip('rspec-puppet support for testing warning()')
end
end
end # mode =>
Expand All @@ -257,7 +257,7 @@
}

it 'should create a warning' do
pending('rspec-puppet support for testing warning()')
skip('rspec-puppet support for testing warning()')
end
end
end # owner =>
Expand All @@ -270,7 +270,7 @@
}

it 'should create a warning' do
pending('rspec-puppet support for testing warning()')
skip('rspec-puppet support for testing warning()')
end
end
end # group =>
Expand All @@ -283,7 +283,7 @@
}

it 'should create a warning' do
pending('rspec-puppet support for testing warning()')
skip('rspec-puppet support for testing warning()')
end
end
end # backup =>
Expand Down
4 changes: 2 additions & 2 deletions spec/unit/defines/concat_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,7 @@
it_behaves_like 'concat', '/etc/foo.bar', { :warn => warn }

it 'should create a warning' do
pending('rspec-puppet support for testing warning()')
skip('rspec-puppet support for testing warning()')
end
end
end
Expand Down Expand Up @@ -386,7 +386,7 @@
it_behaves_like 'concat', '/etc/foo.bar', { :gnu => 'foo'}

it 'should create a warning' do
pending('rspec-puppet support for testing warning()')
skip('rspec-puppet support for testing warning()')
end
end
end
Expand Down

0 comments on commit cff25a8

Please sign in to comment.