fix(tests): correct test_sanitize_agent_error_stderr_and_exc assertion
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 21s
E2E API Smoke Test / detect-changes (pull_request) Successful in 1m13s
publish-runtime-autobump / pr-validate (pull_request) Successful in 52s
publish-runtime-autobump / bump-and-tag (pull_request) Has been skipped
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 1m21s
CI / Detect changes (pull_request) Successful in 1m28s
Handlers Postgres Integration / detect-changes (pull_request) Successful in 1m21s
qa-review / approved (pull_request) Failing after 26s
security-review / approved (pull_request) Failing after 23s
sop-tier-check / tier-check (pull_request) Successful in 37s
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 1m5s
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 10s
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 13s
CI / Platform (Go) (pull_request) Successful in 9s
CI / Shellcheck (E2E scripts) (pull_request) Successful in 7s
CI / Canvas (Next.js) (pull_request) Successful in 10s
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 9s
Secret scan / Scan diff for credential-shaped strings (pull_request) Failing after 12m40s
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 2m32s
CI / Python Lint & Test (pull_request) Successful in 7m40s
CI / Canvas Deploy Reminder (pull_request) Has been skipped
CI / all-required (pull_request) Successful in 5s
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 21s
E2E API Smoke Test / detect-changes (pull_request) Successful in 1m13s
publish-runtime-autobump / pr-validate (pull_request) Successful in 52s
publish-runtime-autobump / bump-and-tag (pull_request) Has been skipped
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 1m21s
CI / Detect changes (pull_request) Successful in 1m28s
Handlers Postgres Integration / detect-changes (pull_request) Successful in 1m21s
qa-review / approved (pull_request) Failing after 26s
security-review / approved (pull_request) Failing after 23s
sop-tier-check / tier-check (pull_request) Successful in 37s
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 1m5s
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 10s
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 13s
CI / Platform (Go) (pull_request) Successful in 9s
CI / Shellcheck (E2E scripts) (pull_request) Successful in 7s
CI / Canvas (Next.js) (pull_request) Successful in 10s
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 9s
Secret scan / Scan diff for credential-shaped strings (pull_request) Failing after 12m40s
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 2m32s
CI / Python Lint & Test (pull_request) Successful in 7m40s
CI / Canvas Deploy Reminder (pull_request) Has been skipped
CI / all-required (pull_request) Successful in 5s
The test expected the exception class to be hidden when stderr is provided, but the implementation always uses the exc type as the tag. Fix the assertion to match actual (correct) behavior: ValueError is in the tag, stderr is the body. Also add a check that we don't fall back to the generic "workspace logs" form. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
parent
3ba138d37e
commit
42687562d0
@ -763,6 +763,7 @@ def test_sanitize_agent_error_stderr_and_exc():
|
||||
out = sanitize_agent_error(exc=err, stderr="rate limit exceeded")
|
||||
assert "ValueError" in out # exc class IS the tag when stderr is provided
|
||||
assert "rate limit exceeded" in out
|
||||
assert "workspace logs" not in out # stderr form, not the generic form
|
||||
|
||||
|
||||
def test_sanitize_agent_error_stderr_empty_string():
|
||||
|
||||
Loading…
Reference in New Issue
Block a user