add error case for CI

Signed-off-by: Erick Wendel <[email protected]>

Author: ErickWendel

app/test/api.test.js

@@ -77,7 +77,7 @@ describe('API Workflow', () => {
         const { _id, ...output } = result.at(0)
         ok(_id)
 
-        deepStrictEqual(statusCode, 200)
+        deepStrictEqual(statusCode, 201)
         deepStrictEqual(output, customers.find(customer => customer.name === name))
     })
 

test-runner/bin/index.js

@@ -18,4 +18,5 @@ const formatter = new Formatter();
 const reporter = new Reporter(formatter);
 const testRunner = new TestRunner(folder, formatter, reporter);
 
-testRunner.runTests();
+const hasError = await testRunner.runTests();
+process.exit(hasError ? 1 : 0)

test-runner/bin/test-runner.js

@@ -22,9 +22,10 @@ class TestRunner {
         for (const file of files) {
             finished.push(this.runTestFile(file));
         }
-        await Promise.all(finished)
+        const results = await Promise.all(finished)
 
         this.reporter.printSummary(this.tests, this.formatter.calcElapsed(this.startedAt));
+        return results.some(result => result === 'error')
 
     }
 
@@ -76,8 +77,9 @@ class TestRunner {
             console.error(err)
         })
 
-        return new Promise(resolve => cp.once('exit', resolve)).finally(() => {
+        return new Promise(resolve => cp.once('exit', resolve)).then(() => {
             this.reporter.updateOutput(this.results, this.formatter);
+            return this.tests.failing > 0 ? 'error' : 'success'
         })
     }
 }