From ffeb6f50f52ea6938644fa2dca020c2365f4b806 Mon Sep 17 00:00:00 2001 From: Ed_ Date: Mon, 23 Feb 2026 16:50:37 -0500 Subject: [PATCH] close live_gui_testing --- .../tracks/live_gui_testing_20260223/index.md | 5 ---- .../live_gui_testing_20260223/metadata.json | 8 ------ .../tracks/live_gui_testing_20260223/plan.md | 27 ------------------- .../tracks/live_gui_testing_20260223/spec.md | 25 ----------------- 4 files changed, 65 deletions(-) delete mode 100644 conductor/tracks/live_gui_testing_20260223/index.md delete mode 100644 conductor/tracks/live_gui_testing_20260223/metadata.json delete mode 100644 conductor/tracks/live_gui_testing_20260223/plan.md delete mode 100644 conductor/tracks/live_gui_testing_20260223/spec.md diff --git a/conductor/tracks/live_gui_testing_20260223/index.md b/conductor/tracks/live_gui_testing_20260223/index.md deleted file mode 100644 index f3d86a8..0000000 --- a/conductor/tracks/live_gui_testing_20260223/index.md +++ /dev/null @@ -1,5 +0,0 @@ -# Track live_gui_testing_20260223 Context - -- [Specification](./spec.md) -- [Implementation Plan](./plan.md) -- [Metadata](./metadata.json) diff --git a/conductor/tracks/live_gui_testing_20260223/metadata.json b/conductor/tracks/live_gui_testing_20260223/metadata.json deleted file mode 100644 index 62e4903..0000000 --- a/conductor/tracks/live_gui_testing_20260223/metadata.json +++ /dev/null @@ -1,8 +0,0 @@ -{ - "track_id": "live_gui_testing_20260223", - "type": "chore", - "status": "new", - "created_at": "2026-02-23T15:43:00Z", - "updated_at": "2026-02-23T15:43:00Z", - "description": "Update all tests to use a live running gui.py with --enable-test-hooks for real-time state and metrics verification." -} diff --git a/conductor/tracks/live_gui_testing_20260223/plan.md b/conductor/tracks/live_gui_testing_20260223/plan.md deleted file mode 100644 index 12e2292..0000000 --- a/conductor/tracks/live_gui_testing_20260223/plan.md +++ /dev/null @@ -1,27 +0,0 @@ -# Implementation Plan: Live GUI Testing Infrastructure - -## Phase 1: Infrastructure & Core Utilities [checkpoint: db251a1] -Establish the mechanism for managing the live GUI process and providing it to tests. - -- [x] Task: Create `tests/conftest.py` with a session-scoped fixture to manage the `gui.py --enable-test-hooks` process. -- [x] Task: Enhance `api_hook_client.py` with robust connection retries and health checks to handle GUI startup time. -- [x] Task: Update `conductor/workflow.md` to formally document the "Live GUI Testing" requirement and the use of the `--enable-test-hooks` flag. -- [x] Task: Conductor - User Manual Verification 'Phase 1: Infrastructure & Core Utilities' (Protocol in workflow.md) - -## Phase 2: Test Suite Migration [checkpoint: 6677a6e] -Migrate existing tests to use the live GUI fixture and API hooks. - -- [x] Task: Refactor `tests/test_api_hook_client.py` and `tests/test_conductor_api_hook_integration.py` to use the live GUI fixture. -- [x] Task: Refactor GUI performance tests (`tests/test_gui_performance_requirements.py`, `tests/test_gui_stress_performance.py`) to verify real metrics (FPS, memory) via hooks. -- [x] Task: Audit and update all remaining tests in `tests/` to ensure they either use the live server or are explicitly marked as pure unit tests. -- [x] Task: Conductor - User Manual Verification 'Phase 2: Test Suite Migration' (Protocol in workflow.md) - -## Phase 3: Conductor Integration & Validation [checkpoint: 637946b] -Ensure the Conductor framework itself supports and enforces this new testing paradigm. - -- [x] Task: Verify that new track creation generates plans that include specific API hook verification tasks. -- [x] Task: Perform a full test run using `run_tests.py` (or equivalent) to ensure 100% pass rate in the new environment. -- [x] Task: Conductor - User Manual Verification 'Phase 3: Conductor Integration & Validation' (Protocol in workflow.md) - -## Phase: Review Fixes -- [x] Task: Apply review suggestions 075d760 diff --git a/conductor/tracks/live_gui_testing_20260223/spec.md b/conductor/tracks/live_gui_testing_20260223/spec.md deleted file mode 100644 index 6a5d60a..0000000 --- a/conductor/tracks/live_gui_testing_20260223/spec.md +++ /dev/null @@ -1,25 +0,0 @@ -# Specification: Live GUI Testing Infrastructure - -## Overview -Update the testing suite to ensure all tests (especially GUI-related and integration tests) communicate with a live running instance of `gui.py` started with the `--enable-test-hooks` argument. This ensures that tests can verify the actual application state and metrics via the built-in API hooks. - -## Functional Requirements -- **Server-Based Testing:** All tests must be updated to interact with the application through its REST API hooks rather than mocking internal components where live verification is possible. -- **Automated GUI Management:** Implement a robust mechanism (preferably a pytest fixture) to start `gui.py --enable-test-hooks` before test execution and ensure it is cleanly terminated after tests complete. -- **Hook Client Integration:** Ensure `api_hook_client.py` is the primary interface for tests to communicate with the running GUI. -- **Documentation Alignment:** Update `conductor/workflow.md` to reflect the requirement for live testing and API hook verification. - -## Non-Functional Requirements -- **Reliability:** The process of starting and stopping the GUI must be stable and not leave orphaned processes. -- **Speed:** The setup/teardown of the live GUI should be optimized to minimize test suite overhead. -- **Observability:** Tests should log communication with the API hooks for easier debugging. - -## Acceptance Criteria -- [ ] All tests in the `tests/` directory pass when executed against a live `gui.py` instance. -- [ ] New track creation (e.g., via `/conductor:newTrack`) generates plans that include specific API hook verification tasks. -- [ ] `conductor/workflow.md` accurately describes the live testing protocol. -- [ ] Real-time UI metrics (FPS, CPU, etc.) are successfully retrieved and verified in at least one performance test. - -## Out of Scope -- Rewriting the entire GUI framework. -- Implementing new API hooks not required for existing test verification.