Adding cwd param to all exec types and fixing rspec and beaker tests

Author: artberri

README.md

@@ -155,9 +155,9 @@ Sets the user that will install NVM.
 
 #### `home`
 
-Indicates the user's home. Only used when `manage_user` is set to `true`.
+Indicates the user's home. It needs to be an existing directory if the `manage_user` is not set to `true`.
 
-Default: `/home/${user}`.
+Default: `/home/${user}` (or `/root` if the user is `root`).
 
 #### `nvm_dir`
 
@@ -257,7 +257,7 @@ This module is CI tested against [open source Puppet](http://docs.puppetlabs.com
 - Ubuntu 12.04 and 14.04
 - Debian 7
 
-This module has been tested in a production OpenLogic7 (Centos7) server in Azure.
+This module has been tested in several production servers (OpenLogic7 (Centos7) and Ubuntu 14.04) in Azure.
 
 This module should also work properly in other distributions and operating systems, such as FreeBSD, Gentoo, and Amazon Linux, but is not formally tested on them.
 

manifests/init.pp

@@ -13,7 +13,10 @@
   $install_node        = $nvm::params::install_node,
 ) inherits ::nvm::params {
 
-  if $home == undef {
+  if $home == undef and $user == 'root' {
+    $final_home = '/root'
+  }
+  elsif $home == undef {
     $final_home = "/home/${user}"
   }
   else {

spec/acceptance/01_nvm_node_install_spec.rb

@@ -0,0 +1,48 @@
+require 'spec_helper_acceptance'
+
+describe 'nvm::node::install define' do
+
+  describe 'running puppet code' do
+    pp = <<-EOS
+        class { 'nvm':
+            user        => 'foo',
+            manage_user => true,
+        }
+
+        nvm::node::install { '4.3.1':
+            user    => 'foo',
+            default => true,
+        }
+
+        nvm::node::install { '0.10.40':
+            user    => 'foo',
+        }
+    EOS
+    let(:manifest) { pp }
+
+    it 'should work with no errors' do
+      apply_manifest(manifest, :catch_failures => true)
+    end
+
+    it 'should be idempotent' do
+      apply_manifest(manifest, :catch_changes => true)
+    end
+
+    describe command('su - foo -c ". /home/foo/.nvm/nvm.sh && nvm --version" -s /bin/bash') do
+      its(:exit_status) { should eq 0 }
+      its(:stdout) { should match /0.29.0/ }
+    end
+
+    describe command('su - foo -c ". /home/foo/.nvm/nvm.sh && node --version" -s /bin/bash') do
+      its(:exit_status) { should eq 0 }
+      its(:stdout) { should match /4.3.1/ }
+    end
+
+    describe command('su - foo -c ". /home/foo/.nvm/nvm.sh && nvm ls" -s /bin/bash') do
+      its(:exit_status) { should eq 0 }
+      its(:stdout) { should match /0.10.40/ }
+    end
+
+  end
+
+end

spec/acceptance/nvm_spec.rb spec/acceptance/02_nvm_spec.rbspec/acceptance/nvm_spec.rb renamed to spec/acceptance/02_nvm_spec.rb

@@ -6,6 +6,7 @@
     pp = <<-EOS
       class { 'nvm':
         user => 'root',
+        home => '/root',
         nvm_dir => '/opt/nvm',
         version => 'v0.29.0',
         profile_path => '/etc/profile.d/nvm.sh',