1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: update test-timers-block-eventloop.js

Browse Source 
When CPU is busy, the above sequential case fails occasionally,
expand the timeout value to fix it.

PR-URL: https://github.com/nodejs/node/pull/16314
Fixes: https://github.com/nodejs/node/issues/16310
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
This commit is contained in:
zhangzifa 2017-10-19 17:06:34 +08:00 committed by Refael Ackermann
parent 5886e204f0
commit 83b8474e64
No known key found for this signature in database
GPG Key ID: CD704BD80FDDDB64
1 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
test/sequential/test-timers-block-eventloop.js
View File

@ -2,21 +2,23 @@
const common = require('../common');
const fs = require('fs');
const platformTimeout = common.platformTimeout;
const t1 = setInterval(() => {
common.busyLoop(12);
}, 10);
common.busyLoop(platformTimeout(12));
}, platformTimeout(10));
const t2 = setInterval(() => {
common.busyLoop(15);
}, 10);
common.busyLoop(platformTimeout(15));
}, platformTimeout(10));
const t3 = setTimeout(common.mustNotCall('eventloop blocked!'), 100);
const t3 =
setTimeout(common.mustNotCall('eventloop blocked!'), platformTimeout(200));
setTimeout(function() {
fs.stat('./nonexistent.txt', (err, stats) => {
fs.stat('/dev/nonexistent', (err, stats) => {
clearInterval(t1);
clearInterval(t2);
clearTimeout(t3);
});
}, 50);
}, platformTimeout(50));