From ba1b4c7b56efb30fcc48645df5b07aad36de85ff Mon Sep 17 00:00:00 2001 From: Antoine du Hamel Date: Wed, 18 Oct 2023 17:18:26 +0200 Subject: [PATCH] test: fix `test-loaders-workers-spawned` flakiness --- test/es-module/test-loaders-workers-spawned.mjs | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/test/es-module/test-loaders-workers-spawned.mjs b/test/es-module/test-loaders-workers-spawned.mjs index a04d8edb041e36..a45bc71e0e261b 100644 --- a/test/es-module/test-loaders-workers-spawned.mjs +++ b/test/es-module/test-loaders-workers-spawned.mjs @@ -48,8 +48,14 @@ describe('Worker threads do not spawn infinitely', { concurrency: true }, () => ]); assert.strictEqual(stderr, ''); + // We are validating that: + // 1. the `--require` flag is run first from the main thread (and prints A). + // 2. the `--require` flag is then run on the loader thread (and print A). + // 3. the `--loader` module is executed (and prints B). + // 4. the `--import` module is evaluated once, on the main thread (and prints C). + // 5. the user code is finally executed (and prints D). // The worker code should always run before the --import, but the console.log might arrive late. - assert.match(stdout, /^A\r?\nA\r?\n(B\r?\nC|C\r?\nB)\r?\nD\r?\n$/); + assert.match(stdout, /^A\r?\n(A\r?\nB\r?\nC|A\r?\nC\r?\nB|C\r?\nA\r?\nB)\r?\nD\r?\n$/); assert.strictEqual(code, 0); assert.strictEqual(signal, null); });