#2007 - Introduce nodejs module

modules/nodejs/manifests/package.pp

 # == Type: nodejs::package
 #
 # Manage nodejs packages.
 #
 # === Parameters:
 #
 # [*ensure*]
 #  Translated directly into the state of installed/uninstalled
 #  package.
 #
-# [*options*]
-#  A list of zero or more options to install the package.
-#
 define nodejs::package (
   $ensure = 'present',
-  $options = [],
 ) {
 
   $check_command = [
     "npm", "list",
     "--global",
     "--parseable",
     $name,
   ]
 
   if ensure_state($ensure) {
     $command = [
       "npm",
       "install", "--global",
-      $options,
       $title,
     ]
 
-    $onlyif = shellquote("test", "!", "`${check_command}`")

[mathias, 2017/07/11 18:24:42]

That doesn't make any sense: The backticks are escaped anyway. And they aren't
even necessary.

+    $unless = shellquote($check_command)
+    $onlyif = undef
   }
   else {
     $command = [
       "npm",
       "uninstall", "--global",
-      $options,
       $title,
     ]
 
-    $onlyif = shellquote("test", "`${check_command}`")
+    $unless = undef
+    $onlyif = shellquote($check_command)
   }
 
-  exec {"state_$title":
-    path => ["/usr/bin"],
+  exec {"nodejs_package_$title":
+    path => ["/usr/bin", "/bin"],
     command => shellquote($command),
     require => Package['nodejs'],
     onlyif => $onlyif,
+    unless => $unless,
   }
 }