ed/manual_slop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore(conductor): Add gui_legacy.py deletion to test stabilization track

Browse Source
This commit is contained in:
Edward R. Gonzalez
2026-03-02 22:16:40 -05:00
parent 6e71960976
commit e8479bf9ab
2 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
conductor/tracks/test_stabilization_20260302/plan.md
View File

@@ -23,8 +23,9 @@
- [ ] Focus on `api_events`, `execution_engine`, `token_usage`, and `agent_capabilities`. - [ ] Focus on `api_events`, `execution_engine`, `token_usage`, and `agent_capabilities`.
- [ ] Task: Resolve Simulation Entry Count Regressions - [ ] Task: Resolve Simulation Entry Count Regressions
- [ ] Fix entry count assertions in `test_context_sim_live` and align mocks. - [ ] Fix entry count assertions in `test_context_sim_live` and align mocks.
- [ ] Task: Remove Legacy `gui_legacy` Test Imports - [ ] Task: Remove Legacy `gui_legacy` Test Imports & File
- [ ] Refactor `tests/test_gui_events.py`, `test_gui_updates.py`, and `test_gui_diagnostics.py` to use `gui_2.py`. - [ ] Refactor `tests/test_gui_events.py`, `test_gui_updates.py`, and `test_gui_diagnostics.py` to use `gui_2.py`.
- [ ] Delete `gui_legacy.py` from the project root.
- [ ] Task: Conductor - User Manual Verification 'Phase 3: Assertions & Legacy Cleanup' (Protocol in workflow.md) - [ ] Task: Conductor - User Manual Verification 'Phase 3: Assertions & Legacy Cleanup' (Protocol in workflow.md)
## Phase 4: Documentation & Final Verification ## Phase 4: Documentation & Final Verification

2
conductor/tracks/test_stabilization_20260302/spec.md
View File

@@ -12,6 +12,7 @@ The goal of this track is to stabilize and unify the project's test suite. This
- Refactor integration/visual tests to exclusively use the `live_gui` pytest fixture. - Refactor integration/visual tests to exclusively use the `live_gui` pytest fixture.
- Eliminate all manual `subprocess.Popen` calls to `gui_2.py` in the `tests/` and `simulation/` directories. - Eliminate all manual `subprocess.Popen` calls to `gui_2.py` in the `tests/` and `simulation/` directories.
- Update legacy tests (e.g., `test_gui_events.py`, `test_gui_diagnostics.py`) that still import the deprecated `gui_legacy.py` to use `gui_2.py`. - Update legacy tests (e.g., `test_gui_events.py`, `test_gui_diagnostics.py`) that still import the deprecated `gui_legacy.py` to use `gui_2.py`.
- Completely remove `gui_legacy.py` from the project to eliminate confusion.
- **Artifact Isolation & Discipline:** - **Artifact Isolation & Discipline:**
- All test-generated files (temporary projects, mocks, sessions) MUST be isolated in `./tests/artifacts/`. - All test-generated files (temporary projects, mocks, sessions) MUST be isolated in `./tests/artifacts/`.
- Prevent leakage into `conductor/tracks/` or project root. - Prevent leakage into `conductor/tracks/` or project root.
@@ -29,6 +30,7 @@ The goal of this track is to stabilize and unify the project's test suite. This
- [ ] Full suite run completes without `RuntimeError: Event loop is closed` warnings. - [ ] Full suite run completes without `RuntimeError: Event loop is closed` warnings.
- [ ] No `subprocess.Popen` calls to `gui_2.py` exist in the test codebase. - [ ] No `subprocess.Popen` calls to `gui_2.py` exist in the test codebase.
- [ ] No test files import `gui_legacy.py`. - [ ] No test files import `gui_legacy.py`.
- [ ] `gui_legacy.py` has been deleted from the repository.
- [ ] All test artifacts are isolated in `./tests/artifacts/`. - [ ] All test artifacts are isolated in `./tests/artifacts/`.
- [ ] All tests previously marked with `pytest.fail` now have passing functional assertions. - [ ] All tests previously marked with `pytest.fail` now have passing functional assertions.
- [ ] Simulation tests pass with correct entry counts. - [ ] Simulation tests pass with correct entry counts.