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 #282 from laurenrother/security
[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 9ad4d463e4939fde48a2eccd19a9b4ab1738bb13..34b394282fd4df1d8c6011fcf479a4370e629570 100644
(file)
--- a/
spec/defines/source_spec.rb
+++ b/
spec/defines/source_spec.rb
@@
-1,5
+1,9
@@
require 'spec_helper'
require 'spec_helper'
+
describe 'apt::source', :type => :define do
describe 'apt::source', :type => :define do
+ let(:facts) { { :lsbdistid => 'Debian' } }
+ GPG_KEY_ID = '4BD6EC30'
+
let :title do
'my_source'
end
let :title do
'my_source'
end
@@
-13,7
+17,7
@@
describe 'apt::source', :type => :define do
:include_src => true,
:required_packages => false,
:key => false,
:include_src => true,
:required_packages => false,
:key => false,
- :key_server =>
'keyserver.ubuntu.com'
,
+ :key_server =>
false
,
:key_content => false,
:key_source => false,
:pin => false
:key_content => false,
:key_source => false,
:pin => false
@@
-27,15
+31,14
@@
describe 'apt::source', :type => :define do
:repos => 'security',
:include_src => false,
:required_packages => 'apache',
:repos => 'security',
:include_src => false,
:required_packages => 'apache',
- :key =>
'key_name'
,
+ :key =>
GPG_KEY_ID
,
:key_server => 'keyserver.debian.com',
:pin => '600',
:key_content => 'ABCD1234'
},
{
:key_server => 'keyserver.debian.com',
:pin => '600',
:key_content => 'ABCD1234'
},
{
- :key =>
'key_name'
,
+ :key =>
GPG_KEY_ID
,
:key_server => 'keyserver.debian.com',
:key_server => 'keyserver.debian.com',
- :key_content => false,
},
{
:ensure => 'absent',
},
{
:ensure => 'absent',
@@
-59,7
+62,7
@@
describe 'apt::source', :type => :define do
end
let :facts do
end
let :facts do
- {:lsbdistcodename => 'karmic'}
+ {:lsbdistcodename => 'karmic'
, :lsbdistid => 'Ubuntu'
}
end
let :params do
end
let :params do
@@
-73,7
+76,7
@@
describe 'apt::source', :type => :define do
let :content do
content = "# #{title}"
if param_hash[:architecture]
let :content do
content = "# #{title}"
if param_hash[:architecture]
- arch = "[arch=#{param_hash[:architecture]}]"
+ arch = "[arch=#{param_hash[:architecture]}]
"
end
content << "\ndeb #{arch}#{param_hash[:location]} #{param_hash[:release]} #{param_hash[:repos]}\n"
end
content << "\ndeb #{arch}#{param_hash[:location]} #{param_hash[:release]} #{param_hash[:repos]}\n"
@@
-134,13
+137,16
@@
describe 'apt::source', :type => :define do
}
it {
}
it {
+ key_server = param_hash[:key_server] || nil
+ key_content = param_hash[:key_content] || nil
+ key_source = param_hash[:key_source] || nil
if param_hash[:key]
should contain_apt__key("Add key: #{param_hash[:key]} from Apt::Source #{title}").with({
"key" => param_hash[:key],
"ensure" => :present,
if param_hash[:key]
should contain_apt__key("Add key: #{param_hash[:key]} from Apt::Source #{title}").with({
"key" => param_hash[:key],
"ensure" => :present,
- "key_server" =>
param_hash[:key_server]
,
- "key_content" =>
param_hash[:key_content]
,
- "key_source" =>
param_hash[:key_source]
,
+ "key_server" =>
key_server
,
+ "key_content" =>
key_content
,
+ "key_source" =>
key_source
,
"before" => "File[#{title}.list]"
})
else
"before" => "File[#{title}.list]"
})
else
@@
-160,7
+166,7
@@
describe 'apt::source', :type => :define do
let(:default_params) { Hash.new }
let(:facts) { Hash.new }
it { expect { should raise_error(Puppet::Error) } }
let(:default_params) { Hash.new }
let(:facts) { Hash.new }
it { expect { should raise_error(Puppet::Error) } }
- let(:facts) { { :lsbdistcodename => 'lucid' } }
+ let(:facts) { { :lsbdistcodename => 'lucid'
, :lsbdistid => 'Ubuntu'
} }
it { should contain_apt__source(title) }
end
end
it { should contain_apt__source(title) }
end
end