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: nodejs#160 Reviewed-By: James M Snell <[email protected]>
juanarbol · Dec 17, 2019 · f756876 · f756876
1 parent f29029b
commit f756876
Showing 1 changed file with 4 additions and 6 deletions.
diff --git 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(() => {