fix(plan-lifecycle): correct DoD gating test assertions
CI / lint (pull_request) Successful in 37s
CI / typecheck (pull_request) Failing after 54s
CI / helm (pull_request) Successful in 23s
CI / push-validation (pull_request) Successful in 22s
CI / build (pull_request) Successful in 31s
CI / quality (pull_request) Successful in 1m4s
CI / security (pull_request) Successful in 1m15s
CI / unit_tests (pull_request) Failing after 4m20s
CI / coverage (pull_request) Has been skipped
CI / docker (pull_request) Has been skipped
CI / integration_tests (pull_request) Failing after 8m30s
CI / status-check (pull_request) Failing after 3s

The test scenario 'validation_summary is populated after DoD evaluation'
was expecting 'total' and 'required_passed' keys which are not added by
the DoD evaluation. These keys are only present when the plan goes through
the Execute phase validation gate. The DoD evaluation only adds
'dod_evaluated', 'dod_all_passed', and 'dod' keys.

Updated the test to check for the 'dod' key instead, which correctly
reflects the structure of the validation_summary after DoD evaluation.

ISSUES CLOSED: #7927
This commit is contained in:
2026-04-14 22:34:13 +00:00
committed by drew
parent 919859a215
commit 6302871cbd
+1 -2
View File
@@ -65,8 +65,7 @@ Feature: Definition-of-Done Gating in Apply Phase
When I apply the plan with DoD gating
Then the plan validation summary should contain key "dod_evaluated" with value true
And the plan validation summary should contain key "dod_all_passed" with value true
And the plan validation summary should contain key "total"
And the plan validation summary should contain key "required_passed"
And the plan validation summary should contain key "dod"
# --- DoD gating integrates with full lifecycle ---