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
(PUP-6856) Always define facts
[puppet-modules/puppetlabs-apt.git]
/
spec
/
defines
/
pin_spec.rb
diff --git
a/spec/defines/pin_spec.rb
b/spec/defines/pin_spec.rb
index e4d5d0ae1af79538f76b85b6f16946911ecc7af3..4f51cf3d715c181bb41b7ebc274faa937fc8376c 100644
(file)
--- a/
spec/defines/pin_spec.rb
+++ b/
spec/defines/pin_spec.rb
@@
-3,23
+3,27
@@
describe 'apt::pin', :type => :define do
let :pre_condition do
'class { "apt": }'
end
let :pre_condition do
'class { "apt": }'
end
- let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian' } }
+ let(:facts) { {
+ :os => { :family => 'Debian', :name => 'Debian', :release => { :major => '7', :full => '7.0' }},
+ :lsbdistid => 'Debian',
+ :osfamily => 'Debian',
+ :lsbdistcodename => 'wheezy',
+ :puppetversion => Puppet.version,
+ } }
let(:title) { 'my_pin' }
context 'defaults' do
it { is_expected.to contain_apt__setting("pref-my_pin").with_content(/Explanation: : my_pin\nPackage: \*\nPin: release a=my_pin\nPin-Priority: 0\n/)}
let(:title) { 'my_pin' }
context 'defaults' do
it { is_expected.to contain_apt__setting("pref-my_pin").with_content(/Explanation: : my_pin\nPackage: \*\nPin: release a=my_pin\nPin-Priority: 0\n/)}
- it { is_expected.to contain_apt__setting("pref-my_pin") }
end
context 'set version' do
let :params do
{
'packages' => 'vim',
end
context 'set version' do
let :params do
{
'packages' => 'vim',
- 'version' =>
'1'
,
+ 'version' =>
"1"
,
}
end
it { is_expected.to contain_apt__setting("pref-my_pin").with_content(/Explanation: : my_pin\nPackage: vim\nPin: version 1\nPin-Priority: 0\n/)}
}
end
it { is_expected.to contain_apt__setting("pref-my_pin").with_content(/Explanation: : my_pin\nPackage: vim\nPin: version 1\nPin-Priority: 0\n/)}
- it { is_expected.to contain_apt__setting("pref-my_pin") }
end
context 'set origin' do
end
context 'set origin' do
@@
-30,7
+34,6
@@
describe 'apt::pin', :type => :define do
}
end
it { is_expected.to contain_apt__setting("pref-my_pin").with_content(/Explanation: : my_pin\nPackage: vim\nPin: origin test\nPin-Priority: 0\n/)}
}
end
it { is_expected.to contain_apt__setting("pref-my_pin").with_content(/Explanation: : my_pin\nPackage: vim\nPin: origin test\nPin-Priority: 0\n/)}
- it { is_expected.to contain_apt__setting("pref-my_pin") }
end
context 'not defaults' do
end
context 'not defaults' do
@@
-80,8
+83,8
@@
describe 'apt::pin', :type => :define do
end
it do
expect {
end
it do
expect {
- is_expected.to compile
- }.to raise_error(Puppet::Error, /
Only integers are allowed
/)
+ subject.call
+ }.to raise_error(Puppet::Error, /
expects a value of type Integer
/)
end
end
end
end
@@
-93,7
+96,7
@@
describe 'apt::pin', :type => :define do
end
it do
expect {
end
it do
expect {
- is_expected.to compile
+ subject.call
}.to raise_error(Puppet::Error, /parameter version cannot be used in general form/)
end
end
}.to raise_error(Puppet::Error, /parameter version cannot be used in general form/)
end
end
@@
-107,7
+110,7
@@
describe 'apt::pin', :type => :define do
end
it do
expect {
end
it do
expect {
- is_expected.to compile
+ subject.call
}.to raise_error(Puppet::Error, /parameters release and origin are mutually exclusive/)
end
end
}.to raise_error(Puppet::Error, /parameters release and origin are mutually exclusive/)
end
end
@@
-122,7
+125,7
@@
describe 'apt::pin', :type => :define do
end
it do
expect {
end
it do
expect {
- is_expected.to compile
+ subject.call
}.to raise_error(Puppet::Error, /parameters release, origin, and version are mutually exclusive/)
end
end
}.to raise_error(Puppet::Error, /parameters release, origin, and version are mutually exclusive/)
end
end
@@
-137,7
+140,7
@@
describe 'apt::pin', :type => :define do
end
it do
expect {
end
it do
expect {
- is_expected.to compile
+ subject.call
}.to raise_error(Puppet::Error, /parameters release, origin, and version are mutually exclusive/)
end
end
}.to raise_error(Puppet::Error, /parameters release, origin, and version are mutually exclusive/)
end
end