From f7a0e2cbea875d44d36ac4801bfe5467fa17cdac Mon Sep 17 00:00:00 2001 From: Mohamed Boudra Date: Sun, 18 Oct 2026 21:11:14 +0700 Subject: [PATCH] test(server): wait up to ten seconds for the OMP relaunch to register MCP tools (#7304) The crash-relaunch test waited with vi.waitFor's default one second for a fresh runtime and MCP child to come up. On the Windows runner with three workers that took longer once in six runs and failed main. The assertions are unchanged; only the budget for the relaunch is realistic now. --- .../server/agent/providers/omp/mcp-bridge.test.ts | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/packages/server/src/server/agent/providers/omp/mcp-bridge.test.ts b/packages/server/src/server/agent/providers/omp/mcp-bridge.test.ts index 20cab26fd5..db6454edd3 100644 --- a/packages/server/src/server/agent/providers/omp/mcp-bridge.test.ts +++ b/packages/server/src/server/agent/providers/omp/mcp-bridge.test.ts @@ -285,10 +285,15 @@ describe("OMP MCP host tools", () => { expect(omp.registeredHostTools().at(-1)?.[0]?.name).toMatch(/^mcp_local__echo_secret/); omp.processExit("crashed"); await omp.startTurn("again"); - await vi.waitFor(() => { - expect(omp.runtimeSessions()).toHaveLength(2); - expect(omp.registeredHostTools().at(-1)?.[0]?.name).toMatch(/^mcp_local__echo_secret/); - }); + // The relaunch spawns a new runtime and MCP child; on a loaded Windows + // runner that has taken longer than vi.waitFor's default one second. + await vi.waitFor( + () => { + expect(omp.runtimeSessions()).toHaveLength(2); + expect(omp.registeredHostTools().at(-1)?.[0]?.name).toMatch(/^mcp_local__echo_secret/); + }, + { timeout: 10_000 }, + ); } finally { await omp.close(); }