From a66fdc18eb2f16e1c20e7bf69e45d19edf817d5c Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Sat, 3 Apr 2021 08:03:04 -0700 Subject: [PATCH] test: fix flaky test-net-timeout The check for an 800ms window makesw assumptions about a setTimeout() not running late etc. Remove it. Refs: https://github.com/nodejs/node/pull/34289 --- test/pummel/test-net-timeout.js | 4 ---- 1 file changed, 4 deletions(-) diff --git a/test/pummel/test-net-timeout.js b/test/pummel/test-net-timeout.js index 5b9f2a01b3823e..f110254a50499d 100644 --- a/test/pummel/test-net-timeout.js +++ b/test/pummel/test-net-timeout.js @@ -45,7 +45,6 @@ const echo_server = net.createServer((socket) => { }); socket.on('data', (d) => { - console.log(d); socket.write(d); }); @@ -105,7 +104,4 @@ process.on('exit', () => { console.log(`diff = ${diff}`); assert.ok(timeout < diff); - - // Allow for 800 milliseconds more - assert.ok(diff < timeout + 800); });