Code Review
/
puppet-modules
/
puppetlabs-apt.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Merge pull request #504 from mhaskel/fix_tests
[puppet-modules/puppetlabs-apt.git]
/
spec
/
defines
/
setting_spec.rb
diff --git
a/spec/defines/setting_spec.rb
b/spec/defines/setting_spec.rb
index 6f09f9ccf084447a62838f3544eefcc4e265f587..16c80163aa960af0f151585f19e64331baa1daec 100644
(file)
--- a/
spec/defines/setting_spec.rb
+++ b/
spec/defines/setting_spec.rb
@@
-10,7
+10,7
@@
describe 'apt::setting' do
describe 'when using the defaults' do
context 'without source or content' do
it do
describe 'when using the defaults' do
context 'without source or content' do
it do
- expect {
is_expected.to compile
}.to raise_error(Puppet::Error, /needs either of /)
+ expect {
subject.call
}.to raise_error(Puppet::Error, /needs either of /)
end
end
end
end
@@
-74,7
+74,7
@@
describe 'apt::setting' do
context 'with source and content' do
let(:params) { default_params.merge({ :source => 'la' }) }
it do
context 'with source and content' do
let(:params) { default_params.merge({ :source => 'la' }) }
it do
- expect {
is_expected.to compile
}.to raise_error(Puppet::Error, /cannot have both /)
+ expect {
subject.call
}.to raise_error(Puppet::Error, /cannot have both /)
end
end
end
end
@@
-82,21
+82,21
@@
describe 'apt::setting' do
let(:title) { 'ext-teddybear' }
let(:params) { default_params }
it do
let(:title) { 'ext-teddybear' }
let(:params) { default_params }
it do
- expect {
is_expected.to compile
}.to raise_error(Puppet::Error, /must start with /)
+ expect {
subject.call
}.to raise_error(Puppet::Error, /must start with /)
end
end
context 'with ensure=banana' do
let(:params) { default_params.merge({ :ensure => 'banana' }) }
it do
end
end
context 'with ensure=banana' do
let(:params) { default_params.merge({ :ensure => 'banana' }) }
it do
- expect {
is_expected.to compile
}.to raise_error(Puppet::Error, /"banana" does not /)
+ expect {
subject.call
}.to raise_error(Puppet::Error, /"banana" does not /)
end
end
context 'with priority=1.2' do
let(:params) { default_params.merge({ :priority => 1.2 }) }
it do
end
end
context 'with priority=1.2' do
let(:params) { default_params.merge({ :priority => 1.2 }) }
it do
- expect {
is_expected.to compile
}.to raise_error(Puppet::Error, /be an integer /)
+ expect {
subject.call
}.to raise_error(Puppet::Error, /be an integer /)
end
end
end
end
end
end