Fixed error reporter

usmanyunusov · Feb 15, 2022 · b3f822f · b3f822f
1 parent 8e8c907
commit b3f822f
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 10 deletions.
diff --git a/lib/index.js b/lib/index.js
@@ -40,11 +40,11 @@ export default async function (opts = { ...defaultOptions, ...opts }) {
     } else {
       await runner.run()
     }
-  } catch (error) {
-    for (const error of toArray(error)) {
+  } catch (errors) {
+    for (const error of toArray(errors)) {
       reporter.error(error)
     }
 
-    throw error
+    throw errors
   }
 }
diff --git a/lib/spinner.js b/lib/spinner.js
@@ -59,12 +59,6 @@ class Spinner {
     return this
   }
 
-  collups() {
-    if (this.spins.length) {
-      this.spins = []
-    }
-  }
-
   create(text) {
     let spin = new Spinner(this.spinner, text, this.level + 1)
     this.spins.push(spin)

diff --git a/lib/tasks-runner.js b/lib/tasks-runner.js
@@ -77,7 +77,7 @@ export async function createTasksRunner({
           throw new TaskRunnerError(errors.join('\n\n'))
         }
 
-        spinner.collups()
+        spinner.spins = []
       } catch (err) {
         throw err
       }

diff --git a/test/tasks-runner.test.js b/test/tasks-runner.test.js
@@ -67,4 +67,26 @@ test('should run handle error', async () => {
   }
 })
 
+test('should run handle success', async () => {
+  const { createTasksRunner } = await esmock('../lib/tasks-runner.js', {
+    '../lib/executor.js': {
+      executor: async () => Promise.resolve('Run done'),
+    },
+  })
+
+  let runner = await createTasksRunner({
+    repoPath: 'test',
+    files: ['a.js', '../../b.css'],
+    config: { '*.js': ['prettier --write', 'prettier --write'], '*.css': () => 'prettier --write' },
+    stream: stdout,
+  })
+
+  const spinner = new MultiSpinner({ stream: stdout })
+
+  await runner.run(spinner)
+  is(stdout.out, '\x1B[?25l\x1B[33m\\\x1B[39m *.js\x1B[2m - 1 file\x1B[22m')
+
+  spinner.stop()
+})
+
 test.run()