Merge pull request #108 from jearls/redhat-does-not-need-dnssec-tools

Remove dnssec-tools from RedHat package list.
ajjahn · May 6, 2015 · 186032c · 186032c
2 parents 94e1841 + e8c3850
commit 186032c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 8 deletions.
diff --git a/manifests/server/params.pp b/manifests/server/params.pp
@@ -20,14 +20,7 @@
       $owner              = 'named'
       $package            = 'bind'
       $service            = 'named'
-      case $::operatingsystemmajrelease {
-        '7': {
-          $necessary_packages = [ 'bind', ]
-        }
-        default: {
-          $necessary_packages = [ 'bind', 'dnssec-tools' ]
-        }
-      }
+      $necessary_packages = [ 'bind', ]
     }
     default: {
       fail("dns::server is incompatible with this osfamily: ${::osfamily}")

diff --git a/spec/classes/dns__server__install_spec.rb b/spec/classes/dns__server__install_spec.rb
@@ -13,5 +13,11 @@
     end
   end
 
+  context "on a RedHat OS" do
+    let(:facts) {{ :osfamily => 'RedHat' }}
+    it { should contain_class('dns::server::params') }
+    it { should contain_package('bind') }
+  end
+
 end