Skip to content

Commit

Permalink
Bad dependency to .load files in apache::mod
Browse files Browse the repository at this point in the history
Fixes error "expected that the catalogue would include File[/etc/apache2/mods-available/proxy_html.load]"
Add compilation specs
  • Loading branch information
Carlos Sanchez committed Mar 11, 2014
1 parent df5df92 commit 5960089
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
4 changes: 2 additions & 2 deletions manifests/mod.pp
Original file line number Diff line number Diff line change
Expand Up @@ -53,10 +53,10 @@
# the module gets installed.
$package_before = $::osfamily ? {
'freebsd' => [
File["${mod_dir}/${mod}.load"],
File["${mod}.load"],
File["${::apache::params::conf_dir}/${::apache::params::conf_file}"]
],
default => File["${mod_dir}/${mod}.load"],
default => File["${mod}.load"],
}
# $_package may be an array
package { $_package:
Expand Down
6 changes: 3 additions & 3 deletions spec/classes/mod/auth_kerb_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
let :pre_condition do
'include apache'
end
context "on a Debian OS" do
context "on a Debian OS", :compile do
let :facts do
{
:osfamily => 'Debian',
Expand All @@ -14,7 +14,7 @@
it { should contain_apache__mod("auth_kerb") }
it { should contain_package("libapache2-mod-auth-kerb") }
end
context "on a RedHat OS" do
context "on a RedHat OS", :compile do
let :facts do
{
:osfamily => 'RedHat',
Expand All @@ -26,7 +26,7 @@
it { should contain_apache__mod("auth_kerb") }
it { should contain_package("mod_auth_kerb") }
end
context "on a FreeBSD OS" do
context "on a FreeBSD OS", :compile do
let :facts do
{
:osfamily => 'FreeBSD',
Expand Down
6 changes: 3 additions & 3 deletions spec/classes/mod/proxy_html_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
'include apache::mod::proxy_http',
]
end
context "on a Debian OS" do
context "on a Debian OS", :compile do
let :facts do
{
:osfamily => 'Debian',
Expand All @@ -18,7 +18,7 @@
it { should contain_apache__mod('proxy_html') }
it { should contain_package("libapache2-mod-proxy-html") }
end
context "on a RedHat OS" do
context "on a RedHat OS", :compile do
let :facts do
{
:osfamily => 'RedHat',
Expand All @@ -30,7 +30,7 @@
it { should contain_apache__mod('proxy_html') }
it { should contain_package("mod_proxy_html") }
end
context "on a FreeBSD OS" do
context "on a FreeBSD OS", :compile do
let :facts do
{
:osfamily => 'FreeBSD',
Expand Down

0 comments on commit 5960089

Please sign in to comment.