test: remove timers from test-tls-socket-close

Fixes: nodejs#49902
lpinca · May 16, 2024 · c679e6f · c679e6f
1 parent 6a2d6df
commit c679e6f
Showing 1 changed file with 18 additions and 13 deletions.
diff --git a/test/parallel/test-tls-socket-close.js b/test/parallel/test-tls-socket-close.js
@@ -15,6 +15,10 @@ const cert = fixtures.readKey('agent2-cert.pem');
 let serverTlsSocket;
 const tlsServer = tls.createServer({ cert, key }, (socket) => {
   serverTlsSocket = socket;
+  socket.on('data', (chunk) => {
+    assert.strictEqual(chunk[0], 46);
+    socket.write('.');
+  });
   socket.on('close', dec);
 });
 
@@ -45,24 +49,25 @@ function connectClient(server) {
     rejectUnauthorized: false
   });
 
-  clientTlsSocket.write('foo', 'utf8', common.mustCall(() => {
+  clientTlsSocket.write('.', 'utf8', common.mustCall(() => {
     assert(netSocket);
-    netSocket.setTimeout(common.platformTimeout(10), common.mustCall(() => {
-      assert(serverTlsSocket);
+  }));
+
+  clientTlsSocket.on('data', (chunk) => {
+    assert.strictEqual(chunk[0], 46);
 
-      netSocket.destroy();
-      assert.strictEqual(netSocket.destroyed, true);
+    netSocket.destroy();
+    assert.strictEqual(netSocket.destroyed, true);
 
+    setImmediate(() => {
+      // Close callbacks are executed after `setImmediate()` callbacks.
+      assert.strictEqual(netSocketCloseEmitted, false);
+      assert.strictEqual(serverTlsSocket.destroyed, false);
       setImmediate(() => {
-        // Close callbacks are executed after `setImmediate()` callbacks.
-        assert.strictEqual(netSocketCloseEmitted, false);
-        assert.strictEqual(serverTlsSocket.destroyed, false);
-        setImmediate(() => {
-          assert.strictEqual(netSocketCloseEmitted, true);
-        });
+        assert.strictEqual(netSocketCloseEmitted, true);
       });
-    }));
-  }));
+    });
+  });
 
   clientTlsSocket.on('close', dec);
 }