Check for status failed

aspect-build · Apr 29, 2024 · dd74275 · dd74275
1 parent eda9d77
commit dd74275
Showing 4 changed files with 4 additions and 9 deletions.
diff --git a/cypress/defs.bzl b/cypress/defs.bzl
@@ -78,8 +78,7 @@ def cypress_module_test(name, runner, cypress = "//:node_modules/cypress", **kwa
     cypress.run({
     headless: true,
     }).then(result => {
-    // Cypress only reports a status on failures
-    if (result.status) {
+    if (result.status === 'failed') {
         process.exit(1);
     }
     })

diff --git a/docs/defs.md b/docs/defs.md
diff --git a/docs/rules.md b/docs/rules.md
diff --git a/e2e/workspace/module_test/runner.js b/e2e/workspace/module_test/runner.js
@@ -12,9 +12,7 @@ async function main() {
     return 1;
   }
 
-  // Cypress only reports a status on failures
-  if (result.status) {
-    console.error("Cypress tests failed with status", result.status);
+  if (result.status === "failed") {
     return 2;
   }