Skip to content

Commit 4c36881

Browse files
committed
[SymForce] Run tests once
We've been running all our tests twice in github actions on passing runs. `ctest --rerun-failed` runs all the tests if none of the tests have ever failed. This should save around 800s on github actions (idk why symengine_symforce_docs_test is so slow on github actions, possibly the lower core count). Tested both the passing and failing cases on github actions [here](aaron-skydio#3) Reviewers: bradley,nathan,chao,hayk Topic: sf-test-once GitOrigin-RevId: 84eeb36ab9f82350dd73c2a9f3c082a1db6278eb
1 parent f8f7eb4 commit 4c36881

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -128,8 +128,11 @@ jobs:
128128
pip install build/lcmtypes/python2.7
129129
export PYTHONPATH=$PYTHONPATH:$(pwd)
130130
${{ matrix.python }} test/symforce_requirements_test.py --update
131-
ctest --test-dir build -j $(nproc) || true
132-
ctest --test-dir build -j $(nproc) --rerun-failed --output-on-failure
131+
EXIT_CODE=0
132+
ctest --test-dir build -j $(nproc) || EXIT_CODE=$?
133+
if [ $EXIT_CODE -ne 0 ]; then
134+
ctest --test-dir build -j $(nproc) --rerun-failed --output-on-failure
135+
fi
133136
134137
# Upload the docs as an artifact
135138
# To view, download the `docs` artifact from the build and matrix entry you're interested in

0 commit comments

Comments
 (0)