Skip to content

Commit

Permalink
Enable RNTuple builds for sanitizer CI workflows
Browse files Browse the repository at this point in the history
  • Loading branch information
tmadlener committed Oct 10, 2024
1 parent ca629d1 commit 1d20b76
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 0 deletions.
1 change: 1 addition & 0 deletions .github/workflows/sanitizers.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ jobs:
-DUSE_EXTERNAL_CATCH2=OFF \
-DENABLE_SIO=ON \
-DENABLE_JULIA=ON \
-DENABLE_RNTUPLE=ON \
-G Ninja ..
echo "::endgroup::"
echo "::group::Build"
Expand Down
1 change: 1 addition & 0 deletions cmake/podioTest.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ function(PODIO_SET_TEST_ENV test)
PODIO_BASE=${PROJECT_SOURCE_DIR}
ENABLE_SIO=${ENABLE_SIO}
PODIO_BUILD_BASE=${PROJECT_BINARY_DIR}
LSAN_OPTIONS=suppressions=${PROJECT_SOURCE_DIR}/tests/root_io/leak_sanitizer_suppressions.txt
)
set_property(TEST ${test}
PROPERTY ENVIRONMENT "${test_environment}"
Expand Down
8 changes: 8 additions & 0 deletions tests/CTestCustom.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,8 @@ if ((NOT "@FORCE_RUN_ALL_TESTS@" STREQUAL "ON") AND (NOT "@USE_SANITIZER@" STREQ
write_python_frame_sio
read_python_frame_sio

write_python_frame_rntuple

relation_range

pyunittest
Expand All @@ -45,10 +47,16 @@ if ((NOT "@FORCE_RUN_ALL_TESTS@" STREQUAL "ON") AND (NOT "@USE_SANITIZER@" STREQ
podio-dump-legacy_sio_v00-16-06
podio-dump-legacy_sio-detailed_v00-16-06

podio-dump-rntuple
podio-dump-rntuple-detailed

datamodel_def_store_roundtrip_root
datamodel_def_store_roundtrip_root_extension
datamodel_def_store_roundtrip_sio
datamodel_def_store_roundtrip_sio_extension
datamodel_def_store_roundtrip_rntuple
datamodel_def_store_roundtrip_rntuple_extension


write_old_data_root
read_new_data_root
Expand Down
2 changes: 2 additions & 0 deletions tests/root_io/leak_sanitizer_suppressions.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
# Ignore leaks from Cling
leak:Cling
2 changes: 2 additions & 0 deletions tests/unittests/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -83,5 +83,7 @@ else()
PODIO_SIOBLOCK_PATH=${PROJECT_BINARY_DIR}/tests
ENVIRONMENT
LD_LIBRARY_PATH=${CMAKE_CURRENT_BINARY_DIR}:${PROJECT_BINARY_DIR}/src:${PROJECT_BINARY_DIR}/tests:$<TARGET_FILE_DIR:ROOT::Tree>:$<$<TARGET_EXISTS:SIO::sio>:$<TARGET_FILE_DIR:SIO::sio>>:$ENV{LD_LIBRARY_PATH}
ENVIRONMENT
LSAN_OPTIONS=suppressions=${PROJECT_SOURCE_DIR}/tests/root_io/leak_sanitizer_suppressions.txt
)
endif()

0 comments on commit 1d20b76

Please sign in to comment.