Commit 7dd49d7
committed
test: spin longer for sequential/test-worker-prof
On Windows with V8 14.1, we are just below the current threshold.
PR-URL: #59805
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>1 parent faba50d commit 7dd49d7
1 file changed
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -19,7 +19,7 @@ if (process.argv[2] === 'child') { | |||
| 19 | 19 | const w = new Worker(` | |
| 20 | 20 | const { parentPort, workerData } = require('worker_threads'); | |
| 21 | 21 | ||
| 22 | - const SPIN_MS = 1000; | ||
| 22 | + const SPIN_MS = 1500; | ||
| 23 | 23 | const start = Date.now(); | |
| 24 | 24 | parentPort.on('message', (data) => { | |
| 25 | 25 | if (Date.now() - start < SPIN_MS) { | |
@@ -76,7 +76,7 @@ if (process.argv[2] === 'child') { | |||
| 76 | 76 | console.log('parent ticks', parentTicks.length); | |
| 77 | 77 | // When not tracking Worker threads, only 1 or 2 ticks would | |
| 78 | 78 | // have been recorded. | |
| 79 | - // prof_sampling_interval is by default 1 millisecond. A higher NODE_TEST_SPIN_MS | ||
| 79 | + // prof_sampling_interval is by default 1 millisecond. A higher SPIN_MS | ||
| 80 | 80 | // should result in more ticks, while 15 should be safe on most machines. | |
| 81 | 81 | assert(workerTicks.length > 15, `worker ticks <= 15:\n${workerTicks.join('\n')}`); | |
| 82 | 82 | assert(parentTicks.length > 15, `parent ticks <= 15:\n${parentTicks.join('\n')}`); | |
0 commit comments