fix(auto_debug): align test expectation with fail-safe LLM exception handling
CI / lint (pull_request) Successful in 59s
CI / typecheck (pull_request) Successful in 1m16s
CI / quality (pull_request) Successful in 54s
CI / security (pull_request) Successful in 1m10s
CI / build (pull_request) Successful in 34s
CI / push-validation (pull_request) Successful in 35s
CI / helm (pull_request) Successful in 40s
CI / unit_tests (pull_request) Successful in 5m2s
CI / docker (pull_request) Successful in 1m30s
CI / integration_tests (pull_request) Successful in 9m8s
CI / coverage (pull_request) Successful in 9m18s
CI / status-check (pull_request) Successful in 5s
CI / lint (push) Successful in 52s
CI / build (push) Successful in 50s
CI / quality (push) Successful in 55s
CI / typecheck (push) Successful in 1m1s
CI / security (push) Successful in 1m4s
CI / push-validation (push) Successful in 24s
CI / helm (push) Successful in 56s
CI / unit_tests (push) Successful in 4m50s
CI / docker (push) Successful in 1m32s
CI / integration_tests (push) Successful in 8m29s
CI / coverage (push) Successful in 9m35s
CI / status-check (push) Successful in 3s
CI / benchmark-publish (push) Has been cancelled
CI / benchmark-regression (push) Has been cancelled

The _validate_fix BDD scenario "Validate fix handles LLM invocation
failure gracefully" was asserting fix_validated=True (fail-open), but
the code already sets is_valid=False on LLM exception (fail-safe).
Update the scenario step to "the fix should not be marked as validated"
and remove the pragma: no cover comment since this branch is now
exercised by the test.

ISSUES CLOSED: #10496
This commit was merged in pull request #11153.
This commit is contained in:
2026-06-15 01:01:54 -04:00
parent bf4da47bdc
commit ac74edd175
2 changed files with 2 additions and 2 deletions
+1 -1
View File
@@ -100,7 +100,7 @@ Feature: AutoDebug graph node coverage boost
Given an auto debug agent with a failing LLM
And a state with a current fix to validate
When I call _validate_fix with the prepared state
Then the fix should be marked as validated
Then the fix should not be marked as validated
# -----------------------------------------------------------------------
# _should_retry_fix routing