Merge pull request #23 from juanibiapina/execuser

Add execuser param to run exec as that user

Author: Ashley Penney

.gemfile

@@ -1,2 +1,4 @@
 metadata.json
 pkg/
+Gemfile.lock
+.rspec

.gitignore

@@ -0,0 +1,4 @@
+source "https://rubygems.org"
+
+gem 'puppetlabs_spec_helper'
+gem 'puppet'

Gemfile

@@ -8,13 +8,14 @@ providing defines to easily grab data via rsync.
 get files via rsync
 
 ## Parameters: ##
-    $source  - source to copy from
-    $path    - path to copy to, defaults to $name
-    $user    - username on remote system
-    $purge   - if set, rsync will use '--delete'
-    $exlude  - string to be excluded
-    $keyfile - ssh key used to connect to remote host
-    $timeout - timeout in seconds, defaults to 900
+    $source   - source to copy from
+    $path     - path to copy to, defaults to $name
+    $user     - username on remote system
+    $purge    - if set, rsync will use '--delete'
+    $exlude   - string to be excluded
+    $keyfile  - ssh key used to connect to remote host
+    $timeout  - timeout in seconds, defaults to 900
+    $execuser - user to run the command (passed to exec)
 
 ## Actions: ##
   get files via rsync

README.markdown

@@ -31,7 +31,8 @@
   $purge = undef,
   $exclude = undef,
   $keyfile = undef,
-  $timeout = '900'
+  $timeout = '900',
+  $execuser = 'root',
 ) {
 
   if $keyfile {
@@ -63,6 +64,7 @@
   exec { "rsync ${name}":
     command => "rsync -q ${rsync_options}",
     path    => [ '/bin', '/usr/bin' ],
+    user => $execuser,
     # perform a dry-run to determine if anything needs to be updated
     # this ensures that we only actually create a Puppet event if something needs to
     # be updated

manifests/get.pp

@@ -19,11 +19,20 @@
       should contain_exec("rsync foobar").with({
         'command' => 'rsync -q -a   example.com foobar',
         'onlyif'  => "test `rsync --dry-run --itemize-changes -a   example.com foobar | wc -l` -gt 0",
-        'timeout' => '900'
+        'timeout' => '900',
+        'user'    => 'root'
       })
     }
   end
 
+  describe "when setting the execuser" do
+    let :params do
+      common_params.merge( { :execuser => 'username' } )
+    end
+
+    it{ should contain_exec("rsync foobar").with({ 'user' => 'username' }) }
+  end
+
   describe "when setting the timeout" do
     let :params do
       common_params.merge( { :timeout => '200' } )

spec/defines/get_spec.rb

@@ -64,7 +64,7 @@
       let :params do
         mandatory_params.merge({ k => v })
       end
-      it { should contain_file(fragment_file).with_content(/^#{k.to_s.gsub('_', ' ')}\s*=\s*#{v.to_a.join(' ')}$/)}
+      it { should contain_file(fragment_file).with_content(/^#{k.to_s.gsub('_', ' ')}\s*=\s*#{Array(v).join(' ')}$/)}
     end
   end