test: guard feishu mock helpers · openclaw/openclaw@2caa0d6

Recent Commits to openclaw:main

steipete · 2026-05-12 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -241,8 +241,10 @@ function mockCallArg<T>(`
`241`	`241`	`_type?: (value: unknown) => value is T,`
`242`	`242`	`): T {`
`243`	`243`	`const call = mock.mock.calls[callIndex];`
`244`		`-expect(call).toBeDefined();`
`245`		`-return call?.[argIndex] as T;`
	`244`	`+if (!call) {`
	`245`	+throw new Error(`Expected mock call at index ${callIndex}`);
	`246`	`+}`
	`247`	`+return call[argIndex] as T;`
`246`	`248`	`}`
`247`	`249`
`248`	`250`	`type FeishuRoutePeer = { id: string; kind: "direct" \| "group" };`

Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,9 @@ describe("createFeishuCommentReplyDispatcher", () => {`
`149`	`149`
`150`	`150`	`expect(status).toBe("done");`
`151`	`151`	`const client = createFeishuClientMock.mock.results[0]?.value;`
`152`		`-expect(client).toBeDefined();`
	`152`	`+if (!client) {`
	`153`	`+throw new Error("Expected Feishu client");`
	`154`	`+}`
`153`	`155`	`expect(deliverCommentThreadTextMock).toHaveBeenCalledWith(client, {`
`154`	`156`	`file_token: "doc_token_1",`
`155`	`157`	`file_type: "docx",`

Original file line number	Diff line number	Diff line change
`@@ -72,8 +72,10 @@ function requireRecord(value: unknown, label: string): Record<string, unknown> {`
`72`	`72`	`function callArg(mock: unknown, callIndex: number, argIndex: number, label: string) {`
`73`	`73`	`const calls = (mock as { mock?: { calls?: Array<Array<unknown>> } }).mock?.calls ?? [];`
`74`	`74`	`const call = calls.at(callIndex);`
`75`		`-expect(call, label).toBeDefined();`
`76`		`-return call?.[argIndex];`
	`75`	`+if (!call) {`
	`76`	+throw new Error(`Expected ${label}`);
	`77`	`+}`
	`78`	`+return call[argIndex];`
`77`	`79`	`}`
`78`	`80`
`79`	`81`	`function expectLoadWebMediaCall(fileName: string, localRoots: unknown[] \| undefined) {`

Original file line number	Diff line number	Diff line change
`@@ -52,8 +52,10 @@ function mockCallArg<T>(`
`52`	`52`	`_type?: (value: unknown) => value is T,`
`53`	`53`	`): T {`
`54`	`54`	`const call = mock.mock.calls[callIndex];`
`55`		`-expect(call).toBeDefined();`
`56`		`-return call?.[argIndex] as T;`
	`55`	`+if (!call) {`
	`56`	+throw new Error(`Expected mock call at index ${callIndex}`);
	`57`	`+}`
	`58`	`+return call[argIndex] as T;`
`57`	`59`	`}`
`58`	`60`
`59`	`61`	`type FeishuDriveRequest = {`
`@@ -633,7 +635,9 @@ describe("registerFeishuDriveTools", () => {`
`633`	`635`	`client?: unknown;`
`634`	`636`	`deliveryContext?: { channel?: string; threadId?: string; to?: string };`
`635`	`637`	`}>(cleanupAmbientCommentTypingReactionMock, 0, 0);`
`636`		`-expect(cleanupRequest.client).toBeDefined();`
	`638`	`+if (!cleanupRequest.client) {`
	`639`	`+throw new Error("Expected cleanup request client");`
	`640`	`+}`
`637`	`641`	`expect(cleanupRequest.deliveryContext).toEqual({`
`638`	`642`	`channel: "feishu",`
`639`	`643`	`to: "comment:docx:doc_1:c1",`
`@@ -707,7 +711,9 @@ describe("registerFeishuDriveTools", () => {`
`707`	`711`	`client?: unknown;`
`708`	`712`	`deliveryContext?: { channel?: string; threadId?: string; to?: string };`
`709`	`713`	`}>(cleanupAmbientCommentTypingReactionMock, 0, 0);`
`710`		`-expect(cleanupRequest.client).toBeDefined();`
	`714`	`+if (!cleanupRequest.client) {`
	`715`	`+throw new Error("Expected cleanup request client");`
	`716`	`+}`
`711`	`717`	`expect(cleanupRequest.deliveryContext).toEqual({`
`712`	`718`	`channel: "feishu",`
`713`	`719`	`to: "comment:docx:doc_1:c1",`

Original file line number	Diff line number	Diff line change
`@@ -91,8 +91,10 @@ function mockCallArg<T>(`
`91`	`91`	`_type?: (value: unknown) => value is T,`
`92`	`92`	`): T {`
`93`	`93`	`const call = mock.mock.calls[callIndex];`
`94`		`-expect(call).toBeDefined();`
`95`		`-return call?.[argIndex] as T;`
	`94`	`+if (!call) {`
	`95`	+throw new Error(`Expected mock call at index ${callIndex}`);
	`96`	`+}`
	`97`	`+return call[argIndex] as T;`
`96`	`98`	`}`
`97`	`99`
`98`	`100`	`function callData<T>(`
`@@ -101,7 +103,9 @@ function callData<T>(`
`101`	`103`	`_type?: (value: unknown) => value is T,`
`102`	`104`	`): T {`
`103`	`105`	`const arg = mockCallArg<{ data?: unknown }>(mock, callIndex, 0);`
`104`		`-expect(arg.data).toBeDefined();`
	`106`	`+if (arg.data === undefined) {`
	`107`	+throw new Error(`Expected mock call data at index ${callIndex}`);
	`108`	`+}`
`105`	`109`	`return arg.data as T;`
`106`	`110`	`}`
`107`	`111`

Original file line number	Diff line number	Diff line change
`@@ -801,8 +801,11 @@ describe("feishuOutbound comment-thread routing", () => {`
`801`	`801`
`802`	`802`	`expect(status).toBe("done");`
`803`	`803`	`expect(deliverCommentThreadTextMock).toHaveBeenCalled();`
`804`		`-expect(cleanupReactionCall()?.client).toBeDefined();`
`805`		`-expect(cleanupReactionCall()?.deliveryContext).toEqual({`
	`804`	`+const cleanupCall = cleanupReactionCall();`
	`805`	`+if (!cleanupCall?.client) {`
	`806`	`+throw new Error("Expected cleanup reaction client");`
	`807`	`+}`
	`808`	`+expect(cleanupCall.deliveryContext).toEqual({`
`806`	`809`	`channel: "feishu",`
`807`	`810`	`to: "comment:docx:doxcn123:7623358762119646411",`
`808`	`811`	`threadId: "reply_ambient_1",`

Original file line number	Diff line number	Diff line change
`@@ -245,7 +245,9 @@ describe("createFeishuReplyDispatcher streaming behavior", () => {`
`245`	`245`	`expected: Record<string, unknown>,`
`246`	`246`	`): Record<string, unknown> {`
`247`	`247`	`const start = streamingInstances[instanceIndex]?.start;`
`248`		`-expect(start, "streaming instance must exist").toBeDefined();`
	`248`	`+if (!start) {`
	`249`	+throw new Error(`Expected streaming instance ${instanceIndex}`);
	`250`	`+}`
`249`	`251`	`expect(start.mock.calls[0]?.[0]).toBe("oc_chat");`
`250`	`252`	`expect(start.mock.calls[0]?.[1]).toBe("chat_id");`
`251`	`253`	`return expectRecordFields(start.mock.calls[0]?.[2], "streaming start options", expected);`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main