lint - remove legacy facts

Author: traylenator

manifests/params.pp

@@ -4,12 +4,12 @@
   $default_user   = 'root'
   $package_ensure = 'installed'
 
-  case $::osfamily {
+  case $facts['os']['family'] {
     'FreeBSD': { $default_group = 'wheel' }
     default: { $default_group = 'root' }
   }
 
-  case $::osfamily {
+  case $facts['os']['family'] {
     'Debian':  {
       $confdir            = '/etc/xinetd.d'
       $conffile           = '/etc/xinetd.conf'
@@ -71,7 +71,7 @@
       $service_status     = undef
     }
     'Linux': {
-      case $::operatingsystem {
+      case $facts['os']['name'] {
         'Amazon': {
           $confdir            = '/etc/xinetd.d'
           $conffile           = '/etc/xinetd.conf'
@@ -83,12 +83,12 @@
           $service_status     = undef
         }
         default: {
-          fail("xinetd: module does not support Linux operatingsystem ${::operatingsystem}")
+          fail("xinetd: module does not support Linux operatingsystem ${facts['os']['name']}")
         }
       }
     }
     default:   {
-      fail("xinetd: module does not support osfamily ${::osfamily}")
+      fail("xinetd: module does not support osfamily ${facts['os']['family']}")
     }
   }
 

manifests/service.pp

@@ -98,7 +98,7 @@
   $redirect                         = undef,
 ) {
 
-  include ::xinetd
+  include xinetd
 
   unless ($server or $redirect) {
     fail('xinetd::service needs either of server or redirect')

spec/classes/xinetd_init_spec.rb

@@ -4,7 +4,7 @@
 
   context 'When using default values' do
     let :facts do
-      { :osfamily => 'Debian' }
+      { operatingsystem: 'Debian',  osfmaily: 'Debian', os: { 'family' => 'Debian' } }
     end
     it {
       should contain_package('xinetd')
@@ -39,7 +39,7 @@
 
   context 'When overriding the default vaules' do
     let :facts do
-      { :osfamily => 'Debian' }
+      { operatingsystem: 'Debian',  osfmaily: 'Debian', os: { 'family' => 'Debian' }}
     end
     let :params do
       { :enabled        => 'tftp nrpe',
@@ -93,7 +93,7 @@
 
   context 'with defaults on Linux' do
     let :facts do
-      { :osfamily => 'Debian' }
+      { operatingsystem: 'Debian',  osfmaily: 'Debian', os: { 'family' => 'Debian' } }
     end
     it {
       should contain_package('xinetd')
@@ -105,7 +105,7 @@
 
   context 'with defaults on FreeBSD' do
     let :facts do
-      { :osfamily => 'FreeBSD' }
+      { operatingsystem: 'FreeBSD',  osfmaily: 'FreeBSD', os: { 'family' => 'FreeBSD' } }
     end
     it {
       should contain_package('security/xinetd')
@@ -117,7 +117,7 @@
 
   context 'with managed confdir' do
     let :facts do
-      { :osfamily => 'Debian' }
+      { operatingsystem: 'Debian',  osfmaily: 'Debian', os: { 'family' => 'Debian' } }
     end
     let :params do
       { :purge_confdir => true }

spec/defines/xinetd_service_spec.rb

@@ -15,7 +15,7 @@
 
   describe "ensure proper user/group are set in FreeBSD" do
     let :facts do
-      { :osfamily => 'FreeBSD' }
+      { operatingsystem: 'FreeBSD',  osfamily: 'FreeBSD', os: { 'family' => 'FreeBSD' }}
     end
 
     let :params do
@@ -29,7 +29,7 @@
   end
 
   let :facts do
-    { :osfamily => 'Debian' }
+    { operatingsystem: 'Debian',  osfamily: 'Debian', os: { 'family' => 'Debian' }}
   end
 
   describe 'with default ensure' do