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 #799 from eimlav/string-fix
[puppet-modules/puppetlabs-apt.git]
/
spec
/
defines
/
key_spec.rb
diff --git
a/spec/defines/key_spec.rb
b/spec/defines/key_spec.rb
index d668780eaa99fd11adade8a408794ebc737579b9..cf596e996b901f71a37e59541be32998886367d8 100644
(file)
--- a/
spec/defines/key_spec.rb
+++ b/
spec/defines/key_spec.rb
@@
-43,10
+43,10
@@
describe 'apt::key' do
let(:facts) do
{
let(:facts) do
{
- os: { family: 'Debian', name: 'Debian', release: { major: '
7', full: '7
.0' } },
+ os: { family: 'Debian', name: 'Debian', release: { major: '
8', full: '8
.0' } },
lsbdistid: 'Debian',
osfamily: 'Debian',
lsbdistid: 'Debian',
osfamily: 'Debian',
- lsbdistcodename: '
wheezy
',
+ lsbdistcodename: '
jessie
',
puppetversion: Puppet.version,
}
end
puppetversion: Puppet.version,
}
end
@@
-99,6
+99,21
@@
describe 'apt::key' do
end
end
end
end
+ describe 'ensure => refreshed' do
+ let :params do
+ {
+ ensure: 'refreshed',
+ }
+ end
+
+ it 'contains the apt_key with refresh => true' do
+ is_expected.to contain_apt_key(title).with(
+ ensure: 'present',
+ refresh: true,
+ )
+ end
+ end
+
describe 'set a bunch of things!' do
let :params do
{
describe 'set a bunch of things!' do
let :params do
{
@@
-117,7
+132,7
@@
describe 'apt::key' do
end
end
end
end
- context 'domain with dash' do
+ context '
when
domain with dash' do
let(:params) do
{
server: 'p-gp.m-it.edu',
let(:params) do
{
server: 'p-gp.m-it.edu',
@@
-130,7
+145,7
@@
describe 'apt::key' do
end
end
end
end
- context 'url' do
+ context '
with
url' do
let :params do
{
server: 'hkp://pgp.mit.edu',
let :params do
{
server: 'hkp://pgp.mit.edu',
@@
-142,7
+157,7
@@
describe 'apt::key' do
server: 'hkp://pgp.mit.edu')
end
end
server: 'hkp://pgp.mit.edu')
end
end
- context 'url with port number' do
+ context '
when
url with port number' do
let :params do
{
server: 'hkp://pgp.mit.edu:80',
let :params do
{
server: 'hkp://pgp.mit.edu:80',
@@
-157,7
+172,7
@@
describe 'apt::key' do
end
describe 'validation' do
end
describe 'validation' do
- context 'domain begin with dash' do
+ context '
when
domain begin with dash' do
let(:params) do
{
server: '-pgp.mit.edu',
let(:params) do
{
server: '-pgp.mit.edu',
@@
-169,7
+184,7
@@
describe 'apt::key' do
end
end
end
end
- context 'domain begin with dot' do
+ context '
when
domain begin with dot' do
let(:params) do
{
server: '.pgp.mit.edu',
let(:params) do
{
server: '.pgp.mit.edu',
@@
-181,7
+196,7
@@
describe 'apt::key' do
end
end
end
end
- context 'domain end with dot' do
+ context '
when
domain end with dot' do
let(:params) do
{
server: 'pgp.mit.edu.',
let(:params) do
{
server: 'pgp.mit.edu.',
@@
-192,7
+207,7
@@
describe 'apt::key' do
is_expected .to raise_error(%r{expects a match})
end
end
is_expected .to raise_error(%r{expects a match})
end
end
- context '
exceed character url
' do
+ context '
when character url exceeded
' do
let :params do
{
server: 'hkp://pgpiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii.mit.edu',
let :params do
{
server: 'hkp://pgpiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii.mit.edu',
@@
-203,7
+218,7
@@
describe 'apt::key' do
is_expected.to raise_error(%r{expects a match})
end
end
is_expected.to raise_error(%r{expects a match})
end
end
- context 'incorrect port number url' do
+ context '
with
incorrect port number url' do
let :params do
{
server: 'hkp://pgp.mit.edu:8008080',
let :params do
{
server: 'hkp://pgp.mit.edu:8008080',
@@
-214,7
+229,7
@@
describe 'apt::key' do
is_expected.to raise_error(%r{expects a match})
end
end
is_expected.to raise_error(%r{expects a match})
end
end
- context '
incorrect protocol for
url' do
+ context '
with incorrect protocol for
url' do
let :params do
{
server: 'abc://pgp.mit.edu:80',
let :params do
{
server: 'abc://pgp.mit.edu:80',
@@
-225,7
+240,7
@@
describe 'apt::key' do
is_expected.to raise_error(%r{expects a match})
end
end
is_expected.to raise_error(%r{expects a match})
end
end
- context 'missing port number url' do
+ context '
with
missing port number url' do
let :params do
{
server: 'hkp://pgp.mit.edu:',
let :params do
{
server: 'hkp://pgp.mit.edu:',
@@
-236,7
+251,7
@@
describe 'apt::key' do
is_expected.to raise_error(%r{expects a match})
end
end
is_expected.to raise_error(%r{expects a match})
end
end
- context 'url ending with a dot' do
+ context '
with
url ending with a dot' do
let :params do
{
server: 'hkp://pgp.mit.edu.',
let :params do
{
server: 'hkp://pgp.mit.edu.',
@@
-247,7
+262,7
@@
describe 'apt::key' do
is_expected.to raise_error(%r{expects a match})
end
end
is_expected.to raise_error(%r{expects a match})
end
end
- context '
url begin
with a dash' do
+ context '
when url begins
with a dash' do
let(:params) do
{
server: 'hkp://-pgp.mit.edu',
let(:params) do
{
server: 'hkp://-pgp.mit.edu',
@@
-258,7
+273,7
@@
describe 'apt::key' do
is_expected.to raise_error(%r{expects a match})
end
end
is_expected.to raise_error(%r{expects a match})
end
end
- context 'invalid key' do
+ context '
with
invalid key' do
let :title do
'Out of rum. Why? Why are we out of rum?'
end
let :title do
'Out of rum. Why? Why are we out of rum?'
end
@@
-268,7
+283,7
@@
describe 'apt::key' do
end
end
end
end
- context 'invalid source' do
+ context '
with
invalid source' do
let :params do
{
source: 'afp://puppetlabs.com/key.gpg',
let :params do
{
source: 'afp://puppetlabs.com/key.gpg',
@@
-276,11
+291,11
@@
describe 'apt::key' do
end
it 'fails' do
end
it 'fails' do
- is_expected.to raise_error(%r{e
xpects a match
})
+ is_expected.to raise_error(%r{e
valuating a Resource
})
end
end
end
end
- context 'invalid content' do
+ context '
with
invalid content' do
let :params do
{
content: [],
let :params do
{
content: [],
@@
-292,7
+307,7
@@
describe 'apt::key' do
end
end
end
end
- context 'invalid server' do
+ context '
with
invalid server' do
let :params do
{
server: 'two bottles of rum',
let :params do
{
server: 'two bottles of rum',
@@
-304,7
+319,7
@@
describe 'apt::key' do
end
end
end
end
- context 'invalid options' do
+ context '
with
invalid options' do
let :params do
{
options: {},
let :params do
{
options: {},
@@
-316,8
+331,8
@@
describe 'apt::key' do
end
end
end
end
- context 'invalid ensure' do
-
%w[foo aabsent absenta apresent presenta
].each do |param|
+ context '
with
invalid ensure' do
+
['foo', 'aabsent', 'absenta', 'apresent', 'presenta', 'refresh', 'arefreshed', 'refresheda'
].each do |param|
let :params do
{
ensure: param,
let :params do
{
ensure: param,
@@
-325,7
+340,7
@@
describe 'apt::key' do
end
it 'fails' do
end
it 'fails' do
- is_expected.to raise_error(%r{for Enum\['absent', 'present'\], got})
+ is_expected.to raise_error(%r{for Enum\['absent', 'present'
, 'refreshed'
\], got})
end
end
end
end
end
end