From c4b57cc652734dc0e04eb0e6ee0cb4cebdf98b9f Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 15 Feb 2026 15:10:15 +0000
Subject: [PATCH 1/3] Initial plan


From e63f5cacafd946e5f13477d363b0e911c98dda58 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 15 Feb 2026 15:19:07 +0000
Subject: [PATCH 2/3] Fix flaky sse-retry conformance test by tolerating
 warning-only results

The conformance test runner exits with code 1 when there are warnings,
even if all checks pass. Timing-sensitive checks like SSE retry can
produce warnings in CI due to network/processing overhead, causing
flaky test failures.

Parse the conformance output to detect warning-only results (0 failures,
>0 warnings) and treat them as passing.

Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>
---
 .../ClientConformanceTests.cs                 | 29 +++++++++++++++++--
 1 file changed, 26 insertions(+), 3 deletions(-)

diff --git a/tests/ModelContextProtocol.AspNetCore.Tests/ClientConformanceTests.cs b/tests/ModelContextProtocol.AspNetCore.Tests/ClientConformanceTests.cs
index 295fc9bb4..7924bf9fa 100644
--- a/tests/ModelContextProtocol.AspNetCore.Tests/ClientConformanceTests.cs
+++ b/tests/ModelContextProtocol.AspNetCore.Tests/ClientConformanceTests.cs
@@ -1,5 +1,6 @@
 using System.Diagnostics;
 using System.Text;
+using System.Text.RegularExpressions;
 using ModelContextProtocol.Tests.Utils;
 
 namespace ModelContextProtocol.ConformanceTests;
@@ -118,10 +119,32 @@ public async Task RunConformanceTest(string scenario)
             );
         }
 
+        var output = outputBuilder.ToString();
+        var error = errorBuilder.ToString();
+        var success = process.ExitCode == 0 || HasOnlyWarnings(output, error);
+
         return (
-            Success: process.ExitCode == 0,
-            Output: outputBuilder.ToString(),
-            Error: errorBuilder.ToString()
+            Success: success,
+            Output: output,
+            Error: error
         );
     }
+
+    /// <summary>
+    /// Checks if the conformance test output indicates that all checks passed with only
+    /// warnings (no actual failures). The conformance runner exits with code 1 for warnings,
+    /// but warnings represent acceptable behavior (e.g., timing tolerances in CI environments).
+    /// </summary>
+    private static bool HasOnlyWarnings(string output, string error)
+    {
+        // The conformance runner outputs a summary line like:
+        //   "Passed: 2/2, 0 failed, 1 warnings"
+        // If there are 0 failures but warnings > 0, the test behavior is acceptable.
+        var combined = output + error;
+        var match = Regex.Match(combined, @"(\d+) failed, (\d+) warnings");
+        return match.Success
+            && match.Groups[1].Value == "0"
+            && int.TryParse(match.Groups[2].Value, out var warnings)
+            && warnings > 0;
+    }
 }

From 1842f77282544eeb24a9f65006abd7d93b314217 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 15 Feb 2026 15:22:49 +0000
Subject: [PATCH 3/3] Use named regex capture groups for readability

Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>
---
 .../ClientConformanceTests.cs                               | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/tests/ModelContextProtocol.AspNetCore.Tests/ClientConformanceTests.cs b/tests/ModelContextProtocol.AspNetCore.Tests/ClientConformanceTests.cs
index 7924bf9fa..f9a5c1ac2 100644
--- a/tests/ModelContextProtocol.AspNetCore.Tests/ClientConformanceTests.cs
+++ b/tests/ModelContextProtocol.AspNetCore.Tests/ClientConformanceTests.cs
@@ -141,10 +141,10 @@ private static bool HasOnlyWarnings(string output, string error)
         //   "Passed: 2/2, 0 failed, 1 warnings"
         // If there are 0 failures but warnings > 0, the test behavior is acceptable.
         var combined = output + error;
-        var match = Regex.Match(combined, @"(\d+) failed, (\d+) warnings");
+        var match = Regex.Match(combined, @"(?<failed>\d+) failed, (?<warnings>\d+) warnings");
         return match.Success
-            && match.Groups[1].Value == "0"
-            && int.TryParse(match.Groups[2].Value, out var warnings)
+            && match.Groups["failed"].Value == "0"
+            && int.TryParse(match.Groups["warnings"].Value, out var warnings)
             && warnings > 0;
     }
 }