ppa: fix empty environment definition in exec ressource when no proxy
$proxy_host = getparam(Class[apt], "proxy_host")
$proxy_port = getparam(Class[apt], "proxy_port")
case $proxy_host {
- false: {
- $proxy_env = ""
+ false, "": {
+ $proxy_env = []
}
default: {$proxy_env = ["http_proxy=http://${proxy_host}:${proxy_port}", "https_proxy=http://${proxy_host}:${proxy_port}"]}
}
} else {
- $proxy_env = ""
+ $proxy_env = []
}
exec { "add-apt-repository-${name}":
environment => $proxy_env,
}
end
end
+ describe 'without a proxy defined' do
+ let :title do
+ 'rspec_ppa'
+ end
+ let :pre_condition do
+ 'class { "apt":
+ proxy_host => false
+ }'
+ end
+ let :filename do
+ "#{title}-#{release}.list"
+ end
+
+ it { should contain_exec("add-apt-repository-#{title}").with(
+ 'environment' => [],
+ 'command' => "/usr/bin/add-apt-repository #{title}",
+ 'creates' => "/etc/apt/sources.list.d/#{filename}",
+ 'require' => ["File[/etc/apt/sources.list.d]", "Package[#{package}]"],
+ 'notify' => "Exec[apt_update]"
+ )
+ }
+ end
+
describe 'behind a proxy' do
let :title do
'rspec_ppa'