From 6923cf348ad30a4eac3ac96511affc50365dbe2b Mon Sep 17 00:00:00 2001 From: Shakker Date: Mon, 11 May 2026 16:24:16 +0100 Subject: [PATCH] test: check doctor allowlist warnings --- src/commands/doctor/shared/empty-allowlist-policy.test.ts | 4 ++-- src/commands/doctor/shared/empty-allowlist-scan.test.ts | 6 ++---- src/commands/doctor/shared/open-policy-allowfrom.test.ts | 4 ++-- 3 files changed, 6 insertions(+), 8 deletions(-) diff --git a/src/commands/doctor/shared/empty-allowlist-policy.test.ts b/src/commands/doctor/shared/empty-allowlist-policy.test.ts index 54911d46a7a..0d9e1d9c9f8 100644 --- a/src/commands/doctor/shared/empty-allowlist-policy.test.ts +++ b/src/commands/doctor/shared/empty-allowlist-policy.test.ts @@ -28,7 +28,7 @@ describe("doctor empty allowlist policy warnings", () => { }); expect(warnings).toEqual([ - expect.stringContaining('channels.signal.dmPolicy is "allowlist" but allowFrom is empty'), + '- channels.signal.dmPolicy is "allowlist" but allowFrom is empty — all DMs will be blocked. Add sender IDs to channels.signal.allowFrom, or run "openclaw doctor --fix" to auto-migrate from pairing store when entries exist.', ]); }); @@ -41,7 +41,7 @@ describe("doctor empty allowlist policy warnings", () => { }); expect(warnings).toEqual([ - expect.stringContaining('channels.imessage.groupPolicy is "allowlist"'), + '- channels.imessage.groupPolicy is "allowlist" but groupAllowFrom is empty — this channel does not fall back to allowFrom, so all group messages will be silently dropped. Add sender IDs to channels.imessage.groupAllowFrom, or set groupPolicy to "open".', ]); }); diff --git a/src/commands/doctor/shared/empty-allowlist-scan.test.ts b/src/commands/doctor/shared/empty-allowlist-scan.test.ts index 07d687f4a2b..1cf4ba49437 100644 --- a/src/commands/doctor/shared/empty-allowlist-scan.test.ts +++ b/src/commands/doctor/shared/empty-allowlist-scan.test.ts @@ -31,10 +31,8 @@ describe("doctor empty allowlist policy scan", () => { ); expect(warnings).toEqual([ - expect.stringContaining('channels.signal.dmPolicy is "allowlist" but allowFrom is empty'), - expect.stringContaining( - 'channels.signal.accounts.work.dmPolicy is "allowlist" but allowFrom is empty', - ), + '- channels.signal.dmPolicy is "allowlist" but allowFrom is empty — all DMs will be blocked. Add sender IDs to channels.signal.allowFrom, or run "openclaw doctor --fix" to auto-migrate from pairing store when entries exist.', + '- channels.signal.accounts.work.dmPolicy is "allowlist" but allowFrom is empty — all DMs will be blocked. Add sender IDs to channels.signal.accounts.work.allowFrom, or run "openclaw doctor --fix" to auto-migrate from pairing store when entries exist.', ]); }); diff --git a/src/commands/doctor/shared/open-policy-allowfrom.test.ts b/src/commands/doctor/shared/open-policy-allowfrom.test.ts index 287a6c1e189..440996175f9 100644 --- a/src/commands/doctor/shared/open-policy-allowfrom.test.ts +++ b/src/commands/doctor/shared/open-policy-allowfrom.test.ts @@ -136,8 +136,8 @@ describe("doctor open-policy allowFrom repair", () => { }); expect(warnings).toEqual([ - expect.stringContaining('channels.signal.allowFrom: set to ["*"]'), - expect.stringContaining('Run "openclaw doctor --fix"'), + '- channels.signal.allowFrom: set to ["*"] (required by dmPolicy="open")', + '- Run "openclaw doctor --fix" to add missing allowFrom wildcards.', ]); }); });