From 272b9fc95cb22d498e34f9e1f8a1345550dac673 Mon Sep 17 00:00:00 2001 From: YeonGyu-Kim Date: Thu, 12 Mar 2026 13:11:44 +0900 Subject: [PATCH] test(tmux): avoid private serverUrl assertions Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus --- src/features/tmux-subagent/manager.test.ts | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/src/features/tmux-subagent/manager.test.ts b/src/features/tmux-subagent/manager.test.ts index dbcdf9435..a4974d8d9 100644 --- a/src/features/tmux-subagent/manager.test.ts +++ b/src/features/tmux-subagent/manager.test.ts @@ -128,6 +128,15 @@ function createWindowState(overrides?: Partial): WindowState { } } +function getManagerServerUrl(manager: object): string { + const serverUrl = Reflect.get(manager, 'serverUrl') + if (typeof serverUrl !== 'string') { + throw new TypeError('expected manager.serverUrl to be a string') + } + + return serverUrl +} + describe('TmuxSessionManager', () => { beforeEach(() => { mockQueryWindowState.mockClear() @@ -245,8 +254,8 @@ describe('TmuxSessionManager', () => { const manager = new TmuxSessionManager(ctx, config, mockTmuxDeps) // then - should use dynamic port, not hardcoded 4096 - const serverUrl = (manager as any).serverUrl as string - expect(serverUrl).toContain(`:${dynamicPort}`) + const serverUrl = getManagerServerUrl(manager) + expect(serverUrl).toBe(`http://localhost:${dynamicPort}`) expect(serverUrl).not.toContain('4096') }) @@ -269,7 +278,7 @@ describe('TmuxSessionManager', () => { const manager = new TmuxSessionManager(ctx, config, mockTmuxDeps) // then - should use env var port, not hardcoded 4096 - const serverUrl = (manager as any).serverUrl as string + const serverUrl = getManagerServerUrl(manager) expect(serverUrl).toBe('http://localhost:8080') expect(serverUrl).not.toContain('4096') } finally {