fix: update tests to expect issuer URLs without trailing slash

Update test_build_metadata to expect stripped trailing slashes from
issuer URLs, matching the serializer behavior. Remove the xfail marker
on the simple-url test case since the serializer now handles Pydantic's
AnyHttpUrl trailing slash addition.

Update test_auth_integration to expect issuer without trailing slash.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: BabyChrist666

tests/client/test_auth.py

@@ -1414,29 +1414,27 @@ async def mock_callback() -> tuple[str, str | None]:
 @pytest.mark.parametrize(
     (
         "issuer_url",
+        "expected_issuer",
         "service_documentation_url",
         "authorization_endpoint",
         "token_endpoint",
         "registration_endpoint",
         "revocation_endpoint",
     ),
     (
-        # Pydantic's AnyUrl incorrectly adds trailing slash to base URLs
-        # This is being fixed in https://github.com/pydantic/pydantic-core/pull/1719 (Pydantic 2.12+)
         pytest.param(
+            "https://auth.example.com",
             "https://auth.example.com",
             "https://auth.example.com/docs",
             "https://auth.example.com/authorize",
             "https://auth.example.com/token",
             "https://auth.example.com/register",
             "https://auth.example.com/revoke",
             id="simple-url",
-            marks=pytest.mark.xfail(
-                reason="Pydantic AnyUrl adds trailing slash to base URLs - fixed in Pydantic 2.12+"
-            ),
         ),
         pytest.param(
             "https://auth.example.com/",
+            "https://auth.example.com",  # trailing slash stripped per RFC 8414
             "https://auth.example.com/docs",
             "https://auth.example.com/authorize",
             "https://auth.example.com/token",
@@ -1445,6 +1443,7 @@ async def mock_callback() -> tuple[str, str | None]:
             id="with-trailing-slash",
         ),
         pytest.param(
+            "https://auth.example.com/v1/mcp",
             "https://auth.example.com/v1/mcp",
             "https://auth.example.com/v1/mcp/docs",
             "https://auth.example.com/v1/mcp/authorize",
@@ -1457,6 +1456,7 @@ async def mock_callback() -> tuple[str, str | None]:
 )
 def test_build_metadata(
     issuer_url: str,
+    expected_issuer: str,
     service_documentation_url: str,
     authorization_endpoint: str,
     token_endpoint: str,
@@ -1472,7 +1472,7 @@ def test_build_metadata(
 
     assert metadata.model_dump(exclude_defaults=True, mode="json") == snapshot(
         {
-            "issuer": Is(issuer_url),
+            "issuer": Is(expected_issuer),
             "authorization_endpoint": Is(authorization_endpoint),
             "token_endpoint": Is(token_endpoint),
             "registration_endpoint": Is(registration_endpoint),

tests/server/mcpserver/auth/test_auth_integration.py

@@ -312,7 +312,7 @@ async def test_metadata_endpoint(self, test_client: httpx.AsyncClient):
         assert response.status_code == 200
 
         metadata = response.json()
-        assert metadata["issuer"] == "https://auth.example.com/"
+        assert metadata["issuer"] == "https://auth.example.com"
         assert metadata["authorization_endpoint"] == "https://auth.example.com/authorize"
         assert metadata["token_endpoint"] == "https://auth.example.com/token"
         assert metadata["registration_endpoint"] == "https://auth.example.com/register"