diff --git a/extensions/msteams/src/graph.test.ts b/extensions/msteams/src/graph.test.ts index 574ce5666bf..613938b304e 100644 --- a/extensions/msteams/src/graph.test.ts +++ b/extensions/msteams/src/graph.test.ts @@ -111,8 +111,10 @@ function fetchCallInit(index: number) { function fetchCallHeader(index: number, name: string) { const headers = fetchCallInit(index)?.headers; - expect(headers).toBeDefined(); - return (headers as Record | undefined)?.[name]; + if (!headers) { + throw new Error(`Expected fetch headers at index ${index}`); + } + return (headers as Record)[name]; } function expectFetchPathContains(index: number, expectedPath: string) { diff --git a/extensions/msteams/src/monitor-handler/message-handler.authz.test.ts b/extensions/msteams/src/monitor-handler/message-handler.authz.test.ts index e83a843c14a..15d2a5dbbeb 100644 --- a/extensions/msteams/src/monitor-handler/message-handler.authz.test.ts +++ b/extensions/msteams/src/monitor-handler/message-handler.authz.test.ts @@ -273,21 +273,28 @@ describe("msteams monitor handler authz", () => { } function recordFromMockCall(value: unknown): Record { - expect(value).toBeDefined(); + if (!value || typeof value !== "object") { + throw new Error("Expected mock call record"); + } return value as Record; } 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]; } function logMeta(logFn: unknown, message: string): Record { const calls = (logFn as { mock?: { calls?: Array<[unknown, unknown?]> } }).mock?.calls ?? []; const call = calls.find(([loggedMessage]) => loggedMessage === message); - expect(call).toBeDefined(); - return recordFromMockCall(call?.[1]); + if (!call) { + throw new Error(`Expected log message: ${message}`); + } + return recordFromMockCall(call[1]); } it("does not treat DM pairing-store entries as group allowlist entries", async () => {