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
/
ppa_spec.rb
diff --git
a/spec/defines/ppa_spec.rb
b/spec/defines/ppa_spec.rb
index 521b42ba2b47897fc99b9c8b59f5769b73dfac59..c110e50f43dfc5c4a54e725b702a7cc7dfad250a 100644
(file)
--- a/
spec/defines/ppa_spec.rb
+++ b/
spec/defines/ppa_spec.rb
@@
-12,6
+12,7
@@
describe 'apt::ppa' do
:operatingsystem => 'Ubuntu',
:osfamily => 'Debian',
:lsbdistid => 'Ubuntu',
:operatingsystem => 'Ubuntu',
:osfamily => 'Debian',
:lsbdistid => 'Ubuntu',
+ :puppetversion => Puppet.version,
}
end
}
end
@@
-50,6
+51,7
@@
describe 'apt::ppa' do
:operatingsystem => 'Ubuntu',
:osfamily => 'Debian',
:lsbdistid => 'Ubuntu',
:operatingsystem => 'Ubuntu',
:osfamily => 'Debian',
:lsbdistid => 'Ubuntu',
+ :puppetversion => Puppet.version,
}
end
}
end
@@
-81,6
+83,7
@@
describe 'apt::ppa' do
:operatingsystem => 'Ubuntu',
:osfamily => 'Debian',
:lsbdistid => 'Ubuntu',
:operatingsystem => 'Ubuntu',
:osfamily => 'Debian',
:lsbdistid => 'Ubuntu',
+ :puppetversion => Puppet.version,
}
end
let :params do
}
end
let :params do
@@
-119,6
+122,7
@@
describe 'apt::ppa' do
:operatingsystem => 'Ubuntu',
:lsbdistid => 'Ubuntu',
:osfamily => 'Debian',
:operatingsystem => 'Ubuntu',
:lsbdistid => 'Ubuntu',
:osfamily => 'Debian',
+ :puppetversion => Puppet.version,
}
end
let :params do
}
end
let :params do
@@
-154,6
+158,7
@@
describe 'apt::ppa' do
:operatingsystem => 'Ubuntu',
:lsbdistid => 'Ubuntu',
:osfamily => 'Debian',
:operatingsystem => 'Ubuntu',
:lsbdistid => 'Ubuntu',
:osfamily => 'Debian',
+ :puppetversion => Puppet.version,
}
end
let :params do
}
end
let :params do
@@
-187,6
+192,7
@@
describe 'apt::ppa' do
:operatingsystem => 'Ubuntu',
:lsbdistid => 'Ubuntu',
:osfamily => 'Debian',
:operatingsystem => 'Ubuntu',
:lsbdistid => 'Ubuntu',
:osfamily => 'Debian',
+ :puppetversion => Puppet.version,
}
end
let :params do
}
end
let :params do
@@
-220,6
+226,7
@@
describe 'apt::ppa' do
:operatingsystem => 'Ubuntu',
:lsbdistid => 'Ubuntu',
:osfamily => 'Debian',
:operatingsystem => 'Ubuntu',
:lsbdistid => 'Ubuntu',
:osfamily => 'Debian',
+ :puppetversion => Puppet.version,
}
end
let :params do
}
end
let :params do
@@
-251,6
+258,7
@@
describe 'apt::ppa' do
:operatingsystem => 'Ubuntu',
:lsbdistid => 'Ubuntu',
:osfamily => 'Debian',
:operatingsystem => 'Ubuntu',
:lsbdistid => 'Ubuntu',
:osfamily => 'Debian',
+ :puppetversion => Puppet.version,
}
end
let(:title) { 'ppa:foo' }
}
end
let(:title) { 'ppa:foo' }
@@
-274,12
+282,13
@@
describe 'apt::ppa' do
:lsbdistid => 'Ubuntu',
:osfamily => 'Debian',
:lsbdistcodeanme => nil,
:lsbdistid => 'Ubuntu',
:osfamily => 'Debian',
:lsbdistcodeanme => nil,
+ :puppetversion => Puppet.version,
}
end
let(:title) { 'ppa:foo' }
it do
expect {
}
end
let(:title) { 'ppa:foo' }
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
@@
-292,12
+301,13
@@
describe 'apt::ppa' do
:operatingsystem => 'Debian',
:lsbdistid => 'debian',
:osfamily => 'Debian',
:operatingsystem => 'Debian',
:lsbdistid => 'debian',
:osfamily => 'Debian',
+ :puppetversion => Puppet.version,
}
end
let(:title) { 'ppa:foo' }
it do
expect {
}
end
let(:title) { 'ppa:foo' }
it do
expect {
- is_expected.to compile
+ subject.call
}.to raise_error(Puppet::Error, /not currently supported on Debian/)
end
end
}.to raise_error(Puppet::Error, /not currently supported on Debian/)
end
end