Merge pull request #10 from intellihr/fail-with-exit-code-1

#9 Exit the process with code 1 when Jest fails
nordcloud · Dec 2, 2019 · ce252ad · ce252ad
2 parents f98b1e7 + 0465080
commit ce252ad
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 2 deletions.
diff --git a/index.js b/index.js
@@ -84,7 +84,16 @@ class ServerlessJestPlugin {
       'create:test:test': () =>
         BbPromise.bind(this).then(() => createTest(this.serverless, this.options)),
       'invoke:test:test': () =>
-        BbPromise.bind(this).then(() => runTests(this.serverless, this.options, this.config)),
+        BbPromise.bind(this)
+          .then(() => runTests(this.serverless, this.options, this.config))
+          .catch((err) => {
+            if (err.success === false) {
+              // This is a successful run but with failed tests
+              process.exit(1);
+            }
+            // Not sure what this is
+            throw err;
+          }),
       'create:function:create': () =>
         BbPromise.bind(this)
           .then(() => createFunction(this.serverless, this.options))

diff --git a/lib/run-tests.js b/lib/run-tests.js
@@ -30,7 +30,13 @@ const runTests = (serverless, options, conf) =>
     process.env['SERVERLESS_TEST_ROOT'] = serverless.config.servicePath;
 
     return runCLI(config, [serverless.config.servicePath])
-      .then((...success) => resolve(...success))
+      .then((output) => {
+        if (output.results.success) {
+          resolve(output);
+        } else {
+          reject(output.results);
+        }
+      })
       .catch(e => reject(e));
   });