From b3cba0d762de46cbc685bf29b41e95376c45c1ca Mon Sep 17 00:00:00 2001
From: morluto <76467478+morluto@users.noreply.github.com>
Date: Sun, 15 Nov 2026 19:31:44 +0800
Subject: [PATCH] test(process): check each capture provider field once instead
 of per side (#11067)

The provider admission matrix ran each of the three provider fields for
both sides through CLI and MCP. Alternating sides keeps every field and
both side labels, and removing the version check or the side label still
fails a remaining row.
---
 .../mcp/processComparisonDiagnosticsMcp.test.ts   | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/tests/boundary/mcp/processComparisonDiagnosticsMcp.test.ts b/tests/boundary/mcp/processComparisonDiagnosticsMcp.test.ts
index aab899adb8..2117289437 100644
--- a/tests/boundary/mcp/processComparisonDiagnosticsMcp.test.ts
+++ b/tests/boundary/mcp/processComparisonDiagnosticsMcp.test.ts
@@ -194,15 +194,12 @@ it.each(
   },
 );
 
-it.each(
-  (["left", "right"] as const).flatMap((side) =>
-    [
-      { id: "unrelated-provider" },
-      { version: "1" },
-      { name: "unrelated provider" },
-    ].map((change) => ({ side, change })),
-  ),
-)(
+// Each provider field is checked once; alternating sides covers both labels.
+it.each([
+  { side: "left", change: { id: "unrelated-provider" } },
+  { side: "right", change: { version: "1" } },
+  { side: "left", change: { name: "unrelated provider" } },
+] as const)(
   "rejects an authenticated $side capture with provider $change",
   async ({ side, change }) => {
     const wrongProvider = captureEvidence(