fix(win): close or destroy tcp socket for net.connect & net.createServer

- use socket.destroy for node 16 compat on windows
YOU54F · Jul 18, 2024 · 86a2f8d · 86a2f8d
1 parent 10a18ff
commit 86a2f8d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/test/matt.consumer.integration.spec.ts b/test/matt.consumer.integration.spec.ts
@@ -38,6 +38,7 @@ const sendMattMessageTCP = (
   return new Promise((resolve) => {
     socket.on('data', (data) => {
       resolve(parseMattMessage(data.toString()));
+      socket.destroy();
     });
   });
 };

diff --git a/test/matt.provider.integration.spec.ts b/test/matt.provider.integration.spec.ts
@@ -40,6 +40,7 @@ const startTCPServer = (host: string, port: number) => {
         sock.write(generateMattMessage('message not understood'));
       }
       sock.write('\n');
+      sock.destroy();
     });
   });
-Original file line number
+Diff line change
@@ Expand Up / @@ -40,6 +40,7 @@ const startTCPServer = (host: string, port: number) => { @@
             sock.write(generateMattMessage('message not understood'));
           }
           sock.write('\n');
+          sock.destroy();
         });
       });
@@ Expand Down @@