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 branch 'proxy'
[puppet-modules/puppetlabs-apt.git]
/
spec
/
classes
/
apt_spec.rb
diff --git
a/spec/classes/apt_spec.rb
b/spec/classes/apt_spec.rb
index 6ab1e9e6d64de8186d01b4585dbe0aaeb8bc7ea1..a66feac78810ce3175580a2dd5ee4b7298fcd2ea 100644
(file)
--- a/
spec/classes/apt_spec.rb
+++ b/
spec/classes/apt_spec.rb
@@
-41,8
+41,6
@@
describe 'apt', :type => :class do
it { should include_class("apt::params") }
it { should include_class("apt::params") }
- it { should contain_package("python-software-properties") }
-
it {
if param_hash[:purge_sources_list]
should contain_file("sources.list").with({
it {
if param_hash[:purge_sources_list]
should contain_file("sources.list").with({
@@
-50,7
+48,7
@@
describe 'apt', :type => :class do
'ensure' => "present",
'owner' => "root",
'group' => "root",
'ensure' => "present",
'owner' => "root",
'group' => "root",
- 'mode' =>
644
,
+ 'mode' =>
"0644"
,
"content" => "# Repos managed by puppet.\n"
})
else
"content" => "# Repos managed by puppet.\n"
})
else
@@
-59,7
+57,7
@@
describe 'apt', :type => :class do
'ensure' => "present",
'owner' => "root",
'group' => "root",
'ensure' => "present",
'owner' => "root",
'group' => "root",
- 'mode' =>
644
,
+ 'mode' =>
"0644"
,
'content' => nil
})
end
'content' => nil
})
end
@@
-72,7
+70,8
@@
describe 'apt', :type => :class do
'owner' => "root",
'group' => "root",
'purge' => true,
'owner' => "root",
'group' => "root",
'purge' => true,
- 'recurse' => true
+ 'recurse' => true,
+ 'notify' => 'Exec[apt_update]'
})
else
should create_file("sources.list.d").with({
})
else
should create_file("sources.list.d").with({
@@
-81,7
+80,8
@@
describe 'apt', :type => :class do
'owner' => "root",
'group' => "root",
'purge' => false,
'owner' => "root",
'group' => "root",
'purge' => false,
- 'recurse' => false
+ 'recurse' => false,
+ 'notify' => 'Exec[apt_update]'
})
end
}
})
end
}
@@
-89,7
+89,6
@@
describe 'apt', :type => :class do
it {
should contain_exec("apt_update").with({
'command' => "/usr/bin/apt-get update",
it {
should contain_exec("apt_update").with({
'command' => "/usr/bin/apt-get update",
- 'subscribe' => ["File[sources.list]", "File[sources.list.d]"],
'refreshonly' => refresh_only_apt_update
})
}
'refreshonly' => refresh_only_apt_update
})
}
@@
-117,10
+116,15
@@
describe 'apt', :type => :class do
if param_hash[:proxy_host]
should contain_file('configure-apt-proxy').with(
'path' => '/etc/apt/apt.conf.d/proxy',
if param_hash[:proxy_host]
should contain_file('configure-apt-proxy').with(
'path' => '/etc/apt/apt.conf.d/proxy',
- 'content' => "Acquire::http::Proxy \"http://#{param_hash[:proxy_host]}:#{param_hash[:proxy_port]}\";"
+ 'content' => "Acquire::http::Proxy \"http://#{param_hash[:proxy_host]}:#{param_hash[:proxy_port]}\";",
+ 'notify' => "Exec[apt_update]"
)
else
)
else
- should_not contain_file('configure_apt_proxy')
+ should contain_file('configure-apt-proxy').with(
+ 'path' => '/etc/apt/apt.conf.d/proxy',
+ 'notify' => 'Exec[apt_update]',
+ 'ensure' => 'absent'
+ )
end
}
end
end
}
end