From 5529473af98f7a5b8744f70df24534adb0dd350b Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sun, 15 Feb 2026 23:43:17 +0000 Subject: [PATCH] perf(test): fold browser server-context helper into utils suite --- src/browser/browser-utils.test.ts | 39 ++++++++++++++++++ ...r-context.list-known-profile-names.test.ts | 40 ------------------- 2 files changed, 39 insertions(+), 40 deletions(-) delete mode 100644 src/browser/server-context.list-known-profile-names.test.ts diff --git a/src/browser/browser-utils.test.ts b/src/browser/browser-utils.test.ts index 4ff4a2f0610..ab23bca95e7 100644 --- a/src/browser/browser-utils.test.ts +++ b/src/browser/browser-utils.test.ts @@ -1,8 +1,11 @@ import { describe, expect, it, vi } from "vitest"; +import type { BrowserServerState } from "./server-context.js"; import { appendCdpPath, getHeadersWithAuth } from "./cdp.helpers.js"; import { __test } from "./client-fetch.js"; +import { resolveBrowserConfig, resolveProfile } from "./config.js"; import { shouldRejectBrowserMutation } from "./csrf.js"; import { toBoolean } from "./routes/utils.js"; +import { listKnownProfileNames } from "./server-context.js"; import { resolveTargetIdFromTabs } from "./target-id.js"; describe("toBoolean", () => { @@ -176,3 +179,39 @@ describe("fetchBrowserJson loopback auth (bridge auth registry)", () => { expect(getBridgeAuthForPort).toHaveBeenCalledWith(port); }); }); + +describe("browser server-context listKnownProfileNames", () => { + it("includes configured and runtime-only profile names", () => { + const resolved = resolveBrowserConfig({ + defaultProfile: "openclaw", + profiles: { + openclaw: { cdpPort: 18800, color: "#FF4500" }, + }, + }); + const openclaw = resolveProfile(resolved, "openclaw"); + if (!openclaw) { + throw new Error("expected openclaw profile"); + } + + const state: BrowserServerState = { + server: null as unknown as BrowserServerState["server"], + port: 18791, + resolved, + profiles: new Map([ + [ + "stale-removed", + { + profile: { ...openclaw, name: "stale-removed" }, + running: null, + }, + ], + ]), + }; + + expect(listKnownProfileNames(state).toSorted()).toEqual([ + "chrome", + "openclaw", + "stale-removed", + ]); + }); +}); diff --git a/src/browser/server-context.list-known-profile-names.test.ts b/src/browser/server-context.list-known-profile-names.test.ts deleted file mode 100644 index 04c897563e9..00000000000 --- a/src/browser/server-context.list-known-profile-names.test.ts +++ /dev/null @@ -1,40 +0,0 @@ -import { describe, expect, it } from "vitest"; -import type { BrowserServerState } from "./server-context.js"; -import { resolveBrowserConfig, resolveProfile } from "./config.js"; -import { listKnownProfileNames } from "./server-context.js"; - -describe("browser server-context listKnownProfileNames", () => { - it("includes configured and runtime-only profile names", () => { - const resolved = resolveBrowserConfig({ - defaultProfile: "openclaw", - profiles: { - openclaw: { cdpPort: 18800, color: "#FF4500" }, - }, - }); - const openclaw = resolveProfile(resolved, "openclaw"); - if (!openclaw) { - throw new Error("expected openclaw profile"); - } - - const state: BrowserServerState = { - server: null as unknown as BrowserServerState["server"], - port: 18791, - resolved, - profiles: new Map([ - [ - "stale-removed", - { - profile: { ...openclaw, name: "stale-removed" }, - running: null, - }, - ], - ]), - }; - - expect(listKnownProfileNames(state).toSorted()).toEqual([ - "chrome", - "openclaw", - "stale-removed", - ]); - }); -});