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
Ensure PPAs in tests have valid form
[puppet-modules/puppetlabs-apt.git]
/
spec
/
defines
/
setting_spec.rb
diff --git
a/spec/defines/setting_spec.rb
b/spec/defines/setting_spec.rb
index 16c80163aa960af0f151585f19e64331baa1daec..a326bdbdd3239c400319eb199b127ef88f5fe6a7 100644
(file)
--- a/
spec/defines/setting_spec.rb
+++ b/
spec/defines/setting_spec.rb
@@
-2,7
+2,7
@@
require 'spec_helper'
describe 'apt::setting' do
let(:pre_condition) { 'class { "apt": }' }
describe 'apt::setting' do
let(:pre_condition) { 'class { "apt": }' }
- let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :lsbdistcodename => 'wheezy' } }
+ let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :lsbdistcodename => 'wheezy'
, :puppetversion => Puppet.version,
} }
let(:title) { 'conf-teddybear' }
let(:default_params) { { :content => 'di' } }
let(:title) { 'conf-teddybear' }
let(:default_params) { { :content => 'di' } }
@@
-16,25
+16,25
@@
describe 'apt::setting' do
context 'with title=conf-teddybear ' do
let(:params) { default_params }
context 'with title=conf-teddybear ' do
let(:params) { default_params }
- it { is_expected.to contain_file('/etc/apt/apt.conf.d/50teddybear').that_notifies('
Exec[apt_u
pdate]') }
+ it { is_expected.to contain_file('/etc/apt/apt.conf.d/50teddybear').that_notifies('
Class[Apt::U
pdate]') }
end
context 'with title=pref-teddybear' do
let(:title) { 'pref-teddybear' }
let(:params) { default_params }
end
context 'with title=pref-teddybear' do
let(:title) { 'pref-teddybear' }
let(:params) { default_params }
- it { is_expected.to contain_file('/etc/apt/preferences.d/
50teddybear').that_notifies('Exec[apt_u
pdate]') }
+ it { is_expected.to contain_file('/etc/apt/preferences.d/
teddybear.pref').that_notifies('Class[Apt::U
pdate]') }
end
context 'with title=list-teddybear' do
let(:title) { 'list-teddybear' }
let(:params) { default_params }
end
context 'with title=list-teddybear' do
let(:title) { 'list-teddybear' }
let(:params) { default_params }
- it { is_expected.to contain_file('/etc/apt/sources.list.d/teddybear.list').that_notifies('
Exec[apt_u
pdate]') }
+ it { is_expected.to contain_file('/etc/apt/sources.list.d/teddybear.list').that_notifies('
Class[Apt::U
pdate]') }
end
context 'with source' do
let(:params) { { :source => 'puppet:///la/die/dah' } }
it {
end
context 'with source' do
let(:params) { { :source => 'puppet:///la/die/dah' } }
it {
- is_expected.to contain_file('/etc/apt/apt.conf.d/50teddybear').that_notifies('
Exec[apt_u
pdate]').with({
+ is_expected.to contain_file('/etc/apt/apt.conf.d/50teddybear').that_notifies('
Class[Apt::U
pdate]').with({
:ensure => 'file',
:owner => 'root',
:group => 'root',
:ensure => 'file',
:owner => 'root',
:group => 'root',
@@
-45,7
+45,7
@@
describe 'apt::setting' do
context 'with content' do
let(:params) { default_params }
context 'with content' do
let(:params) { default_params }
- it { is_expected.to contain_file('/etc/apt/apt.conf.d/50teddybear').that_notifies('
Exec[apt_u
pdate]').with({
+ it { is_expected.to contain_file('/etc/apt/apt.conf.d/50teddybear').that_notifies('
Class[Apt::U
pdate]').with({
:ensure => 'file',
:owner => 'root',
:group => 'root',
:ensure => 'file',
:owner => 'root',
:group => 'root',
@@
-61,7
+61,7
@@
describe 'apt::setting' do
apt::setting { "list-teddybear": content => "foo" }
'
end
apt::setting { "list-teddybear": content => "foo" }
'
end
- let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :lsbdistcodename => 'wheezy' } }
+ let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :lsbdistcodename => 'wheezy'
, :puppetversion => Puppet.version,
} }
let(:title) { 'conf-teddybear' }
let(:default_params) { { :content => 'di' } }
let(:title) { 'conf-teddybear' }
let(:default_params) { { :content => 'di' } }
@@
-103,12
+103,12
@@
describe 'apt::setting' do
describe 'with priority=100' do
let(:params) { default_params.merge({ :priority => 100 }) }
describe 'with priority=100' do
let(:params) { default_params.merge({ :priority => 100 }) }
- it { is_expected.to contain_file('/etc/apt/apt.conf.d/100teddybear').that_notifies('
Exec[apt_u
pdate]') }
+ it { is_expected.to contain_file('/etc/apt/apt.conf.d/100teddybear').that_notifies('
Class[Apt::U
pdate]') }
end
describe 'with ensure=absent' do
let(:params) { default_params.merge({ :ensure => 'absent' }) }
end
describe 'with ensure=absent' do
let(:params) { default_params.merge({ :ensure => 'absent' }) }
- it { is_expected.to contain_file('/etc/apt/apt.conf.d/50teddybear').that_notifies('
Exec[apt_u
pdate]').with({
+ it { is_expected.to contain_file('/etc/apt/apt.conf.d/50teddybear').that_notifies('
Class[Apt::U
pdate]').with({
:ensure => 'absent',
})}
end
:ensure => 'absent',
})}
end