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 #521 from hunner/fix_stubs
[puppet-modules/puppetlabs-apt.git]
/
spec
/
defines
/
source_spec.rb
diff --git
a/spec/defines/source_spec.rb
b/spec/defines/source_spec.rb
index 3900158a1612cc48b8995195c40e5dae655515d0..26c8fa78d29788cb09ea7e7763955388388620c9 100644
(file)
--- a/
spec/defines/source_spec.rb
+++ b/
spec/defines/source_spec.rb
@@
-17,12
+17,13
@@
describe 'apt::source' do
{
:lsbdistid => 'Debian',
:lsbdistcodename => 'wheezy',
{
:lsbdistid => 'Debian',
:lsbdistcodename => 'wheezy',
- :osfamily => 'Debian'
+ :osfamily => 'Debian',
+ :puppetversion => Puppet.version,
}
end
it do
expect {
}
end
it do
expect {
- is_expected.to compile
+ subject.call
}.to raise_error(Puppet::Error, /source entry without specifying a location/)
end
end
}.to raise_error(Puppet::Error, /source entry without specifying a location/)
end
end
@@
-31,7
+32,8
@@
describe 'apt::source' do
{
:lsbdistid => 'Debian',
:lsbdistcodename => 'wheezy',
{
:lsbdistid => 'Debian',
:lsbdistcodename => 'wheezy',
- :osfamily => 'Debian'
+ :osfamily => 'Debian',
+ :puppetversion => Puppet.version,
}
end
let(:params) { { :location => 'hello.there', } }
}
end
let(:params) { { :location => 'hello.there', } }
@@
-48,7
+50,8
@@
describe 'apt::source' do
{
:lsbdistid => 'Debian',
:lsbdistcodename => 'wheezy',
{
:lsbdistid => 'Debian',
:lsbdistcodename => 'wheezy',
- :osfamily => 'Debian'
+ :osfamily => 'Debian',
+ :puppetversion => Puppet.version,
}
end
}
end
@@
-186,7
+189,8
@@
describe 'apt::source' do
{
:lsbdistid => 'Debian',
:lsbdistcodename => 'wheezy',
{
:lsbdistid => 'Debian',
:lsbdistcodename => 'wheezy',
- :osfamily => 'Debian'
+ :osfamily => 'Debian',
+ :puppetversion => Puppet.version,
}
end
let :params do
}
end
let :params do
@@
-207,7
+211,8
@@
describe 'apt::source' do
{
:lsbdistid => 'Debian',
:lsbdistcodename => 'wheezy',
{
:lsbdistid => 'Debian',
:lsbdistcodename => 'wheezy',
- :osfamily => 'Debian'
+ :osfamily => 'Debian',
+ :puppetversion => Puppet.version,
}
end
let :params do
}
end
let :params do
@@
-229,7
+234,8
@@
describe 'apt::source' do
{
:lsbdistid => 'Debian',
:lsbdistcodename => 'wheezy',
{
:lsbdistid => 'Debian',
:lsbdistcodename => 'wheezy',
- :osfamily => 'Debian'
+ :osfamily => 'Debian',
+ :puppetversion => Puppet.version,
}
end
let :params do
}
end
let :params do
@@
-249,14
+255,15
@@
describe 'apt::source' do
let :facts do
{
:lsbdistid => 'Debian',
let :facts do
{
:lsbdistid => 'Debian',
- :osfamily => 'Debian'
+ :osfamily => 'Debian',
+ :puppetversion => Puppet.version,
}
end
let(:params) { { :location => 'hello.there', } }
it do
expect {
}
end
let(:params) { { :location => 'hello.there', } }
it do
expect {
- is_expected.to compile
+ subject.call
}.to raise_error(Puppet::Error, /lsbdistcodename fact not available: release parameter required/)
end
end
}.to raise_error(Puppet::Error, /lsbdistcodename fact not available: release parameter required/)
end
end
@@
-266,7
+273,8
@@
describe 'apt::source' do
{
:lsbdistid => 'Debian',
:lsbdistcodename => 'wheezy',
{
:lsbdistid => 'Debian',
:lsbdistcodename => 'wheezy',
- :osfamily => 'Debian'
+ :osfamily => 'Debian',
+ :puppetversion => Puppet.version,
}
end
let :params do
}
end
let :params do
@@
-278,7
+286,7
@@
describe 'apt::source' do
it do
expect {
it do
expect {
- is_expected.to compile
+ subject.call
}.to raise_error(Puppet::Error, /invalid value for pin/)
end
end
}.to raise_error(Puppet::Error, /invalid value for pin/)
end
end