Skip to content

Commit cd536f0

Browse files
CI: run the Windows tests through ctest with --repeat until-pass (#1205)
The Windows and pixi-Windows jobs ran the gtest executable once, so a single wall-clock timing test missing its budget on a loaded runner failed the whole PR (RepeatTestAsync.RepeatTestAsync, SequenceTripleActionTest.TripleAction). Use ctest --repeat until-pass:3, as the macOS job already does. Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
1 parent 16e1917 commit cd536f0

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

‎.github/workflows/cmake_windows.yml‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,4 +49,8 @@ jobs:
4949

5050
- name: Run tests
5151
working-directory: ${{ github.workspace }}/build
52-
run: $env:PATH+=";${{ matrix.build_type }}"; tests/${{ matrix.build_type }}/behaviortree_cpp_test.exe
52+
# Some tests assert tight wall-clock timing, which the shared runners can
53+
# not always meet: --repeat until-pass absorbs that flakiness (as on macOS).
54+
run: |
55+
$env:PATH += ";${{ github.workspace }}\build\${{ matrix.build_type }}"
56+
ctest -C ${{ matrix.build_type }} --output-on-failure --repeat until-pass:3

‎pixi.toml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ channels = ["conda-forge"]
77
platforms = ["linux-64", "win-64"]
88

99
[target.win-64.tasks]
10-
test = "PATH=\"$PATH;build/Release\" build/tests/Release/behaviortree_cpp_test.exe"
10+
test = "PATH=\"$PATH;$PIXI_PROJECT_ROOT/build/Release\" ctest --test-dir build -C Release --output-on-failure --repeat until-pass:3"
1111

1212
[target.linux-64.tasks]
1313
test = "./build/tests/behaviortree_cpp_test"

0 commit comments

Comments
 (0)