Skip to content

Commit 7016b93

Browse files
Merge pull request #926 from LalatenduMohanty/fix-preserve-failing-test-output
fix(e2e): preserve failing test output before next test cleanup
2 parents 6231755 + fdcb2ff commit 7016b93

2 files changed

Lines changed: 11 additions & 1 deletion

File tree

.github/workflows/test.yaml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,9 @@ jobs:
133133
uses: actions/upload-artifact@v6
134134
with:
135135
name: ${{ matrix.test-script }}-py${{ matrix.python-version }}-rs${{ matrix.rust-version }}-${{ matrix.os }}
136-
path: e2e-output
136+
path: |
137+
e2e-output
138+
e2e-failed-*
137139
138140
- name: Upload coverage
139141
uses: actions/upload-artifact@v6

e2e/ci_suite_framework.sh

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,14 @@ run_test() {
5454
echo "PASSED: $test_name"
5555
else
5656
echo "FAILED: $test_name"
57+
# Preserve output for debugging before next test cleans it up
58+
local outdir
59+
outdir="$(dirname "$SCRIPTDIR")/e2e-output"
60+
local failed_dir
61+
failed_dir="$(dirname "$SCRIPTDIR")/e2e-failed-${test_name}"
62+
if [ -d "$outdir" ]; then
63+
mv "$outdir" "$failed_dir" || true
64+
fi
5765
FAILED_TESTS+=("$test_name")
5866
# Continue running other tests instead of exiting immediately
5967
# This provides more comprehensive feedback in CI

0 commit comments

Comments
 (0)