test: fix test-loaders-workers-spawned flakiness

nodejs · Oct 18, 2023 · ba1b4c7 · ba1b4c7
1 parent aa8282e
commit ba1b4c7
Showing 1 changed file with 7 additions and 1 deletion.
diff --git 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);
   });