Update ProtectedResourceMetadataTests.cs

Author: localden

tests/ModelContextProtocol.Tests/Protocol/Auth/ProtectedResourceMetadataTests.cs

@@ -1,6 +1,8 @@
-// filepath: c:\Users\ddelimarsky\source\csharp-sdk\tests\ModelContextProtocol.Tests\Protocol\Auth\ProtectedResourceMetadataTests.cs
 using ModelContextProtocol.Protocol.Auth;
+using ModelContextProtocol.Utils.Json;
+using System;
 using System.Text.Json;
+using Xunit;
 
 namespace ModelContextProtocol.Tests.Protocol.Auth;
 
@@ -25,15 +27,15 @@ public void ProtectedResourceMetadata_JsonSerialization_Works()
 
         // Assert
         Assert.NotNull(deserialized);
-        Assert.Equal("http://localhost:7071", deserialized.Resource.ToString());
-        Assert.Equal("https://login.microsoftonline.com/tenant/v2.0", deserialized.AuthorizationServers[0].ToString());
+        Assert.Equal(metadata.Resource, deserialized.Resource);
+        Assert.Equal(metadata.AuthorizationServers[0], deserialized.AuthorizationServers[0]);
         Assert.Equal("header", deserialized.BearerMethodsSupported![0]);
         Assert.Equal(2, deserialized.ScopesSupported!.Length);
         Assert.Contains("mcp.tools", deserialized.ScopesSupported!);
         Assert.Contains("mcp.prompts", deserialized.ScopesSupported!);
-        Assert.Equal("https://example.com/docs", deserialized.ResourceDocumentation!.ToString());
+        Assert.Equal(metadata.ResourceDocumentation, deserialized.ResourceDocumentation);
     }
-
+    
     [Fact]
     public void ProtectedResourceMetadata_JsonDeserialization_WorksWithStringProperties()
     {
@@ -51,40 +53,12 @@ public void ProtectedResourceMetadata_JsonDeserialization_WorksWithStringPropert
 
         // Assert
         Assert.NotNull(deserialized);
-        Assert.Equal("http://localhost:7071", deserialized.Resource.ToString());
-        Assert.Equal("https://login.microsoftonline.com/tenant/v2.0", deserialized.AuthorizationServers[0].ToString());
-        Assert.Equal("header", deserialized.BearerMethodsSupported![0]);
-        Assert.Equal(2, deserialized.ScopesSupported!.Length);
-        Assert.Contains("mcp.tools", deserialized.ScopesSupported!);
-        Assert.Contains("mcp.prompts", deserialized.ScopesSupported!);
-        Assert.Equal("https://example.com/docs", deserialized.ResourceDocumentation!.ToString());
-    }
-
-    [Fact]
-    public void ResourceMetadata_JsonSerialization_Works()
-    {
-        // Arrange
-        var metadata = new ProtectedResourceMetadata
-        {
-            Resource = new Uri("http://localhost:7071"),
-            AuthorizationServers = [new Uri("https://login.microsoftonline.com/tenant/v2.0")],
-            BearerMethodsSupported = ["header"],
-            ScopesSupported = ["mcp.tools", "mcp.prompts"],
-            ResourceDocumentation = new Uri("https://example.com/docs")
-        };
-
-        // Act
-        var json = JsonSerializer.Serialize(metadata);
-        var deserialized = JsonSerializer.Deserialize<ProtectedResourceMetadata>(json);
-
-        // Assert
-        Assert.NotNull(deserialized);
-        Assert.Equal("http://localhost:7071", deserialized.Resource.ToString());
-        Assert.Equal("https://login.microsoftonline.com/tenant/v2.0", deserialized.AuthorizationServers[0].ToString());
+        Assert.Equal(new Uri("http://localhost:7071"), deserialized.Resource);
+        Assert.Equal(new Uri("https://login.microsoftonline.com/tenant/v2.0"), deserialized.AuthorizationServers[0]);
         Assert.Equal("header", deserialized.BearerMethodsSupported![0]);
         Assert.Equal(2, deserialized.ScopesSupported!.Length);
         Assert.Contains("mcp.tools", deserialized.ScopesSupported!);
         Assert.Contains("mcp.prompts", deserialized.ScopesSupported!);
-        Assert.Equal("https://example.com/docs", deserialized.ResourceDocumentation!.ToString());
+        Assert.Equal(new Uri("https://example.com/docs"), deserialized.ResourceDocumentation);
     }
 }