Skip to content

Commit 42ffefa

Browse files
henrypark133claude
andauthored
fix: remove -x from coverage pytest to prevent suite-blocking failures (#1360)
One flaky test (test_builtin_echo_tool timeout) was stopping the entire e2e coverage suite via -x, preventing 118+ remaining tests from running and generating coverage data. Tests are independent (each gets a fresh browser context via the function-scoped page fixture), so removing -x is safe. Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
1 parent 2020270 commit 42ffefa

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/coverage.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@ jobs:
174174
175175
- name: Run E2E tests
176176
run: |
177-
pytest tests/e2e/ -v -x --timeout=120
177+
pytest tests/e2e/ -v --timeout=120
178178
env:
179179
RUST_LOG: ironclaw=info
180180
RUST_BACKTRACE: "1"

0 commit comments

Comments
 (0)