Skip to content

Commit 8d04323

Browse files
committed
test: fix flaky interval test
PR-URL: #18140 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
1 parent 75032c9 commit 8d04323

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

test/sequential/test-timers-setInterval-excludes-callback-duration.js renamed to test/sequential/test-timers-set-interval-excludes-callback-duration.js

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,14 @@ const Timer = process.binding('timer_wrap').Timer;
44
const assert = require('assert');
55

66
let cntr = 0;
7-
let first, second;
7+
let first;
88
const t = setInterval(() => {
9-
common.busyLoop(50);
109
cntr++;
1110
if (cntr === 1) {
1211
first = Timer.now();
12+
common.busyLoop(100);
1313
} else if (cntr === 2) {
14-
second = Timer.now();
15-
assert(Math.abs(second - first - 100) < 10);
14+
assert(Timer.now() - first < 120);
1615
clearInterval(t);
1716
}
1817
}, 100);

0 commit comments

Comments
 (0)