Use LoggedTest

Author: halter73

tests/ModelContextProtocol.Tests/StdioServerIntegrationTests.cs

@@ -1,14 +1,15 @@
 using ModelContextProtocol.Client;
 using ModelContextProtocol.Protocol.Transport;
+using ModelContextProtocol.Tests.Utils;
 using System.Diagnostics;
 using System.Runtime.InteropServices;
 
 namespace ModelContextProtocol.Tests;
 
-public class StdioServerIntegrationTests
+public class StdioServerIntegrationTests(ITestOutputHelper testOutputHelper) : LoggedTest(testOutputHelper)
 {
     public static bool CanSendSigInt { get; } = RuntimeInformation.IsOSPlatform(OSPlatform.Linux) || RuntimeInformation.IsOSPlatform(OSPlatform.OSX);
-    internal const int SIGINT = 2;
+    private const int SIGINT = 2;
 
     [Fact(Skip = "Platform not supported by this test.", SkipUnless = nameof(CanSendSigInt))]
     public async Task SigInt_DisposesTestServerWithHosting_Gracefully()
@@ -42,10 +43,11 @@ public async Task SigInt_DisposesTestServerWithHosting_Gracefully()
 
         await using var client = await McpClientFactory.CreateAsync(serverConfig,
             createTransportFunc: (_, _) => new TestClientTransport(streamServerTransport),
+            loggerFactory: LoggerFactory,
             cancellationToken: TestContext.Current.CancellationToken);
 
-        // I considered writing a similar test for windows using Ctrl-C, then saw that dotnet watch doesn't even send a Ctrl-C
-        // signal because it's such a pain without support CREATE_NEW_PROCESS_GROUP in System.Diagnostics.Process.
+        // I considered writing a similar test for Windows using Ctrl-C, then saw that dotnet watch doesn't even send a Ctrl-C
+        // signal because it's such a pain without support for CREATE_NEW_PROCESS_GROUP in System.Diagnostics.Process.
         // https://github.com/dotnet/sdk/blob/43b1c12e3362098a23ca1018503eb56516840b6a/src/BuiltInTools/dotnet-watch/Internal/ProcessRunner.cs#L277-L303
         // https://github.com/dotnet/runtime/issues/109432, https://github.com/dotnet/runtime/issues/44944
         Assert.Equal(0, kill(process.Id, SIGINT));
@@ -59,13 +61,11 @@ public async Task SigInt_DisposesTestServerWithHosting_Gracefully()
     }
 
     [DllImport("libc", SetLastError = true)]
-    internal static extern int kill(int pid, int sig);
+    private static extern int kill(int pid, int sig);
 
     private sealed class TestClientTransport(ITransport sessionTransport) : IClientTransport
     {
         public Task<ITransport> ConnectAsync(CancellationToken cancellationToken = default)
-        {
-            return Task.FromResult(sessionTransport);
-        }
+            => Task.FromResult(sessionTransport);
     }
 }