Clean up ctest configuration and CI test configuration. (#7620)
authorAina Niemetz <aina.niemetz@gmail.com>
Tue, 9 Nov 2021 20:21:08 +0000 (12:21 -0800)
committerGitHub <noreply@github.com>
Tue, 9 Nov 2021 20:21:08 +0000 (20:21 +0000)
commita71f9a9dfdc512274635d80be9e3662ed745b14c
tree8a3880a32554a4bc3b4982b19e06509e2aa328a0
parent5279adad97c7e85ca36ebc9497fa1b6c801c7ab6
Clean up ctest configuration and CI test configuration. (#7620)

Previously, on CI, unit tests and api tests were run twice since we use
a ctest exclude rule based on labels (-LE) which includes unit and api
tests, but then run them separately again. This cleans up the CI test
configuration.

Further, unit gtest unit tests were added with gtest_add_tests, which
adds every test of a unit test binary as a single test target to ctest.
In theory, this may speed up testing (because more parallelism) but in
practice it slows it down due to the start up overhead. It also clutters
CI output. This cleans up the gtest configuration to add the gtest unit
tests per test binary rather then per test of a test binary.
.github/actions/run-tests/action.yml
.github/workflows/ci.yml
test/unit/CMakeLists.txt