From f7568760a2df0d0be95487d71c247bb46c9be911 Mon Sep 17 00:00:00 2001 From: Ouyang Yadong Date: Wed, 9 Oct 2019 21:32:05 +0800 Subject: [PATCH] test: add tests for quic `idleTimeout` It would be better to add a test to ensure that the timeout is greater than the triple PTO if we can get the PTO. PR-URL: https://github.com/nodejs/quic/pull/160 Reviewed-By: James M Snell --- test/parallel/test-quic-idle-timeout.js | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/test/parallel/test-quic-idle-timeout.js b/test/parallel/test-quic-idle-timeout.js index 0dab601eed..646fbe0cfb 100644 --- a/test/parallel/test-quic-idle-timeout.js +++ b/test/parallel/test-quic-idle-timeout.js @@ -71,12 +71,10 @@ const idleTimeout = common.platformTimeout(1000); idleTimeout, }); - server.on('session', common.mustCall((serverSession) => { - serverSession.on('close', common.mustCall(() => { - client.close(); - server.close(); - assert(Date.now() - start < idleTimeout + error); - })); + server.on('session', common.mustCall(() => { + client.close(); + server.close(); + assert(Date.now() - start < idleTimeout + error); })); server.on('ready', common.mustCall(() => {