support spaces

Author: 75lb

dist/index.cjs

@@ -121,8 +121,8 @@ class Explain extends JsdocCommand {
 
   async _runJsdoc () {
     const cmd = this.options.source.length
-      ? `node ${this.jsdocPath} ${toSpawnArgs(this.jsdocOptions).join(' ')} -X ${this.tempFileSet.files.join(' ')}`
-      : `node ${this.jsdocPath} ${toSpawnArgs(this.jsdocOptions).join(' ')} -X ${this.inputFileSet.files.join(' ')}`;
+      ? `node "${this.jsdocPath}" ${toSpawnArgs(this.jsdocOptions).join(' ')} -X ${this.tempFileSet.files.join(' ')}`
+      : `node "${this.jsdocPath}" ${toSpawnArgs(this.jsdocOptions).join(' ')} -X ${this.inputFileSet.files.join(' ')}`;
 
     let jsdocOutput = { stdout: '', stderr: '' };
     try {

lib/explain.js

@@ -24,8 +24,8 @@ class Explain extends JsdocCommand {
 
   async _runJsdoc () {
     const cmd = this.options.source.length
-      ? `node ${this.jsdocPath} ${toSpawnArgs(this.jsdocOptions).join(' ')} -X ${this.tempFileSet.files.join(' ')}`
-      : `node ${this.jsdocPath} ${toSpawnArgs(this.jsdocOptions).join(' ')} -X ${this.inputFileSet.files.join(' ')}`
+      ? `node "${this.jsdocPath}" ${toSpawnArgs(this.jsdocOptions).join(' ')} -X ${this.tempFileSet.files.join(' ')}`
+      : `node "${this.jsdocPath}" ${toSpawnArgs(this.jsdocOptions).join(' ')} -X ${this.inputFileSet.files.join(' ')}`
 
     let jsdocOutput = { stdout: '', stderr: '' }
     try {

test/explain.js

@@ -49,4 +49,11 @@ test.set('.explain({ files }): files is empty', async function () {
   )
 })
 
+test.set('Spaces in jsdoc command path', async function () {
+  process.env.JSDOC_PATH = 'test/fixture/folder with spaces/jsdoc'
+  const f = new Fixture('class-all')
+  let output = await jsdoc.explain({ files: f.sourcePath })
+  a.ok(/a class with all of the things/.test(JSON.stringify(output)))
+})
+
 export { test, only, skip }