Skip to content

Commit 165acc0

Browse files
cortinicometa-codesync[bot]
authored andcommitted
Make Fantom CI retries rerun the test suite (#58672)
Summary: Pull Request resolved: #58672 Fantom CI retries used Jest's `--onlyFailures` cache after a failed full-suite run. Process-level failures such as `SIGSEGV` are not recorded as failed test results, so Jest selected no tests and both retries exited immediately. Run the full Fantom suite on each retry so transient runner crashes get real retry attempts. Changelog: [Internal] ___ Differential Revision: D121570534 fbshipit-source-id: 7d325eb8786848cbb018d9db6fea7e51650bb9e1
1 parent 88b9c3f commit 165acc0

1 file changed

Lines changed: 3 additions & 9 deletions

File tree

‎.github/actions/run-fantom-tests/action.yml‎

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -27,17 +27,11 @@ runs:
2727
run: |
2828
for attempt in 1 2 3; do
2929
echo "Attempt $attempt of 3"
30-
if [ "$attempt" -eq 1 ]; then
31-
if yarn fantom; then
32-
exit 0
33-
fi
34-
else
35-
if yarn fantom --onlyFailures; then
36-
exit 0
37-
fi
30+
if yarn fantom; then
31+
exit 0
3832
fi
3933
if [ "$attempt" -lt 3 ]; then
40-
echo "Attempt $attempt failed. Retrying only failed tests..."
34+
echo "Attempt $attempt failed. Retrying..."
4135
fi
4236
done
4337
echo "All 3 attempts failed."

0 commit comments

Comments
 (0)