Options for exec is optional (#13)

Options are optional
timoxley · Mar 17, 2017 · c18c8c5 · c18c8c5
1 parent 7a5127b
commit c18c8c5
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -22,9 +22,10 @@ npmExec.execSync = npmExecSync
 module.exports = npmExec
 
 function npmExec (command, options, fn) {
-  var a = normalizeExecArgs(command, options)
+  var a = normalizeExecArgs(command, options, fn)
   command = a[0]
   options = a[1]
+  fn = a[2]
   return exec(command, augmentOptionsSync(options), fn)
 }
 

diff --git a/test/exec.js b/test/exec.js
@@ -30,6 +30,16 @@ test('passing args', function (t) {
   })
 })
 
+test('options are optional', function (t) {
+  var badPath = 'not-exist-adsjk'
+
+  npmRun(badPath, function (err, stdout, stderr) {
+    t.ok(err, 'has error')
+    t.equal(err.code, 127)
+    t.end()
+  })
+})
+
 test('includes all .bin dirs in all parent node_modules folders', function (t) {
   t.test('no nesting', function (t) {
     npmRun('level1', {cwd: level[0]}, function (err, stdout, stderr) {