From 46db8337728da5dbf9084b37a85beaf6716feba7 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Tue, 7 Apr 2026 13:36:41 +0100 Subject: [PATCH] test: speed up channel probe tests --- extensions/feishu/src/probe.test.ts | 12 ++---------- extensions/irc/src/probe.test.ts | 9 ++------- extensions/slack/src/probe.test.ts | 9 ++------- 3 files changed, 6 insertions(+), 24 deletions(-) diff --git a/extensions/feishu/src/probe.test.ts b/extensions/feishu/src/probe.test.ts index 827d15aa134..f27b543e33f 100644 --- a/extensions/feishu/src/probe.test.ts +++ b/extensions/feishu/src/probe.test.ts @@ -1,4 +1,5 @@ -import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from "vitest"; +import { afterEach, beforeEach, describe, expect, it, vi } from "vitest"; +import { clearProbeCache, FEISHU_PROBE_REQUEST_TIMEOUT_MS, probeFeishu } from "./probe.js"; const createFeishuClientMock = vi.hoisted(() => vi.fn()); @@ -6,10 +7,6 @@ vi.mock("./client.js", () => ({ createFeishuClient: createFeishuClientMock, })); -let FEISHU_PROBE_REQUEST_TIMEOUT_MS: typeof import("./probe.js").FEISHU_PROBE_REQUEST_TIMEOUT_MS; -let probeFeishu: typeof import("./probe.js").probeFeishu; -let clearProbeCache: typeof import("./probe.js").clearProbeCache; - const DEFAULT_CREDS = { appId: "cli_123", appSecret: "secret" } as const; // pragma: allowlist secret const DEFAULT_SUCCESS_RESPONSE = { code: 0, @@ -100,11 +97,6 @@ async function readSequentialDefaultProbePair() { } describe("probeFeishu", () => { - beforeAll(async () => { - ({ FEISHU_PROBE_REQUEST_TIMEOUT_MS, probeFeishu, clearProbeCache } = - await import("./probe.js")); - }); - beforeEach(() => { clearProbeCache(); vi.restoreAllMocks(); diff --git a/extensions/irc/src/probe.test.ts b/extensions/irc/src/probe.test.ts index 6ecf7c3f83f..0aacc95aa6b 100644 --- a/extensions/irc/src/probe.test.ts +++ b/extensions/irc/src/probe.test.ts @@ -1,4 +1,5 @@ -import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from "vitest"; +import { afterEach, beforeEach, describe, expect, it, vi } from "vitest"; +import { probeIrc } from "./probe.js"; const resolveIrcAccountMock = vi.hoisted(() => vi.fn()); const buildIrcConnectOptionsMock = vi.hoisted(() => vi.fn()); @@ -16,13 +17,7 @@ vi.mock("./client.js", () => ({ connectIrcClient: connectIrcClientMock, })); -let probeIrc: typeof import("./probe.js").probeIrc; - describe("probeIrc", () => { - beforeAll(async () => { - ({ probeIrc } = await import("./probe.js")); - }); - beforeEach(() => { resolveIrcAccountMock.mockReset(); buildIrcConnectOptionsMock.mockReset(); diff --git a/extensions/slack/src/probe.test.ts b/extensions/slack/src/probe.test.ts index aff577e2849..02c063d22fa 100644 --- a/extensions/slack/src/probe.test.ts +++ b/extensions/slack/src/probe.test.ts @@ -1,4 +1,5 @@ -import { beforeAll, beforeEach, describe, expect, it, vi } from "vitest"; +import { beforeEach, describe, expect, it, vi } from "vitest"; +import { probeSlack } from "./probe.js"; const authTestMock = vi.hoisted(() => vi.fn()); const createSlackWebClientMock = vi.hoisted(() => vi.fn()); @@ -12,13 +13,7 @@ vi.mock("openclaw/plugin-sdk/text-runtime", () => ({ withTimeout: withTimeoutMock, })); -let probeSlack: typeof import("./probe.js").probeSlack; - describe("probeSlack", () => { - beforeAll(async () => { - ({ probeSlack } = await import("./probe.js")); - }); - beforeEach(() => { authTestMock.mockReset(); createSlackWebClientMock.mockReset();