Merge pull request #18 from aripalo/master

refreshonly true for killing vbox processes and add tests

Author: fromonesrc

manifests/init.pp

@@ -6,10 +6,9 @@
 class virtualbox {
 
   exec { 'Kill Virtual Box Processes':
-    command => 'pkill "VBoxXPCOMIPCD" || true && pkill "VBoxSVC" || true && pkill "VBoxHeadless" || true',
-    path    => '/usr/bin:/usr/sbin:/bin:/usr/local/bin',
-    #returns => [0,1],
-    #refreshonly => true,
+    command     => 'pkill "VBoxXPCOMIPCD" || true && pkill "VBoxSVC" || true && pkill "VBoxHeadless" || true',
+    path        => '/usr/bin:/usr/sbin:/bin:/usr/local/bin',
+    refreshonly => true,
   }
 
   package { 'VirtualBox-4.3.2-90405':

spec/classes/virtualbox_spec.rb

@@ -1,6 +1,13 @@
 require 'spec_helper'
 
 describe 'virtualbox' do
+  it do
+    should contain_exec('Kill Virtual Box Processes').with({
+      :command => 'pkill "VBoxXPCOMIPCD" || true && pkill "VBoxSVC" || true && pkill "VBoxHeadless" || true',
+      :path    => '/usr/bin:/usr/sbin:/bin:/usr/local/bin',
+      :refreshonly => true,
+    })
+  end
   it do
     should contain_package('VirtualBox-4.3.2-90405').with({
       :ensure   => 'installed',