add unit tests

manifests/config.pp

@@ -17,7 +17,7 @@
   # optional:  python-ldap, python-memcache, memcached, python-sqlite
 
   if ($::osfamily == 'RedHat' and $::operatingsystemrelease =~ /^7\.\d+/) or (
-  $::osfamily == 'Debian' and $::operatingsystemmajrelease =~ /8|15\.10/) {
+  $::graphite::params::service_provider == 'debian' and $::operatingsystemmajrelease =~ /8|15\.10/) {
     $initscript_notify = [Exec['graphite-reload-systemd'],]
 
     exec { 'graphite-reload-systemd':

manifests/config_apache.pp

@@ -91,8 +91,6 @@
     hasstatus  => true;
   }
 
-
-
   # Deploy configfiles
   file {
     "${::graphite::params::apache_dir}/ports.conf":

spec/classes/graphite_config_apache_spec.rb

@@ -9,10 +9,26 @@
 
   shared_context 'RedHat supported platforms' do
     it { is_expected.to contain_package('httpd').only_with({ :name => 'httpd', :ensure => 'installed' }) }
+    it { is_expected.to contain_package('mod_wsgi').only_with({ :name => 'mod_wsgi', :ensure => 'installed', 'require' => 'Package[httpd]'}) }
+    it { is_expected.to contain_service('httpd').only_with({
+        :name       => 'httpd',
+        :ensure     => 'running',
+        :enable     => 'true',
+        :hasrestart => 'true',
+        :hasstatus  => 'true'})
+    }
   end
 
   shared_context 'Debian supported platforms' do
     it { is_expected.to contain_package('apache2').only_with({ :name => 'apache2', :ensure => 'installed' }) }
+    it { is_expected.to contain_package('libapache2-mod-wsgi').only_with({ :name => 'libapache2-mod-wsgi', :ensure => 'installed', 'require' => 'Package[apache2]'}) }
+    it { is_expected.to contain_service('apache2').only_with({
+        :name       => 'apache2',
+        :ensure     => 'running',
+        :enable     => 'true',
+        :hasrestart => 'true',
+        :hasstatus  => 'true'})
+    }
   end
 
   context 'Unsupported OS' do

spec/classes/graphite_install_spec.rb

@@ -53,6 +53,7 @@
     it { is_expected.to contain_package('pyOpenSSL').with_provider(nil) }
     it { is_expected.to contain_package('python-memcached').with_provider(nil) }
     it { is_expected.to contain_package('python-zope-interface').with_provider(nil) }
+    it { is_expected.to contain_package('python-tzlocal').with_provider(nil) }
   end
 
   shared_context 'RedHat 6 platforms' do

spec/classes/graphite_spec.rb

@@ -12,12 +12,13 @@
       let(:facts) do
         facts
       end
-    
+
       it { is_expected.to compile.with_all_deps }
+      it { is_expected.to contain_class('graphite::params') }
       it { is_expected.to contain_anchor('graphite::begin').that_comes_before('Class[graphite::install]') }
       it { is_expected.to contain_class('graphite::install').that_notifies('Class[graphite::config]') }
       it { is_expected.to contain_class('graphite::config').that_comes_before('Anchor[graphite::end]') }
-      it { is_expected.to contain_anchor('graphite::end') }      
+      it { is_expected.to contain_anchor('graphite::end') }
 
     end
   end