From 007b2fa198b18f1506d157a9a98ac1fab5497e34 Mon Sep 17 00:00:00 2001 From: Anna Henningsen Date: Sun, 17 Feb 2019 20:59:05 +0100 Subject: [PATCH] test: increase run time in test-worker-prof This test has occasionally been failing on Windows since it was added, with 6 instead of 15 ticks being seen. Increasing the duration of the test should make it more reliable. PR-URL: https://github.com/nodejs/node/pull/26172 Reviewed-By: Luigi Pinca Reviewed-By: Daniel Bevenius Reviewed-By: James M Snell Reviewed-By: Ruben Bridgewater --- test/parallel/test-worker-prof.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/parallel/test-worker-prof.js b/test/parallel/test-worker-prof.js index 5b0703f5f9b5ca..d4344a8057f06f 100644 --- a/test/parallel/test-worker-prof.js +++ b/test/parallel/test-worker-prof.js @@ -15,7 +15,7 @@ if (!common.isMainThread) if (process.argv[2] === 'child') { const spin = ` const start = Date.now(); - while (Date.now() - start < 200); + while (Date.now() - start < 1000); `; new Worker(spin, { eval: true }); eval(spin); @@ -32,10 +32,10 @@ for (const logfile of logfiles) { const lines = fs.readFileSync(logfile, 'utf8').split('\n'); const ticks = lines.filter((line) => /^tick,/.test(line)).length; - // Test that at least 20 ticks have been recorded for both parent and child + // Test that at least 15 ticks have been recorded for both parent and child // threads. When not tracking Worker threads, only 1 or 2 ticks would // have been recorded. - // When running locally on x64 Linux, this number is usually at least 150 + // When running locally on x64 Linux, this number is usually at least 700 // for both threads, so 15 seems like a very safe threshold. assert(ticks >= 15, `${ticks} >= 15`); }