Skip to content
Permalink
Browse files

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: #30677
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
Trott authored and MylesBorins committed Nov 26, 2019
1 parent 564e477 commit d3bac601c3eb88ad4e570461b116c8931b868bbc
Showing with 1 addition and 1 deletion.
  1. +1 −1 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();

0 comments on commit d3bac60

Please sign in to comment.
You can’t perform that action at this time.