diff --git a/src/commands/doctor.migrates-slack-discord-dm-policy-aliases.e2e.test.ts b/src/commands/doctor.migrates-slack-discord-dm-policy-aliases.e2e.test.ts new file mode 100644 index 000000000..e72da14d0 --- /dev/null +++ b/src/commands/doctor.migrates-slack-discord-dm-policy-aliases.e2e.test.ts @@ -0,0 +1,48 @@ +import { describe, expect, it, vi } from "vitest"; +import { readConfigFileSnapshot, writeConfigFile } from "./doctor.e2e-harness.js"; + +describe("doctor command", () => { + it("migrates Slack/Discord dm.policy keys to dmPolicy aliases", { timeout: 60_000 }, async () => { + readConfigFileSnapshot.mockResolvedValue({ + path: "/tmp/openclaw.json", + exists: true, + raw: "{}", + parsed: { + channels: { + slack: { dm: { enabled: true, policy: "open", allowFrom: ["*"] } }, + discord: { + dm: { enabled: true, policy: "allowlist", allowFrom: ["123"] }, + }, + }, + }, + valid: true, + config: { + channels: { + slack: { dm: { enabled: true, policy: "open", allowFrom: ["*"] } }, + discord: { dm: { enabled: true, policy: "allowlist", allowFrom: ["123"] } }, + }, + }, + issues: [], + legacyIssues: [], + }); + + const { doctorCommand } = await import("./doctor.js"); + const runtime = { log: vi.fn(), error: vi.fn(), exit: vi.fn() }; + + await doctorCommand(runtime, { nonInteractive: true, repair: true }); + + expect(writeConfigFile).toHaveBeenCalledTimes(1); + const written = writeConfigFile.mock.calls[0]?.[0] as Record; + const channels = (written.channels ?? {}) as Record; + const slack = (channels.slack ?? {}) as Record; + const discord = (channels.discord ?? {}) as Record; + + expect(slack.dmPolicy).toBe("open"); + expect(slack.allowFrom).toEqual(["*"]); + expect(slack.dm).toEqual({ enabled: true }); + + expect(discord.dmPolicy).toBe("allowlist"); + expect(discord.allowFrom).toEqual(["123"]); + expect(discord.dm).toEqual({ enabled: true }); + }); +});