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 9da8b235feff81e8623d255d8d2796ba7dc77454..34b394282fd4df1d8c6011fcf479a4370e629570 100644
(file)
--- a/
spec/defines/source_spec.rb
+++ b/
spec/defines/source_spec.rb
@@
-1,6
+1,9
@@
require 'spec_helper'
require 'spec_helper'
+
describe 'apt::source', :type => :define do
let(:facts) { { :lsbdistid => 'Debian' } }
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
@@
-14,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
@@
-28,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',
@@
-135,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