HomePhabricator
Diffusion Comm f3f886ab6d54

[Services] Fix backup test suite build

Description

[Services] Fix backup test suite build

Summary:
I would like to add the test suites to the CI, but they have to
build first

Also, a big change is building the testsuite by default, however, this is
the common practice, and generally people opt-out of building tests by doing
-DBUILD_TESTING=OFF when they invoke cmake.

https://linear.app/comm/issue/ENG-1794

Test Plan:

nix develop
cd services/backup
rm -rf build && mkdir build && cd build && cmake .. && make -j
ctest # should fail as AWS needs to be configured, but run

# without tests
nix develop
cd services/backup
rm -rf build && mkdir build && cd build && cmake .. -DBUILD_TESTING=OFF && make -j
ctest # should fail to find tests

Reviewers: atul, abosh, varun, max, karol

Reviewed By: atul

Subscribers: ashoat, tomek

Differential Revision: https://phab.comm.dev/D5117

Details

Provenance
jonAuthored on Sep 12 2022, 1:11 PM
Reviewer
atul
Differential Revision
D5117: [Services] Fix backup test suite build
Parents
rCOMM07375b672901: [Services] Fix blob test build
Branches
Unknown
Tags
Unknown