Fix Python test names

Author: SteveSandersonMS

python/e2e/test_multi_client.py

@@ -183,7 +183,7 @@ async def configure_multi_test(request, mctx):
 
 
 class TestMultiClientBroadcast:
-    async def test_both_clients_see_tool_events(self, mctx: MultiClientContext):
+    async def test_both_clients_see_tool_request_and_completion_events(self, mctx: MultiClientContext):
         """Both clients see tool request and completion events."""
 
         class SeedParams(BaseModel):
@@ -230,7 +230,7 @@ def magic_number(params: SeedParams, invocation: ToolInvocation) -> str:
 
         await session2.disconnect()
 
-    async def test_one_client_approves_permission(self, mctx: MultiClientContext):
+    async def test_one_client_approves_permission_and_both_see_the_result(self, mctx: MultiClientContext):
         """One client approves a permission request and both see the result."""
         permission_requests = []
 
@@ -276,11 +276,11 @@ async def test_one_client_approves_permission(self, mctx: MultiClientContext):
         assert len(c1_perm_completed) > 0
         assert len(c2_perm_completed) > 0
         for event in c1_perm_completed + c2_perm_completed:
-            assert event.data.result.kind == "approved"
+            assert event.data.result.kind.value == "approved"
 
         await session2.disconnect()
 
-    async def test_one_client_rejects_permission(self, mctx: MultiClientContext):
+    async def test_one_client_rejects_permission_and_both_see_the_result(self, mctx: MultiClientContext):
         """One client rejects a permission request and both see the result."""
         # Client 1 creates a session and denies all permission requests
         session1 = await mctx.client1.create_session(
@@ -327,12 +327,12 @@ async def test_one_client_rejects_permission(self, mctx: MultiClientContext):
         assert len(c1_perm_completed) > 0
         assert len(c2_perm_completed) > 0
         for event in c1_perm_completed + c2_perm_completed:
-            assert event.data.result.kind == "denied-interactively-by-user"
+            assert event.data.result.kind.value == "denied-interactively-by-user"
 
         await session2.disconnect()
 
     @pytest.mark.timeout(90)
-    async def test_two_clients_register_different_tools(self, mctx: MultiClientContext):
+    async def test_two_clients_register_different_tools_and_agent_uses_both(self, mctx: MultiClientContext):
         """Two clients register different tools and agent uses both."""
 
         class CountryCodeParams(BaseModel):
@@ -424,8 +424,8 @@ def ephemeral_tool(params: InputParams, invocation: ToolInvocation) -> str:
         ephemeral_response = await get_final_assistant_message(session1)
         assert "EPHEMERAL_test2" in (ephemeral_response.data.content or "")
 
-        # Force disconnect client 2
-        await mctx.client2.stop()
+        # Force disconnect client 2 without destroying the shared session
+        await mctx.client2.force_stop()
 
         # Give the server time to process the connection close
         await asyncio.sleep(0.5)