#2007 - Introduce nodejs module

modules/nodejs/manifests/package.pp

 # == Type: nodejs::package
 #
 # Manage nodejs packages.
 #
 # === Parameters:
 #
-# [*options*]
-#  A list of zero or more options to install the package.
+# [*ensure*]
+#  Translated directly into the state of installed/uninstalled
+#  package.
 #
 define nodejs::package (

[mathias, 2017/07/09 07:26:50]

Shouldn't there be some $ensure parameter in order to allow for un-installing an
NPM package?

-  $options = [],
+  $ensure = 'present',
 ) {
 
-  $install_command = [
-    "npm", "install",
-    $options,
-    $title,
+  $check_command = [
+    "npm", "list",
+    "--global",
+    "--parseable",
+    $name,
   ]
 
-  exec {"install_$title":
-    path => ["/usr/bin"],
-    command => shellquote($install_command),
+  if ensure_state($ensure) {
+    $command = [
+      "npm",
+      "install", "--global",
+      $title,
+    ]
+
+    $unless = shellquote($check_command)
+    $onlyif = undef
+  }
+  else {
+    $command = [
+      "npm",
+      "uninstall", "--global",
+      $title,
+    ]
+
+    $unless = undef
+    $onlyif = shellquote($check_command)
+  }
+
+  exec {"nodejs_package_$title":
+    path => ["/usr/bin", "/bin"],
+    command => shellquote($command),
     require => Package['nodejs'],
-    onlyif => "test ! -x /usr/bin/${title}",

[mathias, 2017/07/09 07:26:50]

Please use the 'creates' parameter instead.

+    onlyif => $onlyif,
+    unless => $unless,
   }
-}
+}
+