From 6bc753624fe667411107f3b1881eac624293391c Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sat, 21 Feb 2026 23:28:23 +0000 Subject: [PATCH] test(browser): dedupe generated-token persistence assertions --- src/browser/control-auth.auto-token.test.ts | 20 +++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/src/browser/control-auth.auto-token.test.ts b/src/browser/control-auth.auto-token.test.ts index 3fa03df89d9..b0b589703dd 100644 --- a/src/browser/control-auth.auto-token.test.ts +++ b/src/browser/control-auth.auto-token.test.ts @@ -34,6 +34,18 @@ describe("ensureBrowserControlAuth", () => { expect(mocks.writeConfigFile).not.toHaveBeenCalled(); }; + const expectGeneratedTokenPersisted = (result: { + generatedToken?: string; + auth: { token?: string }; + }) => { + expect(result.generatedToken).toMatch(/^[0-9a-f]{48}$/); + expect(result.auth.token).toBe(result.generatedToken); + expect(mocks.writeConfigFile).toHaveBeenCalledTimes(1); + const persisted = mocks.writeConfigFile.mock.calls[0]?.[0]; + expect(persisted?.gateway?.auth?.mode).toBe("token"); + expect(persisted?.gateway?.auth?.token).toBe(result.generatedToken); + }; + beforeEach(() => { vi.restoreAllMocks(); mocks.loadConfig.mockReset(); @@ -69,13 +81,7 @@ describe("ensureBrowserControlAuth", () => { }); const result = await ensureBrowserControlAuth({ cfg, env: {} as NodeJS.ProcessEnv }); - - expect(result.generatedToken).toMatch(/^[0-9a-f]{48}$/); - expect(result.auth.token).toBe(result.generatedToken); - expect(mocks.writeConfigFile).toHaveBeenCalledTimes(1); - const persisted = mocks.writeConfigFile.mock.calls[0]?.[0]; - expect(persisted?.gateway?.auth?.mode).toBe("token"); - expect(persisted?.gateway?.auth?.token).toBe(result.generatedToken); + expectGeneratedTokenPersisted(result); }); it("skips auto-generation in test env", async () => {