nut: add spec test

manifests/plugin/nut.pp

@@ -4,7 +4,7 @@
   $upss       = { },
 ) {
 
-  include ::collectd
+  include collectd
 
 
   collectd::plugin { 'nut':

manifests/plugin/nut/ups.pp

@@ -3,18 +3,16 @@
   $ensure          = 'present',
 ) {
 
-  include ::collectd
-  include ::collectd::plugin::nut
+  include collectd
+  include collectd::plugin::nut
 
-  $conf_dir = $::collectd::plugin_conf_dir
-
-  validate_string($name)
+  $conf_dir = $collectd::plugin_conf_dir
 
   file { "${conf_dir}/nut-ups-${name}.conf":
     ensure  => $ensure,
-    mode    => '0640',
-    owner   => 'root',
-    group   => $::collectd::root_group,
+    mode    => $collectd::config_mode,
+    owner   => $collectd::config_owner,
+    group   => $collectd::config_group,
     content => template('collectd/plugin/nut/ups.conf.erb'),
     notify  => Service['collectd'],
   }

spec/classes/collectd_plugin_nut_spec.rb

@@ -0,0 +1,36 @@
+require 'spec_helper'
+
+describe 'collectd::plugin::nut', type: :class do
+  let :pre_condition do
+    'include ::collectd'
+  end
+  on_supported_os(baseline_os_hash).each do |os, facts|
+    context "on #{os} " do
+      let :facts do
+        facts
+      end
+
+
+      context ':ensure => present, default params' do
+        it 'Will create /etc/collectd.d/10-nut.conf' do
+          is_expected.to contain_file('nut.load').
+            with(ensure: 'present',
+                 path: '/etc/collectd.d/10-nut.conf',
+                 content: %r{LoadPlugin nut})
+        end
+      end
+
+      context ':ensure => present, single entry' do
+        let :params do
+          { upss: [ 'ups1@localhost' ] }
+        end
+        it 'Will create /etc/collectd.d/nut-ups-ups1@localhost.conf' do
+          is_expected.to contain_file('/etc/collectd.d/nut-ups-ups1@localhost.conf').
+            with(ensure: 'present',
+                 path: '/etc/collectd.d/nut-ups-ups1@localhost.conf',
+                 content: %r{UPS "ups1@localhost"})
+        end
+      end
+    end
+  end
+end