From 10cb26a7bd90d565f39a899d3230ca75cf5b0049 Mon Sep 17 00:00:00 2001 From: aramissennyeydd Date: Wed, 8 Apr 2026 21:06:47 -0400 Subject: [PATCH] use a set instead and clear the set Signed-off-by: aramissennyeydd --- packages/backend-openapi-utils/src/testUtils.ts | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/packages/backend-openapi-utils/src/testUtils.ts b/packages/backend-openapi-utils/src/testUtils.ts index afb431c75b..f5244e9e6c 100644 --- a/packages/backend-openapi-utils/src/testUtils.ts +++ b/packages/backend-openapi-utils/src/testUtils.ts @@ -16,9 +16,8 @@ import { Express } from 'express'; import { Server } from 'node:http'; import { Proxy } from './proxy/setup'; -import { randomUUID } from 'node:crypto'; -const proxiesToCleanup: Record = {}; +const proxiesToCleanup: Set = new Set(); /** * !!! THIS CURRENTLY ONLY SUPPORTS SUPERTEST !!! @@ -30,8 +29,7 @@ const proxiesToCleanup: Record = {}; */ export async function wrapServer(app: Express): Promise { const proxy = new Proxy(); - const proxyId = randomUUID(); - proxiesToCleanup[proxyId] = proxy; + proxiesToCleanup.add(proxy); await proxy.setup(); const server = app.listen(proxy.forwardTo.port); @@ -51,10 +49,11 @@ function registerHooks() { registered = true; afterAll(async () => { - const stopPromises = Object.entries(proxiesToCleanup).map(([key, proxy]) => - proxy.stop().finally(() => delete proxiesToCleanup[key]), + const stopPromises = Array.from(proxiesToCleanup).map(proxy => + proxy.stop(), ); await Promise.allSettled(stopPromises); + proxiesToCleanup.clear(); }); }