test: speed up telegram suites

src/telegram/bot.create-telegram-bot.accepts-group-messages-mentionpatterns-match-without-botusername.test.ts

@@ -1,8 +1,7 @@
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
 import { escapeRegExp, formatEnvelopeTimestamp } from "../../test/helpers/envelope-timestamp.js";
-
-let createTelegramBot: typeof import("./bot.js").createTelegramBot;
-let resetInboundDedupe: typeof import("../auto-reply/reply/inbound-dedupe.js").resetInboundDedupe;
+import { resetInboundDedupe } from "../auto-reply/reply/inbound-dedupe.js";
+import { createTelegramBot } from "./bot.js";
 
 const { sessionStorePath } = vi.hoisted(() => ({
   sessionStorePath: `/tmp/openclaw-telegram-${Math.random().toString(16).slice(2)}.json`,
@@ -137,11 +136,11 @@ const getOnHandler = (event: string) => {
 
 const ORIGINAL_TZ = process.env.TZ;
 describe("createTelegramBot", () => {
-  beforeEach(async () => {
-    vi.resetModules();
-    ({ resetInboundDedupe } = await import("../auto-reply/reply/inbound-dedupe.js"));
-    ({ createTelegramBot } = await import("./bot.js"));
+  beforeAll(async () => {
     replyModule = await import("../auto-reply/reply.js");
+  });
+
+  beforeEach(() => {
     process.env.TZ = "UTC";
     resetInboundDedupe();
     loadConfig.mockReturnValue({

src/telegram/bot.create-telegram-bot.applies-topic-skill-filters-system-prompts.test.ts

@@ -1,7 +1,6 @@
-import { beforeEach, describe, expect, it, vi } from "vitest";
-
-let createTelegramBot: typeof import("./bot.js").createTelegramBot;
-let resetInboundDedupe: typeof import("../auto-reply/reply/inbound-dedupe.js").resetInboundDedupe;
+import { beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
+import { resetInboundDedupe } from "../auto-reply/reply/inbound-dedupe.js";
+import { createTelegramBot } from "./bot.js";
 
 const { sessionStorePath } = vi.hoisted(() => ({
   sessionStorePath: `/tmp/openclaw-telegram-${Math.random().toString(16).slice(2)}.json`,
@@ -135,11 +134,11 @@ const getOnHandler = (event: string) => {
 };
 
 describe("createTelegramBot", () => {
-  beforeEach(async () => {
-    vi.resetModules();
-    ({ resetInboundDedupe } = await import("../auto-reply/reply/inbound-dedupe.js"));
-    ({ createTelegramBot } = await import("./bot.js"));
+  beforeAll(async () => {
     replyModule = await import("../auto-reply/reply.js");
+  });
+
+  beforeEach(() => {
     resetInboundDedupe();
     loadConfig.mockReturnValue({
       channels: {

src/telegram/bot.create-telegram-bot.blocks-all-group-messages-grouppolicy-is.test.ts

@@ -1,7 +1,6 @@
-import { beforeEach, describe, expect, it, vi } from "vitest";
-
-let createTelegramBot: typeof import("./bot.js").createTelegramBot;
-let resetInboundDedupe: typeof import("../auto-reply/reply/inbound-dedupe.js").resetInboundDedupe;
+import { beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
+import { resetInboundDedupe } from "../auto-reply/reply/inbound-dedupe.js";
+import { createTelegramBot } from "./bot.js";
 
 const { sessionStorePath } = vi.hoisted(() => ({
   sessionStorePath: `/tmp/openclaw-telegram-${Math.random().toString(16).slice(2)}.json`,
@@ -135,11 +134,11 @@ const getOnHandler = (event: string) => {
 };
 
 describe("createTelegramBot", () => {
-  beforeEach(async () => {
-    vi.resetModules();
-    ({ resetInboundDedupe } = await import("../auto-reply/reply/inbound-dedupe.js"));
-    ({ createTelegramBot } = await import("./bot.js"));
+  beforeAll(async () => {
     replyModule = await import("../auto-reply/reply.js");
+  });
+
+  beforeEach(() => {
     resetInboundDedupe();
     loadConfig.mockReturnValue({
       channels: {

src/telegram/bot.create-telegram-bot.dedupes-duplicate-callback-query-updates-by-update.test.ts

@@ -1,7 +1,6 @@
-import { beforeEach, describe, expect, it, vi } from "vitest";
-
-let createTelegramBot: typeof import("./bot.js").createTelegramBot;
-let resetInboundDedupe: typeof import("../auto-reply/reply/inbound-dedupe.js").resetInboundDedupe;
+import { beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
+import { resetInboundDedupe } from "../auto-reply/reply/inbound-dedupe.js";
+import { createTelegramBot } from "./bot.js";
 
 const { sessionStorePath } = vi.hoisted(() => ({
   sessionStorePath: `/tmp/openclaw-telegram-${Math.random().toString(16).slice(2)}.json`,
@@ -135,11 +134,11 @@ const getOnHandler = (event: string) => {
 };
 
 describe("createTelegramBot", () => {
-  beforeEach(async () => {
-    vi.resetModules();
-    ({ resetInboundDedupe } = await import("../auto-reply/reply/inbound-dedupe.js"));
-    ({ createTelegramBot } = await import("./bot.js"));
+  beforeAll(async () => {
     replyModule = await import("../auto-reply/reply.js");
+  });
+
+  beforeEach(() => {
     resetInboundDedupe();
     loadConfig.mockReturnValue({
       channels: {

src/telegram/bot.create-telegram-bot.installs-grammy-throttler.test.ts

@@ -1,11 +1,9 @@
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
 import { escapeRegExp, formatEnvelopeTimestamp } from "../../test/helpers/envelope-timestamp.js";
+import { resetInboundDedupe } from "../auto-reply/reply/inbound-dedupe.js";
+import { createTelegramBot, getTelegramSequentialKey } from "./bot.js";
 import { resolveTelegramFetch } from "./fetch.js";
 
-let createTelegramBot: typeof import("./bot.js").createTelegramBot;
-let getTelegramSequentialKey: typeof import("./bot.js").getTelegramSequentialKey;
-let resetInboundDedupe: typeof import("../auto-reply/reply/inbound-dedupe.js").resetInboundDedupe;
-
 const { sessionStorePath } = vi.hoisted(() => ({
   sessionStorePath: `/tmp/openclaw-telegram-throttler-${Math.random().toString(16).slice(2)}.json`,
 }));
@@ -141,11 +139,11 @@ const getOnHandler = (event: string) => {
 const ORIGINAL_TZ = process.env.TZ;
 
 describe("createTelegramBot", () => {
-  beforeEach(async () => {
-    vi.resetModules();
-    ({ resetInboundDedupe } = await import("../auto-reply/reply/inbound-dedupe.js"));
-    ({ createTelegramBot, getTelegramSequentialKey } = await import("./bot.js"));
+  beforeAll(async () => {
     replyModule = await import("../auto-reply/reply.js");
+  });
+
+  beforeEach(() => {
     process.env.TZ = "UTC";
     resetInboundDedupe();
     loadConfig.mockReturnValue({

src/telegram/bot.create-telegram-bot.matches-tg-prefixed-allowfrom-entries-case-insensitively.test.ts

@@ -1,7 +1,6 @@
-import { beforeEach, describe, expect, it, vi } from "vitest";
-
-let createTelegramBot: typeof import("./bot.js").createTelegramBot;
-let resetInboundDedupe: typeof import("../auto-reply/reply/inbound-dedupe.js").resetInboundDedupe;
+import { beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
+import { resetInboundDedupe } from "../auto-reply/reply/inbound-dedupe.js";
+import { createTelegramBot } from "./bot.js";
 
 const { sessionStorePath } = vi.hoisted(() => ({
   sessionStorePath: `/tmp/openclaw-telegram-${Math.random().toString(16).slice(2)}.json`,
@@ -135,11 +134,11 @@ const getOnHandler = (event: string) => {
 };
 
 describe("createTelegramBot", () => {
-  beforeEach(async () => {
-    vi.resetModules();
-    ({ resetInboundDedupe } = await import("../auto-reply/reply/inbound-dedupe.js"));
-    ({ createTelegramBot } = await import("./bot.js"));
+  beforeAll(async () => {
     replyModule = await import("../auto-reply/reply.js");
+  });
+
+  beforeEach(() => {
     resetInboundDedupe();
     loadConfig.mockReturnValue({
       channels: {

src/telegram/bot.create-telegram-bot.matches-usernames-case-insensitively-grouppolicy-is.test.ts

@@ -1,7 +1,6 @@
-import { beforeEach, describe, expect, it, vi } from "vitest";
-
-let createTelegramBot: typeof import("./bot.js").createTelegramBot;
-let resetInboundDedupe: typeof import("../auto-reply/reply/inbound-dedupe.js").resetInboundDedupe;
+import { beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
+import { resetInboundDedupe } from "../auto-reply/reply/inbound-dedupe.js";
+import { createTelegramBot } from "./bot.js";
 
 const { sessionStorePath } = vi.hoisted(() => ({
   sessionStorePath: `/tmp/openclaw-telegram-${Math.random().toString(16).slice(2)}.json`,
@@ -135,11 +134,11 @@ const getOnHandler = (event: string) => {
 };
 
 describe("createTelegramBot", () => {
-  beforeEach(async () => {
-    vi.resetModules();
-    ({ resetInboundDedupe } = await import("../auto-reply/reply/inbound-dedupe.js"));
-    ({ createTelegramBot } = await import("./bot.js"));
+  beforeAll(async () => {
     replyModule = await import("../auto-reply/reply.js");
+  });
+
+  beforeEach(() => {
     resetInboundDedupe();
     loadConfig.mockReturnValue({
       channels: {

src/telegram/bot.create-telegram-bot.routes-dms-by-telegram-accountid-binding.test.ts

@@ -1,7 +1,6 @@
-import { beforeEach, describe, expect, it, vi } from "vitest";
-
-let createTelegramBot: typeof import("./bot.js").createTelegramBot;
-let resetInboundDedupe: typeof import("../auto-reply/reply/inbound-dedupe.js").resetInboundDedupe;
+import { beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
+import { resetInboundDedupe } from "../auto-reply/reply/inbound-dedupe.js";
+import { createTelegramBot } from "./bot.js";
 
 const { sessionStorePath } = vi.hoisted(() => ({
   sessionStorePath: `/tmp/openclaw-telegram-${Math.random().toString(16).slice(2)}.json`,
@@ -135,11 +134,11 @@ const getOnHandler = (event: string) => {
 };
 
 describe("createTelegramBot", () => {
-  beforeEach(async () => {
-    vi.resetModules();
-    ({ resetInboundDedupe } = await import("../auto-reply/reply/inbound-dedupe.js"));
-    ({ createTelegramBot } = await import("./bot.js"));
+  beforeAll(async () => {
     replyModule = await import("../auto-reply/reply.js");
+  });
+
+  beforeEach(() => {
     resetInboundDedupe();
     loadConfig.mockReturnValue({
       channels: {

src/telegram/bot.create-telegram-bot.sends-replies-without-native-reply-threading.test.ts

@@ -1,10 +1,9 @@
 import fs from "node:fs";
 import os from "node:os";
 import path from "node:path";
-import { beforeEach, describe, expect, it, vi } from "vitest";
-
-let createTelegramBot: typeof import("./bot.js").createTelegramBot;
-let resetInboundDedupe: typeof import("../auto-reply/reply/inbound-dedupe.js").resetInboundDedupe;
+import { beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
+import { resetInboundDedupe } from "../auto-reply/reply/inbound-dedupe.js";
+import { createTelegramBot } from "./bot.js";
 
 const { sessionStorePath } = vi.hoisted(() => ({
   sessionStorePath: `/tmp/openclaw-telegram-reply-threading-${Math.random()
@@ -140,11 +139,11 @@ const getOnHandler = (event: string) => {
 };
 
 describe("createTelegramBot", () => {
-  beforeEach(async () => {
-    vi.resetModules();
-    ({ resetInboundDedupe } = await import("../auto-reply/reply/inbound-dedupe.js"));
-    ({ createTelegramBot } = await import("./bot.js"));
+  beforeAll(async () => {
     replyModule = await import("../auto-reply/reply.js");
+  });
+
+  beforeEach(() => {
     resetInboundDedupe();
     loadConfig.mockReturnValue({
       channels: {

src/telegram/bot.test.ts

@@ -1,18 +1,17 @@
 import fs from "node:fs";
 import os from "node:os";
 import path from "node:path";
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
 import { escapeRegExp, formatEnvelopeTimestamp } from "../../test/helpers/envelope-timestamp.js";
 import { expectInboundContextContract } from "../../test/helpers/inbound-contract.js";
 import {
   listNativeCommandSpecs,
   listNativeCommandSpecsForConfig,
 } from "../auto-reply/commands-registry.js";
+import { resetInboundDedupe } from "../auto-reply/reply/inbound-dedupe.js";
+import { createTelegramBot, getTelegramSequentialKey } from "./bot.js";
 import { resolveTelegramFetch } from "./fetch.js";
 
-let createTelegramBot: typeof import("./bot.js").createTelegramBot;
-let getTelegramSequentialKey: typeof import("./bot.js").getTelegramSequentialKey;
-let resetInboundDedupe: typeof import("../auto-reply/reply/inbound-dedupe.js").resetInboundDedupe;
 let replyModule: typeof import("../auto-reply/reply.js");
 const { listSkillCommandsForAgents } = vi.hoisted(() => ({
   listSkillCommandsForAgents: vi.fn(() => []),
@@ -175,11 +174,11 @@ const getOnHandler = (event: string) => {
 
 const ORIGINAL_TZ = process.env.TZ;
 describe("createTelegramBot", () => {
-  beforeEach(async () => {
-    vi.resetModules();
-    ({ resetInboundDedupe } = await import("../auto-reply/reply/inbound-dedupe.js"));
-    ({ createTelegramBot, getTelegramSequentialKey } = await import("./bot.js"));
+  beforeAll(async () => {
     replyModule = await import("../auto-reply/reply.js");
+  });
+
+  beforeEach(() => {
     process.env.TZ = "UTC";
     resetInboundDedupe();
     loadConfig.mockReturnValue({

src/telegram/bot/helpers.expand-text-links.test.ts

@@ -1,52 +0,0 @@
-import { describe, expect, it } from "vitest";
-import { expandTextLinks } from "./helpers.js";
-
-describe("expandTextLinks", () => {
-  it("returns text unchanged when no entities are provided", () => {
-    expect(expandTextLinks("Hello world")).toBe("Hello world");
-    expect(expandTextLinks("Hello world", null)).toBe("Hello world");
-    expect(expandTextLinks("Hello world", [])).toBe("Hello world");
-  });
-
-  it("returns text unchanged when there are no text_link entities", () => {
-    const entities = [
-      { type: "mention", offset: 0, length: 5 },
-      { type: "bold", offset: 6, length: 5 },
-    ];
-    expect(expandTextLinks("@user hello", entities)).toBe("@user hello");
-  });
-
-  it("expands a single text_link entity", () => {
-    const text = "Check this link for details";
-    const entities = [{ type: "text_link", offset: 11, length: 4, url: "https://example.com" }];
-    expect(expandTextLinks(text, entities)).toBe(
-      "Check this [link](https://example.com) for details",
-    );
-  });
-
-  it("expands multiple text_link entities", () => {
-    const text = "Visit Google or GitHub for more";
-    const entities = [
-      { type: "text_link", offset: 6, length: 6, url: "https://google.com" },
-      { type: "text_link", offset: 16, length: 6, url: "https://github.com" },
-    ];
-    expect(expandTextLinks(text, entities)).toBe(
-      "Visit [Google](https://google.com) or [GitHub](https://github.com) for more",
-    );
-  });
-
-  it("handles adjacent text_link entities", () => {
-    const text = "AB";
-    const entities = [
-      { type: "text_link", offset: 0, length: 1, url: "https://a.example" },
-      { type: "text_link", offset: 1, length: 1, url: "https://b.example" },
-    ];
-    expect(expandTextLinks(text, entities)).toBe("[A](https://a.example)[B](https://b.example)");
-  });
-
-  it("preserves offsets from the original string", () => {
-    const text = " Hello world";
-    const entities = [{ type: "text_link", offset: 1, length: 5, url: "https://example.com" }];
-    expect(expandTextLinks(text, entities)).toBe(" [Hello](https://example.com) world");
-  });
-});

src/telegram/bot/helpers.test.ts

@@ -2,6 +2,7 @@ import { describe, expect, it } from "vitest";
 import {
   buildTelegramThreadParams,
   buildTypingThreadParams,
+  expandTextLinks,
   normalizeForwardedContext,
   resolveTelegramForumThreadId,
 } from "./helpers.js";
@@ -120,3 +121,53 @@ describe("normalizeForwardedContext", () => {
     expect(ctx?.date).toBe(111);
   });
 });
+
+describe("expandTextLinks", () => {
+  it("returns text unchanged when no entities are provided", () => {
+    expect(expandTextLinks("Hello world")).toBe("Hello world");
+    expect(expandTextLinks("Hello world", null)).toBe("Hello world");
+    expect(expandTextLinks("Hello world", [])).toBe("Hello world");
+  });
+
+  it("returns text unchanged when there are no text_link entities", () => {
+    const entities = [
+      { type: "mention", offset: 0, length: 5 },
+      { type: "bold", offset: 6, length: 5 },
+    ];
+    expect(expandTextLinks("@user hello", entities)).toBe("@user hello");
+  });
+
+  it("expands a single text_link entity", () => {
+    const text = "Check this link for details";
+    const entities = [{ type: "text_link", offset: 11, length: 4, url: "https://example.com" }];
+    expect(expandTextLinks(text, entities)).toBe(
+      "Check this [link](https://example.com) for details",
+    );
+  });
+
+  it("expands multiple text_link entities", () => {
+    const text = "Visit Google or GitHub for more";
+    const entities = [
+      { type: "text_link", offset: 6, length: 6, url: "https://google.com" },
+      { type: "text_link", offset: 16, length: 6, url: "https://github.com" },
+    ];
+    expect(expandTextLinks(text, entities)).toBe(
+      "Visit [Google](https://google.com) or [GitHub](https://github.com) for more",
+    );
+  });
+
+  it("handles adjacent text_link entities", () => {
+    const text = "AB";
+    const entities = [
+      { type: "text_link", offset: 0, length: 1, url: "https://a.example" },
+      { type: "text_link", offset: 1, length: 1, url: "https://b.example" },
+    ];
+    expect(expandTextLinks(text, entities)).toBe("[A](https://a.example)[B](https://b.example)");
+  });
+
+  it("preserves offsets from the original string", () => {
+    const text = " Hello world";
+    const entities = [{ type: "text_link", offset: 1, length: 5, url: "https://example.com" }];
+    expect(expandTextLinks(text, entities)).toBe(" [Hello](https://example.com) world");
+  });
+});

src/telegram/send.returns-undefined-empty-input.test.ts

@@ -596,16 +596,12 @@ describe("sendStickerTelegram", () => {
     expect(res.chatId).toBe(chatId);
   });
 
-  it("throws error when fileId is empty", async () => {
-    await expect(sendStickerTelegram("123", "", { token: "tok" })).rejects.toThrow(
-      /file_id is required/i,
-    );
-  });
-
-  it("throws error when fileId is whitespace only", async () => {
-    await expect(sendStickerTelegram("123", "   ", { token: "tok" })).rejects.toThrow(
-      /file_id is required/i,
-    );
+  it("throws error when fileId is blank", async () => {
+    for (const fileId of ["", "   "]) {
+      await expect(sendStickerTelegram("123", fileId, { token: "tok" })).rejects.toThrow(
+        /file_id is required/i,
+      );
+    }
   });
 
   it("includes message_thread_id for forum topic messages", async () => {