From 9c21d7016b84032942365da51b3a90ed80b0a828 Mon Sep 17 00:00:00 2001 From: Tim van der Meij Date: Sun, 8 Mar 2026 12:40:06 +0100 Subject: [PATCH] Check for passed tests first in `test/integration/jasmine-boot.js` This makes the order of checks consistent with the one in `test/reporter.js` and improves safety because now any status other than passed will be treated as a failure (also if Jasmine adds more statuses later on). --- test/integration/jasmine-boot.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/integration/jasmine-boot.js b/test/integration/jasmine-boot.js index 8379cea4a..fd4bb5c4f 100644 --- a/test/integration/jasmine-boot.js +++ b/test/integration/jasmine-boot.js @@ -57,11 +57,11 @@ async function runTests(results) { return; } ++results.runs; - if (result.failedExpectations.length > 0) { + if (result.failedExpectations.length === 0) { + console.log(`TEST-PASSED | ${result.description}`); + } else { ++results.failures; console.log(`TEST-UNEXPECTED-FAIL | ${result.description}`); - } else { - console.log(`TEST-PASSED | ${result.description}`); } }, specStarted(result) {},