7c0d10b392
This commit changes $release to default to Facter's $lsbdistcodename and fall back to a Parse Error if $release is not set and $lsbdistcodename does not exist. Previously $release was hardcoded to karmic. This commit also modifies apt::ppa to use $release and sets the files to be ensured so that they are not purged when purge_sources_list_d is set to true.
53 lines
1.4 KiB
Ruby
53 lines
1.4 KiB
Ruby
require 'spec_helper'
|
|
describe 'apt::ppa', :type => :define do
|
|
['ppa:dans_ppa', 'dans_ppa','ppa:dans-daily/ubuntu'].each do |t|
|
|
describe "with title #{t}" do
|
|
let :pre_condition do
|
|
'class { "apt": }'
|
|
end
|
|
let :facts do
|
|
{:lsbdistcodename => 'natty'}
|
|
end
|
|
let :title do
|
|
t
|
|
end
|
|
let :release do
|
|
"natty"
|
|
end
|
|
let :filename do
|
|
t.sub(/^ppa:/,'').gsub('/','-') << "-" << "#{release}.list"
|
|
end
|
|
|
|
it { should contain_exec("apt-update-#{t}").with(
|
|
'command' => '/usr/bin/aptitude update',
|
|
'refreshonly' => true
|
|
)
|
|
}
|
|
|
|
it { should contain_exec("add-apt-repository-#{t}").with(
|
|
'command' => "/usr/bin/add-apt-repository #{t}",
|
|
'notify' => "Exec[apt-update-#{t}]",
|
|
'creates' => "/etc/apt/sources.list.d/#{filename}"
|
|
)
|
|
}
|
|
|
|
it { should create_file("/etc/apt/sources.list.d/#{filename}").with(
|
|
'ensure' => 'file',
|
|
'require' => "Exec[add-apt-repository-#{t}]"
|
|
)
|
|
}
|
|
end
|
|
end
|
|
|
|
describe "without Class[apt] should raise a Puppet::Error" do
|
|
let(:release) { "natty" }
|
|
let(:title) { "ppa" }
|
|
it { expect { should contain_apt__ppa(title) }.to raise_error(Puppet::Error) }
|
|
end
|
|
|
|
describe "without release should raise a Puppet::Error" do
|
|
let(:title) { "ppa:" }
|
|
it { expect { should contain_apt__ppa(:release) }.to raise_error(Puppet::Error) }
|
|
end
|
|
|
|
end
|