fix(BA-5650-H): align tests with renamed action/data fields

After collapsing service signatures and removing ``owner_access_key``
from action DTOs, propagate the rename to remaining call sites:

- Drop ``owner_access_key`` kwargs from session action constructors in
  service tests (MatchSessionsAction, GetStatusHistoryAction,
  DestroySessionAction, CompleteAction, GetSessionInfoAction,
  DownloadFilesAction, GetDirectAccessInfoAction, RenameSessionAction,
  GetContainerLogsAction, ListFilesAction, InterruptSessionAction).
- Rename ``user_uuid``/``access_key`` → ``owner_id`` (drop access_key)
  on ``SessionData`` constructions in tests.
- ``SessionTransitionInfo`` keeps ``access_key``; cache_invalidation
  reverted to ``info.access_key``.
- Terminator conftest now uses ``main_access_key`` for
  ``TerminatingSessionData``.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: jopemachine

tests/unit/manager/services/session/test_session_service.py

@@ -296,7 +296,6 @@ async def test_success(
 
         action = MatchSessionsAction(
             id_or_name_prefix="test",
-            owner_access_key=sample_access_key,
             user_id=sample_user_id,
         )
         result = await session_service.match_sessions(action)
@@ -319,7 +318,6 @@ async def test_no_matches(
 
         action = MatchSessionsAction(
             id_or_name_prefix="nonexistent",
-            owner_access_key=sample_access_key,
             user_id=sample_user_id,
         )
         result = await session_service.match_sessions(action)
@@ -384,7 +382,6 @@ async def test_multiple_matches(
 
         action = MatchSessionsAction(
             id_or_name_prefix="test",
-            owner_access_key=sample_access_key,
             user_id=sample_user_id,
         )
         result = await session_service.match_sessions(action)
@@ -421,7 +418,6 @@ async def test_success(
 
         action = GetStatusHistoryAction(
             session_name="test-session",
-            owner_access_key=sample_access_key,
         )
         result = await session_service.get_status_history(action)
 
@@ -442,7 +438,6 @@ async def test_session_not_found(
 
         action = GetStatusHistoryAction(
             session_name="nonexistent",
-            owner_access_key=sample_access_key,
         )
 
         with pytest.raises(SessionNotFound):
@@ -463,7 +458,6 @@ async def test_empty_status_history(
 
         action = GetStatusHistoryAction(
             session_name="test-session",
-            owner_access_key=sample_access_key,
         )
         result = await session_service.get_status_history(action)
 
@@ -501,7 +495,6 @@ async def test_success_cancelled(
             session_name="test-session",
             forced=False,
             recursive=False,
-            owner_access_key=sample_access_key,
         )
         result = await session_service.destroy_session(action)
 
@@ -535,7 +528,6 @@ async def test_success_terminated(
             session_name="test-session",
             forced=False,
             recursive=False,
-            owner_access_key=sample_access_key,
         )
         result = await session_service.destroy_session(action)
 
@@ -570,7 +562,6 @@ async def test_force_terminate_directly_terminated(
             session_name="test-session",
             forced=True,
             recursive=False,
-            owner_access_key=sample_access_key,
         )
         result = await session_service.destroy_session(action)
 
@@ -605,7 +596,6 @@ async def test_recursive_destroy(
             session_name="test-session",
             forced=False,
             recursive=True,
-            owner_access_key=sample_access_key,
         )
         result = await session_service.destroy_session(action)
 
@@ -637,7 +627,6 @@ async def test_no_sessions_to_destroy(
             session_name="nonexistent",
             forced=False,
             recursive=False,
-            owner_access_key=sample_access_key,
         )
         result = await session_service.destroy_session(action)
 
@@ -674,7 +663,6 @@ async def test_success(
 
         action = CompleteAction(
             session_name="test-session",
-            owner_access_key=sample_access_key,
             code="print('Hello')",
             options=None,
         )
@@ -700,7 +688,6 @@ async def test_session_not_found(
 
         action = CompleteAction(
             session_name="nonexistent",
-            owner_access_key=sample_access_key,
             code="print('Hello')",
             options=None,
         )
@@ -766,7 +753,6 @@ async def test_success(
 
         action = GetSessionInfoAction(
             session_name="test-session",
-            owner_access_key=sample_access_key,
         )
         result = await session_service.get_session_info(action)
 
@@ -794,7 +780,6 @@ async def test_success_with_no_container_id(
 
         action = GetSessionInfoAction(
             session_name="test-session",
-            owner_access_key=sample_access_key,
         )
         result = await session_service.get_session_info(action)
 
@@ -814,7 +799,6 @@ async def test_session_not_found(
 
         action = GetSessionInfoAction(
             session_name="nonexistent",
-            owner_access_key=sample_access_key,
         )
 
         with pytest.raises(SessionNotFound):
@@ -845,7 +829,6 @@ async def test_success(
         action = DownloadFilesAction(
             user_id=sample_user_id,
             session_name="test-session",
-            owner_access_key=sample_access_key,
             files=["test_file.txt"],
         )
         result = await session_service.download_files(action)
@@ -872,7 +855,6 @@ async def test_session_not_found(
         action = DownloadFilesAction(
             user_id=sample_user_id,
             session_name="nonexistent",
-            owner_access_key=sample_access_key,
             files=["test_file.txt"],
         )
 
@@ -895,7 +877,6 @@ async def test_too_many_files(
         action = DownloadFilesAction(
             user_id=sample_user_id,
             session_name="test-session",
-            owner_access_key=sample_access_key,
             files=["file1.txt", "file2.txt", "file3.txt", "file4.txt", "file5.txt", "file6.txt"],
         )
 
@@ -925,7 +906,6 @@ async def test_success(
 
         action = GetDirectAccessInfoAction(
             session_name="test-session",
-            owner_access_key=sample_access_key,
         )
         result = await session_service.get_direct_access_info(action)
 
@@ -948,7 +928,6 @@ async def test_session_not_found(
 
         action = GetDirectAccessInfoAction(
             session_name="nonexistent",
-            owner_access_key=sample_access_key,
         )
 
         with pytest.raises(SessionNotFound):
@@ -976,7 +955,6 @@ async def test_success(
 
         action = RenameSessionAction(
             session_name="test-session",
-            owner_access_key=sample_access_key,
             new_name="new-session-name",
         )
         result = await session_service.rename_session(action)
@@ -1001,7 +979,6 @@ async def test_not_running_session(
 
         action = RenameSessionAction(
             session_name="test-session",
-            owner_access_key=sample_access_key,
             new_name="new-session-name",
         )
 
@@ -1031,7 +1008,6 @@ async def test_success(
 
         action = RestartSessionAction(
             session_name="test-session",
-            owner_access_key=sample_access_key,
         )
         result = await session_service.restart_session(action)
 
@@ -1055,7 +1031,6 @@ async def test_session_not_found(
 
         action = RestartSessionAction(
             session_name="nonexistent",
-            owner_access_key=sample_access_key,
         )
 
         with pytest.raises(SessionNotFound):
@@ -1084,7 +1059,6 @@ async def test_success(
 
         action = ShutdownServiceAction(
             session_name="test-session",
-            owner_access_key=sample_access_key,
             service_name="test-service",
         )
         result = await session_service.shutdown_service(action)
@@ -1108,7 +1082,6 @@ async def test_session_not_found(
 
         action = ShutdownServiceAction(
             session_name="nonexistent",
-            owner_access_key=sample_access_key,
             service_name="test-service",
         )
 
@@ -1155,7 +1128,6 @@ async def mock_next() -> MagicMock | None:
 
         action = UploadFilesAction(
             session_name="test-session",
-            owner_access_key=sample_access_key,
             reader=mock_reader,
         )
         result = await session_service.upload_files(action)
@@ -1180,7 +1152,6 @@ async def test_session_not_found(
 
         action = UploadFilesAction(
             session_name="nonexistent",
-            owner_access_key=sample_access_key,
             reader=mock_reader,
         )
 
@@ -1226,7 +1197,6 @@ async def test_success(
         action = ExecuteSessionAction(
             session_name="test-session",
             api_version=(4, 0),
-            owner_access_key=sample_access_key,
             params=params,
         )
         result = await session_service.execute_session(action)
@@ -1259,7 +1229,6 @@ async def test_session_not_found(
         action = ExecuteSessionAction(
             session_name="nonexistent",
             api_version=(4, 0),
-            owner_access_key=sample_access_key,
             params=params,
         )
 
@@ -1289,7 +1258,6 @@ async def test_success(
 
         action = InterruptSessionAction(
             session_name="test-session",
-            owner_access_key=sample_access_key,
         )
         result = await session_service.interrupt(action)
 
@@ -1312,7 +1280,6 @@ async def test_session_not_found(
 
         action = InterruptSessionAction(
             session_name="nonexistent",
-            owner_access_key=sample_access_key,
         )
 
         with pytest.raises(SessionNotFound):
@@ -1346,7 +1313,6 @@ async def test_success(
             user_id=sample_user_id,
             path="/home/work",
             session_name="test-session",
-            owner_access_key=sample_access_key,
         )
         result = await session_service.list_files(action)
 
@@ -1372,7 +1338,6 @@ async def test_session_not_found(
             user_id=sample_user_id,
             path="/home/work",
             session_name="nonexistent",
-            owner_access_key=sample_access_key,
         )
 
         with pytest.raises(SessionNotFound):
@@ -1405,7 +1370,6 @@ async def test_success(
 
         action = GetContainerLogsAction(
             session_name="test-session",
-            owner_access_key=sample_access_key,
             kernel_id=None,  # Optional - get logs from main kernel
         )
         result = await session_service.get_container_logs(action)
@@ -1430,7 +1394,6 @@ async def test_session_not_found(
 
         action = GetContainerLogsAction(
             session_name="nonexistent",
-            owner_access_key=sample_access_key,
             kernel_id=None,
         )