test: align channel fixtures with open dm policy · opencl...

Recent Commits to openclaw:main

steipete · 2026-04-29 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -408,11 +408,11 @@ describe("BlueBubbles webhook monitor", () => {`
`408`	`408`	`expect(sendMessageBlueBubbles).not.toHaveBeenCalled();`
`409`	`409`	`});`
`410`	`410`
`411`		`-it("allows all DMs when dmPolicy=open", async () => {`
	`411`	`+it("allows wildcard DMs when dmPolicy=open", async () => {`
`412`	`412`	`setupWebhookTarget({`
`413`	`413`	`account: createMockAccount({`
`414`	`414`	`dmPolicy: "open",`
`415`		`-allowFrom: [],`
	`415`	`+allowFrom: ["*"],`
`416`	`416`	`}),`
`417`	`417`	`});`
`418`	`418`
`@@ -483,6 +483,7 @@ describe("BlueBubbles webhook monitor", () => {`
`483`	`483`	`account: createMockAccount({`
`484`	`484`	`groupPolicy: "allowlist",`
`485`	`485`	`dmPolicy: "open",`
	`486`	`+allowFrom: [],`
`486`	`487`	`}),`
`487`	`488`	`});`
`488`	`489`
`@@ -1876,7 +1877,7 @@ describe("BlueBubbles webhook monitor", () => {`
`1876`	`1877`	`expect(mockDispatchReplyWithBufferedBlockDispatcher).not.toHaveBeenCalled();`
`1877`	`1878`	`});`
`1878`	`1879`
`1879`		`-it("does not auto-authorize DM control commands in open mode without allowlists", async () => {`
	`1880`	`+it("drops DM control commands in open mode without allowlists", async () => {`
`1880`	`1881`	`mockHasControlCommand.mockReturnValue(true);`
`1881`	`1882`
`1882`	`1883`	`setupWebhookTarget({`
`@@ -1894,12 +1895,7 @@ describe("BlueBubbles webhook monitor", () => {`
`1894`	`1895`
`1895`	`1896`	`await dispatchWebhookPayload(payload);`
`1896`	`1897`
`1897`		`-expect(mockDispatchReplyWithBufferedBlockDispatcher).toHaveBeenCalled();`
`1898`		`-const latestDispatch =`
`1899`		`-mockDispatchReplyWithBufferedBlockDispatcher.mock.calls[`
`1900`		`-mockDispatchReplyWithBufferedBlockDispatcher.mock.calls.length - 1`
`1901`		`-]?.[0];`
`1902`		`-expect(latestDispatch?.ctx?.CommandAuthorized).toBe(false);`
	`1898`	`+expect(mockDispatchReplyWithBufferedBlockDispatcher).not.toHaveBeenCalled();`
`1903`	`1899`	`});`
`1904`	`1900`	`});`
`1905`	`1901`

Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ export function createMockAccount(`
`34`	`34`	`password: "test-password",`
`35`	`35`	`dmPolicy: "open",`
`36`	`36`	`groupPolicy: "open",`
`37`		`-allowFrom: [],`
	`37`	`+allowFrom: ["*"],`
`38`	`38`	`groupAllowFrom: [],`
`39`	`39`	`...overrides,`
`40`	`40`	`},`

Original file line number	Diff line number	Diff line change
`@@ -1008,7 +1008,7 @@ function makeReactionListenerParams(overrides?: {`
`1008`	`1008`	`groupDmEnabled: overrides?.groupDmEnabled ?? true,`
`1009`	`1009`	`groupDmChannels: overrides?.groupDmChannels ?? [],`
`1010`	`1010`	`dmPolicy: overrides?.dmPolicy ?? "open",`
`1011`		`-allowFrom: overrides?.allowFrom ?? [],`
	`1011`	`+allowFrom: overrides?.allowFrom ?? ["*"],`
`1012`	`1012`	`groupPolicy: overrides?.groupPolicy ?? "open",`
`1013`	`1013`	`allowNameMatching: overrides?.allowNameMatching ?? false,`
`1014`	`1014`	`guildEntries: overrides?.guildEntries,`

Original file line number	Diff line number	Diff line change
`@@ -200,6 +200,7 @@ describe("Discord ACP bind here end-to-end flow", () => {`
`200`	`200`	`client: createDmClient("dm-1"),`
`201`	`201`	`botUserId: "bot-1",`
`202`	`202`	`}),`
	`203`	`+allowFrom: ["*"],`
`203`	`204`	`});`
`204`	`205`
`205`	`206`	`expect(preflight).not.toBeNull();`

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ function createConfig(): OpenClawConfig {`
`35`	`35`	`return {`
`36`	`36`	`channels: {`
`37`	`37`	`discord: {`
`38`		`-dm: { enabled: true, policy: "open" },`
	`38`	`+dm: { enabled: true, policy: "open", allowFrom: ["*"] },`
`39`	`39`	`},`
`40`	`40`	`},`
`41`	`41`	`} as OpenClawConfig;`
`@@ -80,7 +80,7 @@ function createConfiguredAcpCase(params: {`
`80`	`80`	`? {`
`81`	`81`	`channels: {`
`82`	`82`	`discord: {`
`83`		`-dm: { enabled: true, policy: "open" },`
	`83`	`+dm: { enabled: true, policy: "open", allowFrom: ["*"] },`
`84`	`84`	`},`
`85`	`85`	`},`
`86`	`86`	`}`

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ function createConfig(params?: { requireMention?: boolean }): OpenClawConfig {`
`40`	`40`	`},`
`41`	`41`	`channels: {`
`42`	`42`	`discord: {`
`43`		`-dm: { enabled: true, policy: "open" },`
	`43`	`+dm: { enabled: true, policy: "open", allowFrom: ["*"] },`
`44`	`44`	`guilds: {`
`45`	`45`	`guild1: {`
`46`	`46`	`requireMention: true,`

Original file line number	Diff line number	Diff line change
`@@ -41,9 +41,11 @@ const lifecycleConfig = createFeishuLifecycleConfig({`
`41`	`41`	`appSecret: "secret_test",`
`42`	`42`	`channelConfig: {`
`43`	`43`	`dmPolicy: "open",`
	`44`	`+allowFrom: ["ou_user1"],`
`44`	`45`	`},`
`45`	`46`	`accountConfig: {`
`46`	`47`	`dmPolicy: "open",`
	`48`	`+allowFrom: ["ou_user1"],`
`47`	`49`	`},`
`48`	`50`	`});`
`49`	`51`
`@@ -53,6 +55,7 @@ const lifecycleAccount = createResolvedFeishuLifecycleAccount({`
`53`	`55`	`appSecret: "secret_test",`
`54`	`56`	`config: {`
`55`	`57`	`dmPolicy: "open",`
	`58`	`+allowFrom: ["ou_user1"],`
`56`	`59`	`},`
`57`	`60`	`});`
`58`	`61`

Original file line number	Diff line number	Diff line change
`@@ -43,9 +43,11 @@ const lifecycleConfig = createFeishuLifecycleConfig({`
`43`	`43`	`appSecret: "secret_test",`
`44`	`44`	`channelConfig: {`
`45`	`45`	`dmPolicy: "open",`
	`46`	`+allowFrom: ["ou_user1"],`
`46`	`47`	`},`
`47`	`48`	`accountConfig: {`
`48`	`49`	`dmPolicy: "open",`
	`50`	`+allowFrom: ["ou_user1"],`
`49`	`51`	`},`
`50`	`52`	`});`
`51`	`53`
`@@ -55,6 +57,7 @@ const lifecycleAccount = createResolvedFeishuLifecycleAccount({`
`55`	`57`	`appSecret: "secret_test",`
`56`	`58`	`config: {`
`57`	`59`	`dmPolicy: "open",`
	`60`	`+allowFrom: ["ou_user1"],`
`58`	`61`	`},`
`59`	`62`	`});`
`60`	`63`

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,6 @@ function createParams(`
`44`	`44`	`cfg,`
`45`	`45`	`accountId: "default",`
`46`	`46`	`opts: undefined,`
`47`		`-allowFrom: [],`
`48`	`47`	`groupAllowFrom: [],`
`49`	`48`	`groupPolicy: "open",`
`50`	`49`	`dmPolicy: "open",`
`@@ -55,6 +54,7 @@ function createParams(`
`55`	`54`	`selfChatCache: undefined,`
`56`	`55`	`logVerbose: undefined,`
`57`	`56`	`...restOverrides,`
	`57`	`+allowFrom: restOverrides.allowFrom ?? ["*"],`
`58`	`58`	`message,`
`59`	`59`	`messageText,`
`60`	`60`	`bodyText,`

Original file line number	Diff line number	Diff line change
`@@ -168,7 +168,7 @@ describe("signal createSignalEventHandler inbound context", () => {`
`168`	`168`	`);`
`169`	`169`	`});`
`170`	`170`
`171`		`-it("does not auto-authorize DM commands in open mode without allowlists", async () => {`
	`171`	`+it("drops DM commands in open mode without allowlists", async () => {`
`172`	`172`	`const handler = createSignalEventHandler(`
`173`	`173`	`createBaseSignalEventHandlerDeps({`
`174`	`174`	`cfg: {`
`@@ -193,8 +193,8 @@ describe("signal createSignalEventHandler inbound context", () => {`
`193`	`193`	`}),`
`194`	`194`	`);`
`195`	`195`
`196`		`-expect(capture.ctx).toBeTruthy();`
`197`		`-expect(capture.ctx?.CommandAuthorized).toBe(false);`
	`196`	`+expect(capture.ctx).toBeUndefined();`
	`197`	`+expect(dispatchInboundMessageMock).not.toHaveBeenCalled();`
`198`	`198`	`});`
`199`	`199`
`200`	`200`	`it("drops quote-only group context from non-allowlisted quoted senders in allowlist mode", async () => {`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main