Always apply dav_svn/authz_svn work-around

The work-around that distros apply for dav_svn/authz_svn module loading
issue is overwritten by this module. Always apply the work-around.

Author: martijndegouw

manifests/mod/dav_svn.pp

@@ -21,15 +21,11 @@
 
   ::apache::mod { 'dav_svn': }
 
-  if $::osfamily == 'Debian' and ! ($::operatingsystemmajrelease in ['6', '9', '16.04', '18.04']) {
-    $loadfile_name = undef
-  } else {
-    $loadfile_name = 'dav_svn_authz_svn.load'
-  }
-
   if $authz_svn_enabled {
     ::apache::mod { 'authz_svn':
-      loadfile_name => $loadfile_name,
+      # authz_svn depends on symbols from the dav_svn module,
+      # therefore, make sure authz_svn is loaded after dav_svn.
+      loadfile_name => 'dav_svn_authz_svn.load',
       require       => Apache::Mod['dav_svn'],
     }
   }

spec/classes/mod/dav_svn_spec.rb

@@ -24,7 +24,7 @@
         it { is_expected.to contain_apache__mod('dav_svn') }
         it { is_expected.to contain_package('libapache2-svn') }
         it { is_expected.to contain_apache__mod('authz_svn') }
-        it { is_expected.to contain_file('authz_svn.load').with_content(%r{LoadModule authz_svn_module}) }
+        it { is_expected.to contain_file('dav_svn_authz_svn.load').with_content(%r{LoadModule authz_svn_module}) }
       end
     end
     context 'on a RedHat OS' do