Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test runner refactor #508

Merged
merged 5 commits into from
Aug 8, 2024
Merged

Test runner refactor #508

merged 5 commits into from
Aug 8, 2024

Conversation

zrax
Copy link
Owner

@zrax zrax commented Aug 8, 2024

Ports the scripts for running tests to Python, which improves cross-platform compatibility.

  • The tests are still run in parallel by default, but the output is sorted again and the default number of jobs is now based on the number of CPU threads available.
  • Fix for using the cmake "check" target in multi-config generators (including MSVC).
  • GitHub actions runners now test both Debug and Release builds.

Closes #506 (no longer relevant since the script is Python now instead of Bash)

zrax added 4 commits August 8, 2024 10:55
This allows us to keep the test output in sorted order while still running the
tests in parallel.  It also now defaults to using the number of available CPU
threads for parallel execution, rather than the previously hard-coded default.
Adds testing of release builds on both MSVC and GCC.
@zrax zrax force-pushed the test_run_refactor branch from 69f5614 to 7aa83f8 Compare August 8, 2024 22:45
@zrax zrax merged commit f37caa8 into master Aug 8, 2024
5 checks passed
@zrax zrax deleted the test_run_refactor branch August 8, 2024 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant