From 82d7617db84d36c5bc0c1ef81aac66d58fbab242 Mon Sep 17 00:00:00 2001
From: Josh Bielick <jbielick@gmail.com>
Date: Wed, 24 Jul 2024 08:51:00 -0400
Subject: [PATCH] update net.Server listen call

Function signature changed in 20.x and now expects an options hash or
different ordered arguments.

https://nodejs.org/docs/latest-v20.x/api/net.html#serverlisten
---
 src/__tests__/_helper.ts | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/src/__tests__/_helper.ts b/src/__tests__/_helper.ts
index 5ac1e3d..d069af4 100644
--- a/src/__tests__/_helper.ts
+++ b/src/__tests__/_helper.ts
@@ -35,6 +35,7 @@ export const mockServer = (): Server => {
     socket.write('+HI {"v":2,"s":"abc","i":3}\r\n');
     server.emit("HI");
   });
+  server.on('error', console.error);
   return server;
 };
 
@@ -45,12 +46,15 @@ type ServerUser = {
 export const mocked = async (fn: ServerUser): Promise<unknown> => {
   const server = mockServer();
   const port = await getPort();
-  server.listen(port, "127.0.0.1");
-  try {
-    return fn(server, port);
-  } finally {
-    await new Promise<void>((resolve) => server.close(() => resolve()));
-  }
+  return new Promise((resolve, reject) => {
+    server.listen({ port, host: "127.0.0.1" }, async () => {
+      try {
+        resolve(await fn(server, port));
+      } finally {
+        server.close(resolve);
+      }
+    });
+  })
 };
 
 mocked.ok = () => ({ socket }: ServerControl) => {