All 9 test cases previously only called `project create` + `project show`
despite documentation claiming to test correction, plan execution, and
subplan spawning workflows. This commit fixes that.
Changes per reviewer blockers:
Blocker 1 (hollow stubs): Each test now invokes the workflow its name
and documentation describe:
- test_correction_workflow.robot: calls `plan correct --mode revert/append`
(with --dry-run for mode tests; actual correction for state transition test)
- test_project_plan_workflow.robot: Plan Execution Workflow now calls
`plan use` + `plan execute` + `plan status`; Project Creation Workflow
uses resource-based project creation and verifies list output
- test_subplan_workflow.robot: Subplan Spawning and Merge Result Validation
inspect `plan tree` for spec-required child_plans/decision_ids fields;
Three-Way Merge Workflow exercises `plan apply`
Blocker 2 (temp dir leaks): All tests now use `Create Temp Git Repo`
(creates inside SUITE_HOME, cleaned by E2E Suite Teardown) instead of
`Create Temp Directory` (tempfile.mkdtemp outside SUITE_HOME, leaked).
The unused `Set Up E2E Project Test` local keyword (which created an
orphaned suite-variable temp dir) is removed from all three files.
Each file gains a proper suite setup keyword that registers the
local/code-review action needed by plan lifecycle tests, following
the pattern established in m6_acceptance.robot.
Address PR #10614 review findings:
- Added [Timeout] 30 minutes to all 9 test cases to prevent CI hangs
- Removed duplicate Create Temp Directory keyword from each file (moved to common_e2e.resource)
- Added Set Up E2E Project Test keyword for consistent temp dir creation via TEST NAME variable
- Added msg= parameters to Should Not Be Empty assertions for better debugging
Signed-off-by: HAL9000 <hal9000@noreply.git.cleverthis.com>
- Add test_project_plan_workflow.robot for project creation and plan execution workflows
- Add test_correction_workflow.robot for revert and append mode correction workflows
- Add test_subplan_workflow.robot for subplan spawning and three-way merge workflows
- All tests use Robot Framework with real CLI execution (no mocking)
- Tests validate spec-required output formats
- Tests skip gracefully if LLM API keys are not configured
Closes#5259
controller-ci-rerun
HAL9000
HAL9000