Skip to content

Commit ba61253

Browse files
bnoordhuisTrott
authored andcommitted
test: scale keepalive timeouts for slow machines
The test was using fixed timeouts and that seems to be causing sporadic test failures on pi1-docker host (which is a very slow machine.) Fixes: #30828 PR-URL: #30834 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Gabriel Schulhof <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
1 parent eac3f0a commit ba61253

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

test/sequential/test-https-server-keep-alive-timeout.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -34,13 +34,13 @@ test(function serverKeepAliveTimeoutWithPipeline(cb) {
3434
common.mustCall((req, res) => {
3535
res.end();
3636
}, 3));
37-
server.setTimeout(500, common.mustCall((socket) => {
37+
server.setTimeout(common.platformTimeout(500), common.mustCall((socket) => {
3838
// End this test and call `run()` for the next test (if any).
3939
socket.destroy();
4040
server.close();
4141
cb();
4242
}));
43-
server.keepAliveTimeout = 50;
43+
server.keepAliveTimeout = common.platformTimeout(50);
4444
server.listen(0, common.mustCall(() => {
4545
const options = {
4646
port: server.address().port,
@@ -57,13 +57,13 @@ test(function serverKeepAliveTimeoutWithPipeline(cb) {
5757

5858
test(function serverNoEndKeepAliveTimeoutWithPipeline(cb) {
5959
const server = https.createServer(serverOptions, common.mustCall(3));
60-
server.setTimeout(500, common.mustCall((socket) => {
60+
server.setTimeout(common.platformTimeout(500), common.mustCall((socket) => {
6161
// End this test and call `run()` for the next test (if any).
6262
socket.destroy();
6363
server.close();
6464
cb();
6565
}));
66-
server.keepAliveTimeout = 50;
66+
server.keepAliveTimeout = common.platformTimeout(50);
6767
server.listen(0, common.mustCall(() => {
6868
const options = {
6969
port: server.address().port,

0 commit comments

Comments
 (0)