Merge pull request #1 from austgate/master

jhoblitt · Feb 3, 2014 · b46ee6a · b46ee6a
2 parents 95cb003 + e9e6361
commit b46ee6a
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 4 deletions.
diff --git a/manifests/params.pp b/manifests/params.pp
@@ -12,7 +12,7 @@
 #
 class sysstat::params {
   case $::osfamily {
-    'redhat': {
+    'debian', 'redhat': {
       $sysstat_package = 'sysstat'
     }
     default: {

diff --git a/spec/classes/sysstat_params_spec.rb b/spec/classes/sysstat_params_spec.rb
@@ -11,17 +11,27 @@
     it { should include_class('sysstat::params') }
   end
 
+  describe 'for osfamily Debian' do
+    let :facts do
+      {
+        :osfamily => 'Debian',
+      }
+    end
+
+    it { should include_class('sysstat::params') }
+  end
+
   describe 'unsupported osfamily' do
     let :facts do 
       {
-        :osfamily        => 'Debian',
-        :operatingsystem => 'Debian',
+        :osfamily        => 'Solaris',
+        :operatingsystem => 'Solaris',
       }
     end
 
     it 'should fail' do
       expect { should include_class('sysstat::params') }.
-        to raise_error(Puppet::Error, /not supported on Debian/)
+        to raise_error(Puppet::Error, /not supported on Solaris/)
     end
   end