Fix flaky abort test: approve permissions to avoid race with deny-by-default

The test_should_abort_a_session test relies on the shell tool actually
starting execution before the abort is sent. With deny-by-default
permissions (github#509), a race exists: if the permission denial response
arrives before the abort, the tool result becomes 'Permission denied'
instead of the snapshotted 'interrupted' message, causing the replay
proxy to fail matching subsequent requests.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: SteveSandersonMS

python/e2e/test_session.py

@@ -4,7 +4,7 @@
 
 import pytest
 
-from copilot import CopilotClient
+from copilot import CopilotClient, PermissionHandler
 from copilot.types import Tool
 
 from .testharness import E2ETestContext, get_final_assistant_message, get_next_event_of_type
@@ -333,7 +333,9 @@ async def test_should_resume_session_with_custom_provider(self, ctx: E2ETestCont
     async def test_should_abort_a_session(self, ctx: E2ETestContext):
         import asyncio
 
-        session = await ctx.client.create_session()
+        session = await ctx.client.create_session(
+            {"on_permission_request": PermissionHandler.approve_all}
+        )
 
         # Set up event listeners BEFORE sending to avoid race conditions
         wait_for_tool_start = asyncio.create_task(