From 3e2bfcd84dd8179df7c92ce819ae5bb071aaa147 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Wed, 8 Apr 2026 14:35:53 +0100 Subject: [PATCH] test: keep shared dm policy contract off channel facades --- .../plugins/contracts/dm-policy.contract.test.ts | 16 ++-------------- 1 file changed, 2 insertions(+), 14 deletions(-) diff --git a/src/channels/plugins/contracts/dm-policy.contract.test.ts b/src/channels/plugins/contracts/dm-policy.contract.test.ts index a76aa33c029..a9b6466286f 100644 --- a/src/channels/plugins/contracts/dm-policy.contract.test.ts +++ b/src/channels/plugins/contracts/dm-policy.contract.test.ts @@ -3,8 +3,6 @@ import { isSignalSenderAllowed, type SignalSender, } from "../../../../test/helpers/channels/dm-policy-contract.js"; -import { isAllowedBlueBubblesSender } from "../../../plugin-sdk/bluebubbles-policy.js"; -import { isMattermostSenderAllowed } from "../../../plugin-sdk/mattermost-policy.js"; import { DM_GROUP_ACCESS_REASON, resolveDmGroupAccessWithLists, @@ -26,12 +24,7 @@ const channelSmokeCases: ChannelSmokeCase[] = [ { name: "bluebubbles", storeAllowFrom: ["attacker-user"], - isSenderAllowed: (allowFrom) => - isAllowedBlueBubblesSender({ - allowFrom, - sender: "attacker-user", - chatId: 101, - }), + isSenderAllowed: (allowFrom) => allowFrom.includes("attacker-user"), }, { name: "signal", @@ -41,12 +34,7 @@ const channelSmokeCases: ChannelSmokeCase[] = [ { name: "mattermost", storeAllowFrom: ["user:attacker-user"], - isSenderAllowed: (allowFrom) => - isMattermostSenderAllowed({ - senderId: "attacker-user", - senderName: "Attacker", - allowFrom, - }), + isSenderAllowed: (allowFrom) => allowFrom.includes("user:attacker-user"), }, ];