360f04dcc1
Puppet 3.5.0 introduced strict variables and the module handles strict variables by using the defined() function. This does not work on prior versions of puppet so we must gate based on that version. Puppet 4 series has a new setting `strict` that may be set to enforce strict variables while `strict_variables` remains unset (see PUP-6358) which causes this conditional to erroniously use non-strict 3.5-era parsing and fail. The new conditional corrects the cases such that strict variable behavior happens on versions 3.5.0 and later.
330 řádky
8,5 KiB
Ruby
330 řádky
8,5 KiB
Ruby
require 'spec_helper'
|
|
|
|
describe 'apt::key', :type => :define do
|
|
let(:facts) { {
|
|
:lsbdistid => 'Debian',
|
|
:osfamily => 'Debian',
|
|
:puppetversion => Puppet.version,
|
|
} }
|
|
GPG_KEY_ID = '47B320EB4C7C375AA9DAE1A01054B7A24BD6EC30'
|
|
|
|
let :title do
|
|
GPG_KEY_ID
|
|
end
|
|
|
|
let :pre_condition do
|
|
'include apt'
|
|
end
|
|
|
|
describe 'normal operation' do
|
|
describe 'default options' do
|
|
it {
|
|
is_expected.to contain_apt_key(title).with({
|
|
:id => title,
|
|
:ensure => 'present',
|
|
:source => nil,
|
|
:server => 'keyserver.ubuntu.com',
|
|
:content => nil,
|
|
:keyserver_options => nil,
|
|
})
|
|
}
|
|
it 'contains the apt_key present anchor' do
|
|
is_expected.to contain_anchor("apt_key #{title} present")
|
|
end
|
|
end
|
|
|
|
describe 'title and key =>' do
|
|
let :title do
|
|
'puppetlabs'
|
|
end
|
|
|
|
let :params do {
|
|
:key => GPG_KEY_ID,
|
|
} end
|
|
|
|
it 'contains the apt_key' do
|
|
is_expected.to contain_apt_key(title).with({
|
|
:id => GPG_KEY_ID,
|
|
:ensure => 'present',
|
|
:source => nil,
|
|
:server => 'keyserver.ubuntu.com',
|
|
:content => nil,
|
|
:keyserver_options => nil,
|
|
})
|
|
end
|
|
it 'contains the apt_key present anchor' do
|
|
is_expected.to contain_anchor("apt_key #{GPG_KEY_ID} present")
|
|
end
|
|
end
|
|
|
|
describe 'ensure => absent' do
|
|
let :params do {
|
|
:ensure => 'absent',
|
|
} end
|
|
|
|
it 'contains the apt_key' do
|
|
is_expected.to contain_apt_key(title).with({
|
|
:id => title,
|
|
:ensure => 'absent',
|
|
:source => nil,
|
|
:server => 'keyserver.ubuntu.com',
|
|
:content => nil,
|
|
:keyserver_options => nil,
|
|
})
|
|
end
|
|
it 'contains the apt_key absent anchor' do
|
|
is_expected.to contain_anchor("apt_key #{title} absent")
|
|
end
|
|
end
|
|
|
|
describe 'set a bunch of things!' do
|
|
let :params do {
|
|
:key_content => 'GPG key content',
|
|
:key_source => 'http://apt.puppetlabs.com/pubkey.gpg',
|
|
:key_server => 'pgp.mit.edu',
|
|
:key_options => 'debug',
|
|
} end
|
|
|
|
it 'contains the apt_key' do
|
|
is_expected.to contain_apt_key(title).with({
|
|
:id => title,
|
|
:ensure => 'present',
|
|
:source => 'http://apt.puppetlabs.com/pubkey.gpg',
|
|
:server => 'pgp.mit.edu',
|
|
:content => params[:key_content],
|
|
:options => 'debug',
|
|
})
|
|
end
|
|
it 'contains the apt_key present anchor' do
|
|
is_expected.to contain_anchor("apt_key #{title} present")
|
|
end
|
|
end
|
|
|
|
context "domain with dash" do
|
|
let(:params) do{
|
|
:key_server => 'p-gp.m-it.edu',
|
|
} end
|
|
it 'contains the apt_key' do
|
|
is_expected.to contain_apt_key(title).with({
|
|
:id => title,
|
|
:server => 'p-gp.m-it.edu',
|
|
})
|
|
end
|
|
end
|
|
|
|
context "url" do
|
|
let :params do
|
|
{
|
|
:key_server => 'hkp://pgp.mit.edu',
|
|
}
|
|
end
|
|
it 'contains the apt_key' do
|
|
is_expected.to contain_apt_key(title).with({
|
|
:id => title,
|
|
:server => 'hkp://pgp.mit.edu',
|
|
})
|
|
end
|
|
end
|
|
context "url with port number" do
|
|
let :params do
|
|
{
|
|
:key_server => 'hkp://pgp.mit.edu:80',
|
|
}
|
|
end
|
|
it 'contains the apt_key' do
|
|
is_expected.to contain_apt_key(title).with({
|
|
:id => title,
|
|
:server => 'hkp://pgp.mit.edu:80',
|
|
})
|
|
end
|
|
end
|
|
end
|
|
|
|
describe 'validation' do
|
|
context "domain begin with dash" do
|
|
let(:params) do{
|
|
:key_server => '-pgp.mit.edu',
|
|
} end
|
|
it 'fails' do
|
|
expect { subject.call } .to raise_error(/does not match/)
|
|
end
|
|
end
|
|
|
|
context "domain begin with dot" do
|
|
let(:params) do{
|
|
:key_server => '.pgp.mit.edu',
|
|
} end
|
|
it 'fails' do
|
|
expect { subject.call } .to raise_error(/does not match/)
|
|
end
|
|
end
|
|
|
|
context "domain end with dot" do
|
|
let(:params) do{
|
|
:key_server => "pgp.mit.edu.",
|
|
} end
|
|
it 'fails' do
|
|
expect { subject.call } .to raise_error(/does not match/)
|
|
end
|
|
end
|
|
context "exceed character url" do
|
|
let :params do
|
|
{
|
|
:key_server => 'hkp://pgpiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii.mit.edu'
|
|
}
|
|
end
|
|
it 'fails' do
|
|
expect { subject.call }.to raise_error(/does not match/)
|
|
end
|
|
end
|
|
context "incorrect port number url" do
|
|
let :params do
|
|
{
|
|
:key_server => 'hkp://pgp.mit.edu:8008080'
|
|
}
|
|
end
|
|
it 'fails' do
|
|
expect { subject.call }.to raise_error(/does not match/)
|
|
end
|
|
end
|
|
context "incorrect protocol for url" do
|
|
let :params do
|
|
{
|
|
:key_server => 'abc://pgp.mit.edu:80'
|
|
}
|
|
end
|
|
it 'fails' do
|
|
expect { subject.call }.to raise_error(/does not match/)
|
|
end
|
|
end
|
|
context "missing port number url" do
|
|
let :params do
|
|
{
|
|
:key_server => 'hkp://pgp.mit.edu:'
|
|
}
|
|
end
|
|
it 'fails' do
|
|
expect { subject.call }.to raise_error(/does not match/)
|
|
end
|
|
end
|
|
context "url ending with a dot" do
|
|
let :params do
|
|
{
|
|
:key_server => 'hkp://pgp.mit.edu.'
|
|
}
|
|
end
|
|
it 'fails' do
|
|
expect { subject.call }.to raise_error(/does not match/)
|
|
end
|
|
end
|
|
context "url begin with a dash" do
|
|
let(:params) do{
|
|
:key_server => "hkp://-pgp.mit.edu",
|
|
} end
|
|
it 'fails' do
|
|
expect { subject.call }.to raise_error(/does not match/)
|
|
end
|
|
end
|
|
context 'invalid key' do
|
|
let :title do
|
|
'Out of rum. Why? Why are we out of rum?'
|
|
end
|
|
it 'fails' do
|
|
expect { subject.call }.to raise_error(/does not match/)
|
|
end
|
|
end
|
|
|
|
context 'invalid source' do
|
|
let :params do {
|
|
:key_source => 'afp://puppetlabs.com/key.gpg',
|
|
} end
|
|
it 'fails' do
|
|
expect { subject.call }.to raise_error(/does not match/)
|
|
end
|
|
end
|
|
|
|
context 'invalid content' do
|
|
let :params do {
|
|
:key_content => [],
|
|
} end
|
|
it 'fails' do
|
|
expect { subject.call }.to raise_error(/is not a string/)
|
|
end
|
|
end
|
|
|
|
context 'invalid server' do
|
|
let :params do {
|
|
:key_server => 'two bottles of rum',
|
|
} end
|
|
it 'fails' do
|
|
expect { subject.call }.to raise_error(/does not match/)
|
|
end
|
|
end
|
|
|
|
context 'invalid keyserver_options' do
|
|
let :params do {
|
|
:key_options => {},
|
|
} end
|
|
it 'fails' do
|
|
expect { subject.call }.to raise_error(/is not a string/)
|
|
end
|
|
end
|
|
|
|
context 'invalid ensure' do
|
|
let :params do
|
|
{
|
|
:ensure => 'foo',
|
|
}
|
|
end
|
|
it 'fails' do
|
|
expect { subject.call }.to raise_error(/does not match/)
|
|
end
|
|
end
|
|
|
|
describe 'duplication' do
|
|
context 'two apt::key resources for same key, different titles' do
|
|
let :pre_condition do
|
|
"#{super()}\napt::key { 'duplicate': key => '#{title}', }"
|
|
end
|
|
|
|
it 'contains the duplicate apt::key resource' do
|
|
is_expected.to contain_apt__key('duplicate').with({
|
|
:key => title,
|
|
:ensure => 'present',
|
|
})
|
|
end
|
|
|
|
it 'contains the original apt::key resource' do
|
|
is_expected.to contain_apt__key(title).with({
|
|
:id => title,
|
|
:ensure => 'present',
|
|
})
|
|
end
|
|
|
|
it 'contains the native apt_key' do
|
|
is_expected.to contain_apt_key('duplicate').with({
|
|
:id => title,
|
|
:ensure => 'present',
|
|
:source => nil,
|
|
:server => 'keyserver.ubuntu.com',
|
|
:content => nil,
|
|
:keyserver_options => nil,
|
|
})
|
|
end
|
|
|
|
it 'does not contain the original apt_key' do
|
|
is_expected.not_to contain_apt_key(title)
|
|
end
|
|
end
|
|
|
|
context 'two apt::key resources, different ensure' do
|
|
let :pre_condition do
|
|
"#{super()}\napt::key { 'duplicate': key => '#{title}', ensure => 'absent', }"
|
|
end
|
|
it 'informs the user of the impossibility' do
|
|
expect { subject.call }.to raise_error(/already ensured as absent/)
|
|
end
|
|
end
|
|
end
|
|
end
|
|
end
|