fix: make opts optional (#3)

npm · Mar 3, 2022 · 50c981a · 50c981a
1 parent 2040e04
commit 50c981a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -6,7 +6,7 @@ const isPipe = (stdio = 'pipe', fd) =>
   : false
 
 // 'extra' object is for decorating the error a bit more
-const promiseSpawn = (cmd, args, opts, extra = {}) => {
+const promiseSpawn = (cmd, args, opts = {}, extra = {}) => {
   const cwd = opts.cwd || process.cwd()
   return promiseSpawnUid(cmd, args, {
     ...opts,

diff --git a/test/promise-spawn.js b/test/promise-spawn.js
@@ -115,6 +115,11 @@ t.test('pass', t => t.resolveMatch(promiseSpawn('pass', [], {stdioString: true},
   a: 1,
 }))
 
+t.test('pass, default opts', t => t.resolveMatch(promiseSpawn('pass', []), {
+  code: 0,
+  signal: null
+}))
+
 t.test('pass, share stdio', t => t.resolveMatch(promiseSpawn('pass', [], { stdio: 'inherit'}, {a: 1}), {
   code: 0,
   signal: null,