diff --git a/extensions/browser/src/browser/routes/agent.existing-session.test.ts b/extensions/browser/src/browser/routes/agent.existing-session.test.ts index f3da7c65265..8336c9a9769 100644 --- a/extensions/browser/src/browser/routes/agent.existing-session.test.ts +++ b/extensions/browser/src/browser/routes/agent.existing-session.test.ts @@ -107,8 +107,9 @@ function getActPostHandler() { } function requireRecord(value: unknown, label: string): Record { - expect(value, label).toBeTypeOf("object"); - expect(value, label).not.toBeNull(); + if (!value || typeof value !== "object") { + throw new Error(`expected ${label}`); + } return value as Record; } diff --git a/extensions/codex/src/app-server/computer-use.test.ts b/extensions/codex/src/app-server/computer-use.test.ts index 7ed89f7be06..43509480a94 100644 --- a/extensions/codex/src/app-server/computer-use.test.ts +++ b/extensions/codex/src/app-server/computer-use.test.ts @@ -35,8 +35,6 @@ async function expectSetupErrorStatus( } function requireRecord(value: unknown, label: string): Record { - expect(typeof value).toBe("object"); - expect(value).not.toBeNull(); if (typeof value !== "object" || value === null) { throw new Error(`${label} was not an object`); } diff --git a/extensions/codex/src/app-server/dynamic-tools.test.ts b/extensions/codex/src/app-server/dynamic-tools.test.ts index b3b54894645..f7cbe464861 100644 --- a/extensions/codex/src/app-server/dynamic-tools.test.ts +++ b/extensions/codex/src/app-server/dynamic-tools.test.ts @@ -69,8 +69,9 @@ function expectInputText(text: string) { } function requireRecord(value: unknown, label: string): Record { - expect(value, label).toBeTypeOf("object"); - expect(value, label).not.toBeNull(); + if (!value || typeof value !== "object") { + throw new Error(`expected ${label}`); + } return value as Record; } diff --git a/extensions/codex/src/app-server/openclaw-owned-tool-runtime-contract.test.ts b/extensions/codex/src/app-server/openclaw-owned-tool-runtime-contract.test.ts index 4e9c36bb322..d5f1c64162f 100644 --- a/extensions/codex/src/app-server/openclaw-owned-tool-runtime-contract.test.ts +++ b/extensions/codex/src/app-server/openclaw-owned-tool-runtime-contract.test.ts @@ -21,8 +21,6 @@ function createContractTool(overrides: Partial): AnyAgentTool { } function requireRecord(value: unknown, label: string): Record { - expect(typeof value).toBe("object"); - expect(value).not.toBeNull(); if (typeof value !== "object" || value === null) { throw new Error(`${label} was not an object`); } diff --git a/extensions/diffs/src/config.test.ts b/extensions/diffs/src/config.test.ts index 4e5ee47f377..dc7342411f5 100644 --- a/extensions/diffs/src/config.test.ts +++ b/extensions/diffs/src/config.test.ts @@ -50,8 +50,9 @@ function compileManifestConfigSchema() { } function requireRecord(value: unknown, label: string): Record { - expect(value, label).toBeTypeOf("object"); - expect(value, label).not.toBeNull(); + if (!value || typeof value !== "object") { + throw new Error(`expected ${label}`); + } return value as Record; } diff --git a/extensions/feishu/src/bot.test.ts b/extensions/feishu/src/bot.test.ts index 9ece817b30e..e65f55d0260 100644 --- a/extensions/feishu/src/bot.test.ts +++ b/extensions/feishu/src/bot.test.ts @@ -1282,7 +1282,6 @@ describe("handleFeishuMessage command authorization", () => { expect(finalized.OriginatingTo).toBe("chat:oc-group"); expect(finalized.SenderId).toBe("ou-allowed"); const groupSessionKey = resolveGroupSessionKey(finalized as never); - expect(groupSessionKey).not.toBeNull(); if (!groupSessionKey) { throw new Error("Expected group session key"); } diff --git a/extensions/feishu/src/docx.test.ts b/extensions/feishu/src/docx.test.ts index 36a06310374..68caea9da07 100644 --- a/extensions/feishu/src/docx.test.ts +++ b/extensions/feishu/src/docx.test.ts @@ -64,8 +64,9 @@ type ToolResultWithDetails = { const WORKSPACE_ROOT = path.resolve("/workspace"); function requireRecord(value: unknown, label: string): Record { - expect(value, label).toBeTypeOf("object"); - expect(value, label).not.toBeNull(); + if (!value || typeof value !== "object") { + throw new Error(`expected ${label}`); + } return value as Record; } diff --git a/extensions/file-transfer/src/shared/node-invoke-policy.test.ts b/extensions/file-transfer/src/shared/node-invoke-policy.test.ts index 91254108499..d46ed5f74dd 100644 --- a/extensions/file-transfer/src/shared/node-invoke-policy.test.ts +++ b/extensions/file-transfer/src/shared/node-invoke-policy.test.ts @@ -108,8 +108,6 @@ function createCtx(overrides: { } function requireRecord(value: unknown, label: string): Record { - expect(typeof value).toBe("object"); - expect(value).not.toBeNull(); if (typeof value !== "object" || value === null) { throw new Error(`${label} was not an object`); } diff --git a/extensions/file-transfer/src/shared/policy.test.ts b/extensions/file-transfer/src/shared/policy.test.ts index 5aa47a81c1c..d7cba058a27 100644 --- a/extensions/file-transfer/src/shared/policy.test.ts +++ b/extensions/file-transfer/src/shared/policy.test.ts @@ -51,8 +51,6 @@ function withConfig(fileTransfer: Record | undefined) { } function expectResultFields(result: unknown, fields: Record) { - expect(typeof result).toBe("object"); - expect(result).not.toBeNull(); if (typeof result !== "object" || result === null) { throw new Error("policy result was not an object"); } diff --git a/extensions/matrix/src/matrix/client.test.ts b/extensions/matrix/src/matrix/client.test.ts index 19ab9984c53..71c91ee7098 100644 --- a/extensions/matrix/src/matrix/client.test.ts +++ b/extensions/matrix/src/matrix/client.test.ts @@ -47,8 +47,6 @@ class MockMatrixClient { } function requireRecord(value: unknown, label: string): Record { - expect(typeof value).toBe("object"); - expect(value).not.toBeNull(); if (typeof value !== "object" || value === null) { throw new Error(`${label} was not an object`); } diff --git a/extensions/matrix/src/matrix/monitor/events.test.ts b/extensions/matrix/src/matrix/monitor/events.test.ts index c79efeef836..f1e5393c8e8 100644 --- a/extensions/matrix/src/matrix/monitor/events.test.ts +++ b/extensions/matrix/src/matrix/monitor/events.test.ts @@ -34,8 +34,6 @@ function expectBodiesExclude(bodies: string[], text: string) { } function requireRecord(value: unknown, label: string): Record { - expect(typeof value).toBe("object"); - expect(value).not.toBeNull(); if (typeof value !== "object" || value === null) { throw new Error(`${label} was not an object`); } diff --git a/extensions/matrix/src/matrix/sdk.test.ts b/extensions/matrix/src/matrix/sdk.test.ts index cd8a263addb..2bfc1208413 100644 --- a/extensions/matrix/src/matrix/sdk.test.ts +++ b/extensions/matrix/src/matrix/sdk.test.ts @@ -19,8 +19,6 @@ function requestUrl(input: RequestInfo | URL | undefined): string { } function requireRecord(value: unknown, label: string): Record { - expect(typeof value).toBe("object"); - expect(value).not.toBeNull(); if (typeof value !== "object" || value === null) { throw new Error(`${label} was not an object`); } diff --git a/extensions/qa-channel/src/channel.test.ts b/extensions/qa-channel/src/channel.test.ts index dd4084e960f..c84b81741f1 100644 --- a/extensions/qa-channel/src/channel.test.ts +++ b/extensions/qa-channel/src/channel.test.ts @@ -28,7 +28,6 @@ function installQaChannelTestRegistry() { } function expectDispatchedContext(ctx: Record | null): Record { - expect(ctx).not.toBeNull(); if (ctx === null) { throw new Error("Expected dispatched context"); } diff --git a/extensions/slack/src/action-runtime.test.ts b/extensions/slack/src/action-runtime.test.ts index a37986e31bb..e9610ce1710 100644 --- a/extensions/slack/src/action-runtime.test.ts +++ b/extensions/slack/src/action-runtime.test.ts @@ -49,8 +49,6 @@ describe("handleSlackAction", () => { } function requireRecord(value: unknown, label: string): Record { - expect(typeof value).toBe("object"); - expect(value).not.toBeNull(); if (typeof value !== "object" || value === null) { throw new Error(`${label} was not an object`); } diff --git a/extensions/voice-call/src/webhook.test.ts b/extensions/voice-call/src/webhook.test.ts index 7d24323d5fb..c803beac144 100644 --- a/extensions/voice-call/src/webhook.test.ts +++ b/extensions/voice-call/src/webhook.test.ts @@ -220,10 +220,12 @@ describe("VoiceCallWebhookServer realtime transcription provider selection", () expect(mocks.getRealtimeTranscriptionProvider).not.toHaveBeenCalled(); expect(mocks.listRealtimeTranscriptionProviders).toHaveBeenCalledWith(null); const mediaStreamHandler = server.getMediaStreamHandler(); - expect(mediaStreamHandler).not.toBeNull(); - expect(mediaStreamHandler?.handleUpgrade).toBeTypeOf("function"); - expect(mediaStreamHandler?.sendAudio).toBeTypeOf("function"); - expect(mediaStreamHandler?.closeAll).toBeTypeOf("function"); + if (!mediaStreamHandler) { + throw new Error("expected media stream handler"); + } + expect(mediaStreamHandler.handleUpgrade).toBeTypeOf("function"); + expect(mediaStreamHandler.sendAudio).toBeTypeOf("function"); + expect(mediaStreamHandler.closeAll).toBeTypeOf("function"); } finally { await server.stop(); } diff --git a/extensions/zalouser/src/send.test.ts b/extensions/zalouser/src/send.test.ts index dab87e4d904..5ea2071e630 100644 --- a/extensions/zalouser/src/send.test.ts +++ b/extensions/zalouser/src/send.test.ts @@ -60,8 +60,6 @@ function sendFailure(error: string, threadId = "thread") { } function requireRecord(value: unknown, label: string): Record { - expect(typeof value).toBe("object"); - expect(value).not.toBeNull(); if (typeof value !== "object" || value === null) { throw new Error(`${label} was not an object`); }