Skip to content

Commit 9c21d70

Browse files
committed
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).
1 parent 44a6354 commit 9c21d70

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

test/integration/jasmine-boot.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,11 +57,11 @@ async function runTests(results) {
5757
return;
5858
}
5959
++results.runs;
60-
if (result.failedExpectations.length > 0) {
60+
if (result.failedExpectations.length === 0) {
61+
console.log(`TEST-PASSED | ${result.description}`);
62+
} else {
6163
++results.failures;
6264
console.log(`TEST-UNEXPECTED-FAIL | ${result.description}`);
63-
} else {
64-
console.log(`TEST-PASSED | ${result.description}`);
6565
}
6666
},
6767
specStarted(result) {},

0 commit comments

Comments
 (0)