fix: handle errors without stack traces when using --verbose (#796)

* handle errors without stack traces when using --verbose

* remove test log

Author: MichaelGoberling

src/BaseCommand.js

@@ -35,7 +35,7 @@ class BaseCommand extends Command {
   async catch (error) {
     const { flags } = await this.parse(this.prototype)
     aioLogger.error(error) // debug log
-    this.error(flags.verbose ? error.stack : error.message)
+    this.error(flags.verbose && error.stack ? error.stack : error.message)
   }
 
   async init () {

test/BaseCommand.test.js

@@ -302,6 +302,26 @@ test('will change error message when aio app outside of the application root (--
   expect(cmd.error).toHaveBeenCalledWith(expect.stringContaining(errorList.join('\n')))
 })
 
+test('will handle errors without stack traces when using --verbose flag', async () => {
+  const cmd = new TheCommand(['--verbose'])
+  cmd.error = jest.fn()
+  const errorWithoutStack = new Error('fake error')
+  delete errorWithoutStack.stack
+  await cmd.catch(errorWithoutStack)
+
+  expect(cmd.error).toHaveBeenCalledWith(expect.stringContaining('fake error'))
+})
+
+test('will handle errors without stack traces when not using --verbose flag', async () => {
+  const cmd = new TheCommand([])
+  cmd.error = jest.fn()
+  const errorWithoutStack = new Error('fake error')
+  delete errorWithoutStack.stack
+  await cmd.catch(errorWithoutStack)
+
+  expect(cmd.error).toHaveBeenCalledWith(expect.stringContaining('fake error'))
+})
+
 test('pjson', async () => {
   const cmd = new TheCommand([])
   cmd.config = { pjson: { name: 'fake', version: '0' } }