From b9dde661a97cdfe83964d4974371cd1163100bdb Mon Sep 17 00:00:00 2001 From: Vincent Koc Date: Tue, 29 Sep 2026 10:22:47 +0700 Subject: [PATCH] refactor(feishu): remove unused send test fixtures (#160913) Co-authored-by: Vincent Koc --- extensions/feishu/src/send.concurrent.test.ts | 11 ----------- .../feishu/src/send.reply-fallback.test.ts | 15 --------------- extensions/feishu/src/send.test.ts | 19 ------------------- 3 files changed, 45 deletions(-) diff --git a/extensions/feishu/src/send.concurrent.test.ts b/extensions/feishu/src/send.concurrent.test.ts index b885c39f3e4e..430ce5503d9f 100644 --- a/extensions/feishu/src/send.concurrent.test.ts +++ b/extensions/feishu/src/send.concurrent.test.ts @@ -33,17 +33,6 @@ vi.mock("openclaw/plugin-sdk/text-chunking", async (importOriginal) => { const actual = await importOriginal(); return { ...actual, convertMarkdownTables: mockConvertMarkdownTables }; }); -vi.mock("./runtime.js", () => ({ - getFeishuRuntime: () => ({ - channel: { - text: { - resolveMarkdownTableMode: vi.fn(() => "preserve"), - convertMarkdownTables: vi.fn((text: string) => text), - }, - }, - }), -})); - let sendMessageFeishu: typeof import("./send.js").sendMessageFeishu; const cfg = {} as ClawdbotConfig; diff --git a/extensions/feishu/src/send.reply-fallback.test.ts b/extensions/feishu/src/send.reply-fallback.test.ts index 127d7bf4af0c..db1b08568414 100644 --- a/extensions/feishu/src/send.reply-fallback.test.ts +++ b/extensions/feishu/src/send.reply-fallback.test.ts @@ -3,25 +3,11 @@ import { isChannelPartialDeliveryError } from "openclaw/plugin-sdk/channel-inbou import { afterAll, beforeAll, beforeEach, describe, expect, it, vi } from "vitest"; const resolveFeishuSendTargetMock = vi.hoisted(() => vi.fn()); -const resolveMarkdownTableModeMock = vi.hoisted(() => vi.fn(() => "preserve")); -const convertMarkdownTablesMock = vi.hoisted(() => vi.fn((text: string) => text)); vi.mock("./send-target.js", () => ({ resolveFeishuSendTarget: resolveFeishuSendTargetMock, })); -vi.mock("./runtime.js", () => ({ - setFeishuRuntime: vi.fn(), - getFeishuRuntime: () => ({ - channel: { - text: { - resolveMarkdownTableMode: resolveMarkdownTableModeMock, - convertMarkdownTables: convertMarkdownTablesMock, - }, - }, - }), -})); - let sendCardFeishu: typeof import("./send.js").sendCardFeishu; let sendMessageFeishu: typeof import("./send.js").sendMessageFeishu; @@ -46,7 +32,6 @@ describe("Feishu reply fallback for withdrawn/deleted targets", () => { afterAll(() => { vi.doUnmock("./send-target.js"); - vi.doUnmock("./runtime.js"); vi.resetModules(); }); diff --git a/extensions/feishu/src/send.test.ts b/extensions/feishu/src/send.test.ts index a1a5582c9516..df0365f6eb22 100644 --- a/extensions/feishu/src/send.test.ts +++ b/extensions/feishu/src/send.test.ts @@ -752,25 +752,6 @@ describe("editMessageFeishu", () => { expect(mockClientUpdate).not.toHaveBeenCalled(); }); - it("patches interactive content for card edits", async () => { - mockClientPatch.mockResolvedValueOnce({ code: 0 }); - - const result = await editMessageFeishu({ - cfg: {} as ClawdbotConfig, - messageId: "om_card", - card: { schema: "2.0" }, - }); - - expect(mockClientPatch).toHaveBeenCalledWith({ - path: { message_id: "om_card" }, - data: { - content: JSON.stringify({ schema: "2.0" }), - }, - }); - expect(mockClientUpdate).not.toHaveBeenCalled(); - expect(result).toEqual({ messageId: "om_card", contentType: "interactive" }); - }); - it.each([ { name: "rich-post", body: { text: "updated body" }, request: mockClientUpdate }, { name: "interactive-card", body: { card: { schema: "2.0" } }, request: mockClientPatch },