Refactor concurrency test assertions for clarity and improved logging

Author: Sanjay

tests/client/test_auth.py

@@ -10,7 +10,6 @@
 
 import httpx
 import pytest
-from inline_snapshot import snapshot
 from pydantic import AnyHttpUrl
 
 from mcp.client.auth import OAuthClientProvider
@@ -997,8 +996,17 @@ def test_build_metadata(
     assert str(metadata.registration_endpoint) == str(expected.registration_endpoint)
     assert metadata.scopes_supported == expected.scopes_supported
     assert metadata.grant_types_supported == expected.grant_types_supported
-    assert metadata.token_endpoint_auth_methods_supported == expected.token_endpoint_auth_methods_supported
+    assert (
+        metadata.token_endpoint_auth_methods_supported
+        == expected.token_endpoint_auth_methods_supported
+    )
     assert str(metadata.service_documentation) == str(expected.service_documentation)
     assert str(metadata.revocation_endpoint) == str(expected.revocation_endpoint)
-    assert metadata.revocation_endpoint_auth_methods_supported == expected.revocation_endpoint_auth_methods_supported
-    assert metadata.code_challenge_methods_supported == expected.code_challenge_methods_supported
+    assert (
+        metadata.revocation_endpoint_auth_methods_supported
+        == expected.revocation_endpoint_auth_methods_supported
+    )
+    assert (
+        metadata.code_challenge_methods_supported
+        == expected.code_challenge_methods_supported
+    )

tests/issues/test_188_concurrency.py

@@ -37,8 +37,10 @@ async def slow_resource():
         duration = end_time - start_time
         # 20 tasks (10 tools + 10 resources) should complete in significantly less time
         # than if they were executed serially (which would take 20 * sleep_time)
-        assert duration < 12 * _sleep_time_seconds  # Increased threshold for CI environments
-        print(f"Concurrent execution duration: {duration}, threshold: {12 * _sleep_time_seconds}")
+        # Increased threshold for CI environments
+        assert duration < 12 * _sleep_time_seconds
+        threshold = 12 * _sleep_time_seconds
+        print(f"Concurrent execution duration: {duration}, threshold: {threshold}")
 
 
 def main():