From e0bfa439534ab0903dd335fc1e5f587c7d3a5f4c Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Tue, 26 Nov 2019 15:45:59 -0800 Subject: [PATCH] test: remove unnecessary common.platformTimeout() call Applying platformTimeout() to the interval is counterproductive. It should be applied to the request timeout duration only. PR-URL: https://github.com/nodejs/node/pull/30677 Reviewed-By: Gireesh Punathil Reviewed-By: James M Snell --- test/sequential/test-http-server-consumed-timeout.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/sequential/test-http-server-consumed-timeout.js b/test/sequential/test-http-server-consumed-timeout.js index 231ec69ac3951d..eb8de276b8ee53 100644 --- a/test/sequential/test-http-server-consumed-timeout.js +++ b/test/sequential/test-http-server-consumed-timeout.js @@ -41,7 +41,7 @@ function runTest(timeoutDuration) { const interval = setInterval(() => { intervalWasInvoked = true; req.write('a'); - }, common.platformTimeout(25)); + }, 25); setTimeout(() => { clearInterval(interval); req.end();