From 0793775a6671fa5768427dbfb494c7184704d8ca Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 11 May 2026 19:41:19 +0100 Subject: [PATCH] test: guard channel provider helpers --- extensions/openai/realtime-transcription-provider.test.ts | 4 +++- extensions/telegram/src/polling-session.test.ts | 8 +++++--- ...eply.web-auto-reply.connection-and-logging.e2e.test.ts | 7 +++++-- 3 files changed, 13 insertions(+), 6 deletions(-) diff --git a/extensions/openai/realtime-transcription-provider.test.ts b/extensions/openai/realtime-transcription-provider.test.ts index ef301f3f933..1a2cb1458e8 100644 --- a/extensions/openai/realtime-transcription-provider.test.ts +++ b/extensions/openai/realtime-transcription-provider.test.ts @@ -86,7 +86,9 @@ async function waitForFakeSocket(): Promise { let socket: FakeWebSocketInstance | undefined; await vi.waitFor(() => { socket = FakeWebSocket.instances[0]; - expect(socket).toBeDefined(); + if (!socket) { + throw new Error("expected session to create a websocket"); + } }); if (!socket) { throw new Error("expected session to create a websocket"); diff --git a/extensions/telegram/src/polling-session.test.ts b/extensions/telegram/src/polling-session.test.ts index 7fd82b4b3bf..bcbadb7fcf8 100644 --- a/extensions/telegram/src/polling-session.test.ts +++ b/extensions/telegram/src/polling-session.test.ts @@ -79,9 +79,11 @@ function statusPatches(source: MockCallSource): Record[] { } function expectPollingConnectedPatch(patch: Record | undefined): void { - expect(patch).toBeDefined(); - expect(patch?.connected).toBe(true); - expect(patch?.mode).toBe("polling"); + if (!patch) { + throw new Error("Expected polling connected patch"); + } + expect(patch.connected).toBe(true); + expect(patch.mode).toBe("polling"); } function makeBot() { diff --git a/extensions/whatsapp/src/auto-reply.web-auto-reply.connection-and-logging.e2e.test.ts b/extensions/whatsapp/src/auto-reply.web-auto-reply.connection-and-logging.e2e.test.ts index ba21d16fe6e..772a8b46b90 100644 --- a/extensions/whatsapp/src/auto-reply.web-auto-reply.connection-and-logging.e2e.test.ts +++ b/extensions/whatsapp/src/auto-reply.web-auto-reply.connection-and-logging.e2e.test.ts @@ -95,8 +95,11 @@ function mockStringMessages(mocked: unknown): string[] { function mockCallArg(mocked: unknown, callIndex: number, argIndex: number): unknown { const calls = (mocked as { mock?: { calls?: unknown[][] } }).mock?.calls; - expect(calls?.[callIndex]).toBeDefined(); - return calls?.[callIndex]?.[argIndex]; + const call = calls?.[callIndex]; + if (!call) { + throw new Error(`Expected mock call at index ${callIndex}`); + } + return call[argIndex]; } async function expectPathMissing(targetPath: string): Promise {