From bf295abb5b37b5b1b93e468527d80ce0e8ae7542 Mon Sep 17 00:00:00 2001 From: Jakub Jelen Date: Wed, 5 Nov 2025 14:55:03 +0100 Subject: [PATCH] tests: Remove the -E which is overridden by followed -E on ctest CLI The threads_pki_rsa was running and working under valgrind for some time already without anyone noticing this syntax does not work. Signed-off-by: Jakub Jelen Reviewed-by: Andreas Schneider (cherry picked from commit 950abbbd8141701d97292dcc926791ec8882e107) --- tests/CMakeLists.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt index e537d53c..a5dc4038 100644 --- a/tests/CMakeLists.txt +++ b/tests/CMakeLists.txt @@ -373,10 +373,10 @@ if (FUZZ_TESTING) endif() add_custom_target(test_memcheck - # FIXME: The threads_pki_rsa test is skipped under valgrind as it times out + # FIXME: The pkd_hello_i1 test is skipped under valgrind as it times out # Passing suppression file is also stupid so lets go with override here: # https://stackoverflow.com/a/56116311 - COMMAND ${CMAKE_CTEST_COMMAND} -E torture_threads_pki_rsa -E pkd_hello_i1 + COMMAND ${CMAKE_CTEST_COMMAND} -E pkd_hello_i1 --output-on-failure --force-new-ctest-process --test-action memcheck --overwrite MemoryCheckSuppressionFile=${CMAKE_SOURCE_DIR}/tests/valgrind.supp WORKING_DIRECTORY "${CMAKE_BINARY_DIR}")