Private
Public Access
0
0

conductor(checkpoint): Phase 2 complete - 18 test files migrated to send_result()

Migrated 11 call-site files + 7 production-affected mock files to use
send_result() instead of send():

Call-site migrations (11 files):
- test_ai_client_cli.py
- test_ai_cache_tracking.py
- test_ai_client_result.py (deleted test_send_deprecated_emits_warning;
  renamed test_send_extracts_data_from_result to
  test_send_result_does_not_emit_deprecation)
- test_api_events.py
- test_deepseek_provider.py (6 sites in 1 file)
- test_gemini_cli_edge_cases.py
- test_gemini_cli_integration.py
- test_gemini_cli_parity_regression.py
- test_gui2_mcp.py
- test_tier4_interceptor.py
- test_token_usage.py

Mock migrations (7 files; pre-empted Phase 1 regressions):
- test_conductor_tech_lead.py (3 mocks)
- test_orchestration_logic.py (4 mocks including the missed
  test_run_worker_lifecycle_blocked)
- test_orchestrator_pm.py (3 mocks)
- test_orchestrator_pm_history.py (1 mock)
- test_phase6_engine.py (1 mock)
- test_run_worker_lifecycle_abort.py (1 mock)
- test_spawn_interception_v2.py (1 mock)

test_rag_integration.py mock migration deferred to RAG track (OOS1).

Verified: 64/64 tests pass in the 18 migrated files.
This commit is contained in:
2026-06-15 17:46:26 -04:00
parent 4592618372
commit da6e084893
@@ -16,9 +16,9 @@ data_structure_strengthening_20260606 = "planned in this track"
mcp_architecture_refactor_20260606 = "transitively"
[phases]
phase_1 = { status = "completed", checkpointsha = "", name = "Production call site migration" }
phase_2 = { status = "in_progress", checkpointsha = "", name = "Test file migration" }
phase_3 = { status = "pending", checkpointsha = "", name = "Qwen test fix" }
phase_1 = { status = "completed", checkpointsha = "b7fd4e4f", name = "Production call site migration" }
phase_2 = { status = "completed", checkpointsha = "", name = "Test file migration" }
phase_3 = { status = "in_progress", checkpointsha = "", name = "Qwen test fix" }
phase_4 = { status = "pending", checkpointsha = "", name = "Symbol parsing test fix" }
phase_5 = { status = "pending", checkpointsha = "", name = "UI Polish test fixes" }
phase_6 = { status = "pending", checkpointsha = "", name = "Deprecation removal" }
@@ -29,7 +29,7 @@ phase_7 = { status = "pending", checkpointsha = "", name = "Docs + housekeep" }
t1_1 = { status = "completed", commit_sha = "bbb3d597", description = "Migrate src/conductor_tech_lead.py:68 to send_result()" }
t1_2 = { status = "completed", commit_sha = "7ea802ab", description = "Migrate src/orchestrator_pm.py:86 to send_result()" }
t1_3 = { status = "completed", commit_sha = "bdd46299", description = "Migrate src/multi_agent_conductor.py:591 to send_result()" }
t1_4 = { status = "in_progress", commit_sha = "", description = "Phase 1 checkpoint" }
t1_4 = { status = "completed", commit_sha = "b7fd4e4f", description = "Phase 1 checkpoint" }
# Phase 2
t2_1 = { status = "pending", commit_sha = "", description = "Migrate test_ai_client_cli.py" }