From 1f684a7d3180fdb54811edcc22aab83530e591a3 Mon Sep 17 00:00:00 2001
From: Jayesh Badwaik <jayesh@badwaik.in>
Date: Thu, 26 Nov 2020 13:57:38 +0530
Subject: [PATCH] - ci: use inbuilt cmake support for memcheck

---
 CMakeLists.txt  | 2 +-
 ci/bin/valgrind | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 866ccfe..e0623fe 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -16,7 +16,7 @@ include(GNUInstallDirs)
 
 option(BUILD_TESTING "Build Tests" OFF)
 option(BUILD_DOC "Build Documentation" OFF)
-enable_testing()
+include(CTest)
 
 #-------------------------------------------------------------------------------
 # Library Build Code
diff --git a/ci/bin/valgrind b/ci/bin/valgrind
index 21b7a40..25ce694 100755
--- a/ci/bin/valgrind
+++ b/ci/bin/valgrind
@@ -38,10 +38,10 @@ pushd $BUILD_DIRECTORY
     -DCMAKE_BUILD_TYPE=$BUILD_TYPE \
     -DBUILD_TESTING=On \
     -DENABLE_COVERAGE=$COVERAGE \
+    -DMEMORYCHECK_COMMAND_OPTIONS="--error-exitcode=1 --leak-check=full" \
     -B$BUILD_DIRECTORY -S$SOURCE_DIRECTORY
   cmake --build $BUILD_DIRECTORY --parallel 1
-  ctest --output-on-failure -L UnitTest
-  valgrind --leak-check=full --error-exitcode=255 ./test/bin/unit.t
+  ctest --output-on-failure -L UnitTest -T memcheck
 
   if [[ $COVERAGE == "On" ]]; then
     ninja vanadium_coverage
-- 
GitLab