WEBDIENSTE/ECC
1
0
Fork 0
mirror of https://github.com/affaan-m/ECC.git synced 2026-09-29 13:05:18 +02:00
Code Issues Packages Projects Releases Wiki Activity

test(observer-status): set process.exitCode so the summary always flushes

Browse Source 
Review feedback on #2878. stdout is async when it is a pipe, which is exactly
how tests/run-all.js runs these files, and process.exit() does not wait for
pending writes — so exiting that way can drop the Passed:/Failed: lines the
aggregator totals, defeating the previous commit. The sibling
tests/ci/ito-*-skill.test.js files already use process.exitCode.

Still exits 1 on a broken counter (Passed: 4, Failed: 7) and 0 when clean.
This commit is contained in:
Nguyen Thanh Dat
2026-08-25 18:37:00 +07:00
parent ed6e5a42ee
commit e3ab97915f
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/skills/observer-status-instinct-count.test.js
+4 -1
View File
@@ -204,7 +204,10 @@ function main() {
console.log(`\n Passed: ${passed}`);
console.log(` Failed: ${failed}`);
if (failed > 0) process.exit(1);
// exitCode, not exit(1): stdout is async when it is a pipe, which is how
// tests/run-all.js runs this, and process.exit() does not wait for pending
// writes — it could drop the two lines above, which the aggregator totals.
if (failed > 0) process.exitCode = 1;
}
main();