From 5f51dd7a0030bf89b18bee9ad2075f68694039c4 Mon Sep 17 00:00:00 2001 From: Cyprien P Date: Fri, 29 Jul 2022 10:47:32 +0200 Subject: [PATCH] (core) Fix nbrowser/CopyPaste test on Mac Summary: `nbrowser/CopyPaste` was failing on Mac. Diff fixes that issue. Test Plan: Update `test/nbrowser/CopyPaste` and `core/test/server/customUtil.ts`. Reviewers: jarek Reviewed By: jarek Differential Revision: https://phab.getgrist.com/D3552 --- test/server/customUtil.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/server/customUtil.ts b/test/server/customUtil.ts index ca10f327..789d7422 100644 --- a/test/server/customUtil.ts +++ b/test/server/customUtil.ts @@ -54,8 +54,8 @@ export async function serveSomething(setup: (app: express.Express) => void, port }); async function shutdown() { - await fromCallback(cb => server.close(cb)); for (const conn of connections) { conn.destroy(); } + await fromCallback(cb => server.close(cb)); } port = (server.address() as AddressInfo).port;