mirror of
https://github.com/XRPLF/rippled.git
synced 2025-11-15 00:25:51 +00:00
8d2dff2e48
This change unifies the build and test jobs into a single job, and adds `ctest` to coverage reporting. The mechanics of coverage reporting is slightly complex and most of it is encapsulated in the `coverage` target. The status quo way of preparing coverage reports involves running a single target `cmake --build . --target coverage`, which does three things: * Build the `rippled` binary (via target dependency) * Prepare coverage reports: * Run `./rippled -u` unit tests. * Gather test output and build reports. This makes it awkward to add an additional `ctest` step between build and coverage reporting steps. The better solution is to split `coverage` target into separate build, followed by `ctest`, followed by test generation. Luckily, the `coverage` target has been designed specifically to support such case; it does not need to build `rippled`, it's just a dependency. Similarly it allows additional tests to be run before gathering test outputs; in principle we could even strip it from running tests and run them separately instead. This means we can keep build, `ctest` and generation of coverage reports as separate steps, as long as the state of build directory is fully (including file timestamps, additional coverage files etc.) preserved between the steps. This means that in order to run `ctest` for coverage reporting we need to integrate build and test into a single job, which this change does.
26 lines
645 B
CMake
26 lines
645 B
CMake
include(isolate_headers)
|
|
|
|
function(xrpl_add_test name)
|
|
set(target ${PROJECT_NAME}.test.${name})
|
|
|
|
file(GLOB_RECURSE sources CONFIGURE_DEPENDS
|
|
"${CMAKE_CURRENT_SOURCE_DIR}/${name}/*.cpp"
|
|
"${CMAKE_CURRENT_SOURCE_DIR}/${name}.cpp"
|
|
)
|
|
add_executable(${target} ${ARGN} ${sources})
|
|
|
|
isolate_headers(
|
|
${target}
|
|
"${CMAKE_SOURCE_DIR}"
|
|
"${CMAKE_SOURCE_DIR}/tests/${name}"
|
|
PRIVATE
|
|
)
|
|
|
|
# Make sure the test isn't optimized away in unity builds
|
|
set_target_properties(${target} PROPERTIES
|
|
UNITY_BUILD_MODE GROUP
|
|
UNITY_BUILD_BATCH_SIZE 0) # Adjust as needed
|
|
|
|
add_test(NAME ${target} COMMAND ${target})
|
|
endfunction()
|