From f2300f4522e0dd4065319865b0db6c95dda69c52 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Fri, 13 Mar 2026 21:53:41 +0000
Subject: [PATCH] test: dedupe msteams policy route fixtures

---
 extensions/msteams/src/policy.test.ts | 58 +++++++++++----------------
 1 file changed, 23 insertions(+), 35 deletions(-)

diff --git a/extensions/msteams/src/policy.test.ts b/extensions/msteams/src/policy.test.ts
index 091e22d1fd8..ac324f3d785 100644
--- a/extensions/msteams/src/policy.test.ts
+++ b/extensions/msteams/src/policy.test.ts
@@ -6,6 +6,27 @@ import {
   resolveMSTeamsRouteConfig,
 } from "./policy.js";
 
+function resolveNamedTeamRouteConfig(allowNameMatching = false) {
+  const cfg: MSTeamsConfig = {
+    teams: {
+      "My Team": {
+        requireMention: true,
+        channels: {
+          "General Chat": { requireMention: false },
+        },
+      },
+    },
+  };
+
+  return resolveMSTeamsRouteConfig({
+    cfg,
+    teamName: "My Team",
+    channelName: "General Chat",
+    conversationId: "ignored",
+    allowNameMatching,
+  });
+}
+
 describe("msteams policy", () => {
   describe("resolveMSTeamsRouteConfig", () => {
     it("returns team and channel config when present", () => {
@@ -51,23 +72,7 @@ describe("msteams policy", () => {
     });
 
     it("blocks team and channel name matches by default", () => {
-      const cfg: MSTeamsConfig = {
-        teams: {
-          "My Team": {
-            requireMention: true,
-            channels: {
-              "General Chat": { requireMention: false },
-            },
-          },
-        },
-      };
-
-      const res = resolveMSTeamsRouteConfig({
-        cfg,
-        teamName: "My Team",
-        channelName: "General Chat",
-        conversationId: "ignored",
-      });
+      const res = resolveNamedTeamRouteConfig();
 
       expect(res.teamConfig).toBeUndefined();
       expect(res.channelConfig).toBeUndefined();
@@ -75,24 +80,7 @@ describe("msteams policy", () => {
     });
 
     it("matches team and channel by name when dangerous name matching is enabled", () => {
-      const cfg: MSTeamsConfig = {
-        teams: {
-          "My Team": {
-            requireMention: true,
-            channels: {
-              "General Chat": { requireMention: false },
-            },
-          },
-        },
-      };
-
-      const res = resolveMSTeamsRouteConfig({
-        cfg,
-        teamName: "My Team",
-        channelName: "General Chat",
-        conversationId: "ignored",
-        allowNameMatching: true,
-      });
+      const res = resolveNamedTeamRouteConfig(true);
 
       expect(res.teamConfig?.requireMention).toBe(true);
       expect(res.channelConfig?.requireMention).toBe(false);