Revert removal of LogTransportMessageParseFailedSensitive call

- Improve test names

Author: halter73

src/ModelContextProtocol/Client/StreamableHttpClientSessionTransport.cs

@@ -222,12 +222,25 @@ private async Task ReceiveUnsolicitedMessagesAsync()
         }
         catch (JsonException ex)
         {
-            LogTransportMessageParseFailed(Name, ex);
+            LogJsonException(ex, data);
         }
 
         return null;
     }
 
+    private void LogJsonException(JsonException ex, string data)
+    {
+        if (_logger.IsEnabled(LogLevel.Trace))
+        {
+            LogTransportMessageParseFailedSensitive(Name, data, ex);
+        }
+        else
+        {
+            LogTransportMessageParseFailed(Name, ex);
+        }
+    }
+
+
     internal static void CopyAdditionalHeaders(HttpRequestHeaders headers, Dictionary<string, string>? additionalHeaders, string? sessionId = null)
     {
         if (sessionId is not null)

tests/ModelContextProtocol.AspNetCore.Tests/MapMcpStreamableHttpTests.cs

@@ -37,7 +37,7 @@ public async Task CanConnect_WithMcpClient_AfterCustomizingRoute(string routePat
     }
 
     [Fact]
-    public async Task StreamableHttp_Mode_Should_Work_With_Root_Endpoint()
+    public async Task StreamableHttpMode_Works_WithRootEndpoint()
     {
         Builder.Services.AddMcpServer(options =>
         {
@@ -63,7 +63,7 @@ public async Task StreamableHttp_Mode_Should_Work_With_Root_Endpoint()
     }
 
     [Fact]
-    public async Task AutoDetect_Mode_Should_Work_With_Root_Endpoint()
+    public async Task AutoDetectMode_Works_WithRootEndpoint()
     {
         Builder.Services.AddMcpServer(options =>
         {
@@ -89,7 +89,7 @@ public async Task AutoDetect_Mode_Should_Work_With_Root_Endpoint()
     }
 
     [Fact]
-    public async Task AutoDetect_Mode_Should_Work_With_Sse_Endpoint()
+    public async Task AutoDetectMode_Works_WithSseEndpoint()
     {
         Assert.SkipWhen(Stateless, "SSE endpoint is disabled in stateless mode.");
 
@@ -117,7 +117,7 @@ public async Task AutoDetect_Mode_Should_Work_With_Sse_Endpoint()
     }
 
     [Fact]
-    public async Task Sse_Mode_Should_Work_With_Sse_Endpoint()
+    public async Task SseMode_Works_WithSseEndpoint()
     {
         Assert.SkipWhen(Stateless, "SSE endpoint is disabled in stateless mode.");
 

tests/ModelContextProtocol.Tests/Transport/SseClientTransportAutoDetectTests.cs

@@ -7,7 +7,7 @@ namespace ModelContextProtocol.Tests.Transport;
 public class SseClientTransportAutoDetectTests(ITestOutputHelper testOutputHelper) : LoggedTest(testOutputHelper)
 {
     [Fact]
-    public async Task AutoDetect_Should_Use_StreamableHttp_When_Server_Supports_It()
+    public async Task AutoDetectMode_UsesStreamableHttp_WhenServerSupportsIt()
     {
         var options = new SseClientTransportOptions
         {
@@ -48,7 +48,7 @@ public async Task AutoDetect_Should_Use_StreamableHttp_When_Server_Supports_It()
     }
 
     [Fact] 
-    public async Task AutoDetect_Should_Fallback_To_Sse_When_StreamableHttp_Fails()
+    public async Task AutoDetectMode_FallsBackToSse_WhenStreamableHttpFails()
     {
         var options = new SseClientTransportOptions
         {