docs(tracks): add live_gui_test_fixes_20260618 to tracks.md (shipped)
Added a new Track section for live_gui_test_fixes_20260618 documenting: - The 2 fixes (Issue 1: GUI subprocess crash; Issue 2: xdist race) - The 8 commits in this track (1 setup + 2 TDD red + 2 TDD green + 2 audit + 1 docs) - The 11/11 tier pass result - The blocks relationship: unblocks sub-track 2 of result_migration_20260616 - Out of scope: the 4 Gemini 503 skip markers (deferred to follow-up track)
This commit is contained in:
@@ -757,6 +757,40 @@ Lightweight chronology; full spec/plan/state per track is in the linked folder.
|
||||
|
||||
---
|
||||
|
||||
|
||||
#### Track: Live GUI Test Infrastructure Fixes (test_execution_sim_live crash + test_live_gui_workspace_exists race) `[track-created: 2026-06-18]` [shipped: 2026-06-18]
|
||||
*Link: [./tracks/live_gui_test_fixes_20260618/](./tracks/live_gui_test_fixes_20260618/), Spec: [./tracks/live_gui_test_fixes_20260618/spec.md](./tracks/live_gui_test_fixes_20260618/spec.md), Plan: [./tracks/live_gui_test_fixes_20260618/plan.md](./tracks/live_gui_test_fixes_20260618/plan.md), Metadata: [./tracks/live_gui_test_fixes_20260618/metadata.json](./tracks/live_gui_test_fixes_20260618/metadata.json), Report: [../../docs/reports/TRACK_COMPLETION_live_gui_test_fixes_20260618.md](../../docs/reports/TRACK_COMPLETION_live_gui_test_fixes_20260618.md)*
|
||||
|
||||
*Status: 2026-06-18 - SHIPPED. 4 phases, 8 atomic commits (1 setup + 4 TDD/test/fix + 2 docs + 1 audit). Pre-conditions for sub-track 2's full closure. Scope: 2 issues fixed; 2 src files modified + 2 test files extended + 1 conftest modified + 2 docs + 2 audit logs. Test result: 11/11 tiers PASS clean (~825s total).*
|
||||
|
||||
*Goal: Fix the 2 documented test infrastructure issues that blocked sub-track 2 (`result_migration_small_files_20260617`) from full closure. The 2 issues were reported as "documented issues" by sub-track 2 Phase 13 (commit `30ca3265`). Both are pre-existing (not regressions from the Result[T] migration).*
|
||||
|
||||
*The 2 fixes:*
|
||||
|
||||
*Issue 1: `test_execution_sim_live` GUI subprocess crash (`tier-3-live_gui`)*
|
||||
- Symptom: GUI subprocess (port 8999) crashes mid-test with `0xC00000FD = STATUS_STACK_OVERFLOW`
|
||||
- Root cause: `imgui.set_window_focus("Response")` was called directly during the response panel render, exhausting the GUI main thread's 1.94 MB stack on Windows
|
||||
- Fix: defer the focus call to the next frame's idle phase via a new `_pending_focus_response` flag (commits `d02c6d56`, `0f796d7d`)
|
||||
- Same root cause as `test_z_negative_flows.py` (documented in `docs/reports/NEGATIVE_FLOWS_INVESTIGATION_20260617_REFINED.md`)
|
||||
|
||||
*Issue 2: `test_live_gui_workspace_exists` xdist race (`tier-1-unit-gui`)*
|
||||
- Symptom: xdist race where the owner worker's teardown removes the shared workspace path before a client worker's test can assert it exists
|
||||
- Root cause: `live_gui_workspace` fixture in `tests/conftest.py:727` returned `handle.workspace` without ensuring the path existed
|
||||
- Fix: call `workspace.mkdir(parents=True, exist_ok=True)` before returning (commits `3fdb2592`, `bf6bc67b`)
|
||||
- Pre-existing on parent commit `4ab7c732` (verified in `tests/artifacts/PHASE14_PARENT_VERIFICATION.log`)
|
||||
|
||||
*Deliverables:*
|
||||
- *1 setup commit (`chore(scripts): relocate Tier 2 state paths to project-relative`) - honors NEVER USE APPDATA directive; the failcount state and write_report failures directory now default to project-relative paths under `tests/artifacts/`*
|
||||
- *2 TDD red + 2 TDD green commits (one pair per issue)*
|
||||
- *1 audit commit (`chore(audit): Phase 14.1 - verify Issue 2 on parent commit 4ab7c732`)*
|
||||
- *1 audit commit (`chore(audit): Phase 4.1 - 11/11 test tiers PASS clean`)*
|
||||
- *2 docs commits (sub-track 2 reports updated with Phase 14 addendum)*
|
||||
- *1 track artifact import commit (`conductor(track): import live_gui_test_fixes_20260618 artifacts`)*
|
||||
|
||||
*`blocks:` sub-track 2 of `result_migration_20260616` (full closure requires the 2 issues fixed).*
|
||||
|
||||
*Out of scope (deferred to follow-up track): the 4 `@pytest.mark.skip` markers for Gemini 503 pre-existing failures (`test_auto_aggregate_skip`, `test_view_mode_summary`, `test_view_mode_default_summary`, `test_view_mode_custom_empty_default_to_summary`). To remove them, mock the Gemini API in `summarize.summarise_file` for tests.*
|
||||
|
||||
## Phase 9: Chore Tracks
|
||||
|
||||
*Initialized: 2026-06-07*
|
||||
|
||||
Reference in New Issue
Block a user