Skip to content

Commit d53a57c

Browse files
r-barnesmeta-codesync[bot]
authored andcommitted
Make tests opt-in via -DTESTS=ON (off by default)
Summary: - Add a `TESTS` CMake option (default `OFF`) that guards `find_package(GTest)`, `include(GoogleTest)`, the `test_main` library, `rebalancer_tests.exe`, `enable_testing()`, `gtest_discover_tests()`, and the Python-based macro/path check tests - Mirrors the existing `EXAMPLES` and `BENCHMARKS` options — pass `-DTESTS=ON` to build tests - Add `TESTS=ON` to `[cmake.defines.test=on]` in the getdeps manifest so CI (which runs `getdeps build` without `--no-tests`) continues to build and run tests automatically; SDK and wheel builds already pass `--no-tests` and are unaffected X-link: https://github.com/facebookincubator/rebalancer/pull/22 Reviewed By: kvelakur Differential Revision: D109102488 Pulled By: r-barnes fbshipit-source-id: 77679978a42988fc1311aa6753ceea18e00d4cde
1 parent f98f38a commit d53a57c

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

build/fbcode_builder/manifests/rebalancer

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ CMAKE_POSITION_INDEPENDENT_CODE=ON
2525
# workflow's --extra-cmake-defines.
2626
[cmake.defines.test=on]
2727
EXAMPLES=ON
28+
TESTS=ON
2829

2930
[dependencies]
3031
boost

0 commit comments

Comments
 (0)