From 3f2fb73cfeb5202acafd7e5d8b4d8a43cd12b1ad Mon Sep 17 00:00:00 2001 From: Vincent Koc Date: Tue, 31 Mar 2026 22:13:03 +0900 Subject: [PATCH] perf(slack): avoid module resets in outbound adapter test --- extensions/slack/src/outbound-adapter.test.ts | 5 ++--- test/fixtures/test-timings.channels.json | 5 ++++- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/extensions/slack/src/outbound-adapter.test.ts b/extensions/slack/src/outbound-adapter.test.ts index 50fb2059254..e76ca48a745 100644 --- a/extensions/slack/src/outbound-adapter.test.ts +++ b/extensions/slack/src/outbound-adapter.test.ts @@ -16,6 +16,7 @@ vi.mock("openclaw/plugin-sdk/plugin-runtime", () => ({ })); let slackOutbound: typeof import("./outbound-adapter.js").slackOutbound; +({ slackOutbound } = await import("./outbound-adapter.js")); describe("slackOutbound", () => { const cfg = { @@ -27,13 +28,11 @@ describe("slackOutbound", () => { }, }; - beforeEach(async () => { - vi.resetModules(); + beforeEach(() => { sendMessageSlackMock.mockReset(); hasHooksMock.mockReset(); runMessageSendingMock.mockReset(); hasHooksMock.mockReturnValue(false); - ({ slackOutbound } = await import("./outbound-adapter.js")); }); it("sends payload media first, then finalizes with blocks", async () => { diff --git a/test/fixtures/test-timings.channels.json b/test/fixtures/test-timings.channels.json index 2630668594a..271cd4e59e7 100644 --- a/test/fixtures/test-timings.channels.json +++ b/test/fixtures/test-timings.channels.json @@ -75,6 +75,9 @@ "extensions/slack/src/monitor/slash.test.ts": { "durationMs": 12500 }, + "extensions/slack/src/outbound-adapter.test.ts": { + "durationMs": 4630 + }, "extensions/whatsapp/src/send.test.ts": { "durationMs": 2700 }, @@ -190,7 +193,7 @@ "durationMs": 950 }, "extensions/slack/src/send.upload.test.ts": { - "durationMs": 923 + "durationMs": 10010 }, "extensions/telegram/src/bot-native-commands.registry.test.ts": { "durationMs": 842