Skip to content

Commit f4247de

Browse files
committed
ci:valgrind: skip deflate due to zlib/os limits
1 parent c75df58 commit f4247de

File tree

3 files changed

+12
-6
lines changed

3 files changed

+12
-6
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ jobs:
129129
130130
- uses: actions/checkout@v4
131131

132-
- run: ctest -Dexclude_label=python -S memcheck.cmake -VV
132+
- run: ctest -Dexclude_label="deflate|python" -S memcheck.cmake -VV
133133

134134

135135
# issue with python loading writer.inc

memcheck.cmake

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,8 +85,8 @@ endif()
8585
ctest_memcheck(
8686
INCLUDE ${include}
8787
INCLUDE_LABEL ${include_label}
88-
EXCLUDE ${exclude}
89-
EXCLUDE_LABEL ${exclude_label}
88+
EXCLUDE "${exclude}"
89+
EXCLUDE_LABEL "${exclude_label}"
9090
RETURN_VALUE ret
9191
CAPTURE_CMAKE_ERROR err
9292
DEFECT_COUNT count

test/CMakeLists.txt

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -74,10 +74,16 @@ set_property(TEST shape PROPERTY FIXTURES_SETUP h5shape)
7474
set_property(TEST layout PROPERTY FIXTURES_REQUIRED test_files)
7575
set_property(TEST layout PROPERTY REQUIRED_FILES ${CMAKE_CURRENT_BINARY_DIR}/test_write.h5)
7676

77-
set_property(TEST deflate_write PROPERTY FIXTURES_SETUP deflate_files)
77+
set_tests_properties(deflate_write PROPERTIES
78+
FIXTURES_SETUP deflate_files
79+
LABELS deflate
80+
)
7881

79-
set_property(TEST deflate_props deflate_read PROPERTY FIXTURES_REQUIRED deflate_files)
80-
set_property(TEST deflate_props deflate_read PROPERTY REQUIRED_FILES ${CMAKE_CURRENT_BINARY_DIR}/deflate1.h5)
82+
set_tests_properties(deflate_props deflate_read PROPERTIES
83+
FIXTURES_REQUIRED deflate_files
84+
REQUIRED_FILES ${CMAKE_CURRENT_BINARY_DIR}/deflate1.h5
85+
LABELS deflate
86+
)
8187

8288
if(h5fortran_COVERAGE)
8389
setup_target_for_coverage_gcovr_html(

0 commit comments

Comments
 (0)