test: assert for exit code (#137)

* chore: bump cli-prompts-test

* test: assert for exit code

Author: jamesgeorge007

e2e/commands/generate.test.js

@@ -26,6 +26,10 @@ test('generate _sidebar.md', t => {
   // Check for existence
   t.true(fs.existsSync(path.join(docsPath, '_sidebar.md')))
 
-  const {stderr} = run(['generate', 'docs'], {cwd: genPath})
+  const {exitCode, stderr} = run(['generate', 'docs'], {
+    cwd: genPath,
+    reject: false
+  })
+  t.is(exitCode, 1)
   t.is(stderr, 'The sidebar file \'_sidebar.md\' already exists.')
 })

e2e/commands/init.test.js

@@ -22,13 +22,22 @@ test.after('cleanup', () => {
 })
 
 test('generates docs directory', t => {
-  run(['init', 'docs'], {cwd: genPath})
+  const {exitCode} = run(['init', 'docs'], {cwd: genPath})
+
+  // Assert for exit code
+  t.is(exitCode, 0)
+
   // Check for existence
   t.true(fs.existsSync(path.join(docsPath, 'README.md')))
   t.true(fs.existsSync(path.join(docsPath, 'index.html')))
 })
 
 test('force generates docs directory with --local flag', async t => {
-  await runPromptWithAnswers(['init', 'docs', '--local'], ['y', ENTER], genPath)
+  const {exitCode} = await runPromptWithAnswers(
+    ['init', 'docs', '--local'],
+    ['y', ENTER],
+    genPath
+  )
+  t.is(exitCode, 0)
   t.true(fs.existsSync(path.join(docsPath, 'vendor')))
 })

lib/commands/generate.js

@@ -22,6 +22,7 @@ module.exports = function (path = '', sidebar) {
       }
 
       console.error(chalk.red(`The sidebar file '${sidebar}' already exists.`))
+      process.exitCode = 1
       return false
     }
   }

package-lock.json

@@ -72,7 +72,7 @@
   },
   "devDependencies": {
     "ava": "^3.13.0",
-    "cli-prompts-test": "^0.2.0",
+    "cli-prompts-test": "^0.3.0",
     "cors": "^2.8.1",
     "eslint": "^6.8.0",
     "eslint-config-xo-space": "^0.22.0",