fix: remove extraneous space when no additional args are passed (#82)

Author: nlf

lib/make-spawn-args.js

@@ -63,14 +63,20 @@ const makeSpawnArgs = options => {
 
     scriptFile = resolve(tmpdir(), `${event}-${Date.now()}.cmd`)
     script += '@echo off\n'
-    script += `${cmd} ${args.map((arg) => escape.cmd(arg, doubleEscape)).join(' ')}`
+    script += cmd
+    if (args.length) {
+      script += ` ${args.map((arg) => escape.cmd(arg, doubleEscape)).join(' ')}`
+    }
   } else {
     const shebang = isAbsolute(scriptShell)
       ? `#!${scriptShell}`
       : `#!/usr/bin/env ${scriptShell}`
     scriptFile = resolve(tmpdir(), `${event}-${Date.now()}.sh`)
     script += `${shebang}\n`
-    script += `${cmd} ${args.map((arg) => escape.sh(arg)).join(' ')}`
+    script += cmd
+    if (args.length) {
+      script += ` ${args.map((arg) => escape.sh(arg)).join(' ')}`
+    }
   }
 
   writeFile(scriptFile, script)

test/make-spawn-args.js

@@ -82,8 +82,7 @@ if (isWindows) {
       }, 'got expected options')
 
       const contents = fs.readFileSync(args[args.length - 1], { encoding: 'utf8' })
-      // the contents will have a trailing space if no args are passed
-      t.equal(contents, `@echo off\nscript "quoted parameter"; second command `)
+      t.equal(contents, `@echo off\nscript "quoted parameter"; second command`)
       t.ok(fs.existsSync(args[args.length - 1]), 'script file was written')
       cleanup()
       t.not(fs.existsSync(args[args.length - 1]), 'cleanup removes script file')