Avoid logging to stale ITestOutputHelper in fixture disposal

Author: halter73

tests/ModelContextProtocol.Tests/SseServerIntegrationTestFixture.cs

@@ -11,9 +11,9 @@ public class SseServerIntegrationTestFixture : IAsyncDisposable
 {
     private readonly Task _serverTask;
     private readonly CancellationTokenSource _stopCts = new();
-    private DelegatingTestOutputHelper _delegatingTestOutputHelper = new();
 
-    private ILoggerFactory _redirectingLoggerFactory;
+    private readonly DelegatingTestOutputHelper _delegatingTestOutputHelper = new();
+    private readonly ILoggerFactory _redirectingLoggerFactory;
 
     public McpClientOptions DefaultOptions { get; }
     public McpServerConfig DefaultConfig { get; }
@@ -50,7 +50,7 @@ public void Initialize(ITestOutputHelper output)
 
     public async ValueTask DisposeAsync()
     {
-        _redirectingLoggerFactory.Dispose();
+        _delegatingTestOutputHelper.CurrentTestOutputHelper = null;
         _stopCts.Cancel();
         try
         {
@@ -59,6 +59,7 @@ public async ValueTask DisposeAsync()
         catch (OperationCanceledException)
         {
         }
+        _redirectingLoggerFactory.Dispose();
         _stopCts.Dispose();
     }