Merge pull request #1 from eshamow/cleanup_lint_errors

Cleaned up lint errors.

manifests/config.pp

@@ -5,7 +5,7 @@
 #
 # Parameters:
 #  - $mode (enforced|permissive|disabled) - sets the operating state for SELinux.
-# 
+#
 # Actions:
 #  Configures SELinux to a specific state (enforced|permissive|disabled)
 #
@@ -34,7 +34,7 @@
     }
 
     case $mode {
-      permissive,disabled: { 
+      permissive,disabled: {
         $sestatus = '0'
         if $mode == 'disabled' and $::selinux_current_mode == 'permissive' {
           notice('A reboot is required to fully disable SELinux. SELinux will operate in Permissive mode until a reboot')
@@ -43,6 +43,9 @@
       enforcing: {
         $sestatus = '1'
       }
+      default : {
+        fail('You must specify a mode (enforced, permissive, or disabled) for selinux operation')
+      }
     }
 
     exec { "change-selinux-status-to-${mode}":

manifests/init.pp

@@ -8,7 +8,7 @@
 #
 # Actions:
 #  This module will configure SELinux and/or deploy SELinux based modules to running
-#  system. 
+#  system.
 #
 # Requires:
 #  - Class[stdlib]. This is Puppet Labs standard library to include additional methods for use within Puppet. [https://github.com/puppetlabs/puppetlabs-stdlib]
@@ -24,7 +24,7 @@
 
   anchor { 'selinux::begin': }
   -> class { 'selinux::config':
-       mode => $mode,
+      mode => $mode,
   }
   -> anchor { 'selinux::end': }
 }

manifests/module.pp

@@ -1,11 +1,11 @@
 # Definition: selinux::module
 #
 # Description
-#  This class will either install or uninstall a SELinux module from a running system. 
-#  This module allows an admin to keep .te files in text form in a repository, while 
-#  allowing the system to compile and manage SELinux modules.   
+#  This class will either install or uninstall a SELinux module from a running system.
+#  This module allows an admin to keep .te files in text form in a repository, while
+#  allowing the system to compile and manage SELinux modules.
 #
-#  Concepts incorporated from: 
+#  Concepts incorporated from:
 #  http://stuckinadoloop.wordpress.com/2011/06/15/puppet-managed-deployment-of-selinux-modules/
 #
 # Parameters:
@@ -23,12 +23,12 @@
 # Sample Usage:
 #  selinux::module{ 'apache':
 #    ensure => 'present',
-#    source => 'puppet:///modules/selinux/apache.te', 
+#    source => 'puppet:///modules/selinux/apache.te',
 #  }
 #
 define selinux::module(
+  $source,
   $ensure  = 'present',
-  $source
 ) {
   # Set Resource Defaults
   File {
@@ -40,8 +40,8 @@
   # Only allow refresh in the event that the initial .te file is updated.
   Exec {
     path         => '/sbin:/usr/sbin:/bin:/usr/bin',
-    refreshonly  => 'true',
-    cwd          => "${selinux::params::sx_mod_dir}",
+    refreshonly  => true,
+    cwd          => $selinux::params::sx_mod_dir,
   }
 
   ## Begin Configuration

manifests/params.pp

@@ -1,3 +1,11 @@
+# Class: selinux::params
+#
+# Description
+#  This class provides default parameters for the selinux class
+#
+# Sample Usage:
+#  mod_dir = $selinux::params::sx_mod_dir
+#
 class selinux::params {
   $sx_mod_dir = '/usr/share/selinux'
 }

tests/disable.pp

@@ -1,9 +1,9 @@
-# Class: 
+# Class:
 #
 # Description
 #
 # Parameters:
-#   
+#
 # Actions:
 #
 # Requires:

tests/enable.pp

@@ -1,16 +1,16 @@
-# Class: 
+# Class:
 #
 # Description
 #
 # Parameters:
-#   
+#
 # Actions:
 #
 # Requires:
 #
 # Sample Usage:
 #
 
-class { 'selinux': 
+class { 'selinux':
   mode => 'enforcing',
 }

tests/module.pp

@@ -1,9 +1,9 @@
-# Class: 
+# Class:
 #
 # Description
 #
 # Parameters:
-#   
+#
 # Actions:
 #
 # Requires: