Merge pull request #63 from koinos/valgrind

Add Valgrind to CI
koinos · Aug 8, 2022 · 5dcbc24 · 5dcbc24
2 parents 5b53c56 + 19eca6f
commit 5dcbc24
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/ci/install.sh b/ci/install.sh
@@ -5,7 +5,7 @@ if [[ -z $BUILD_DOCKER ]]; then
    sudo -E apt-get -yq --no-install-suggests --no-install-recommends --allow-downgrades --allow-remove-essential --allow-change-held-packages install clang-11 llvm-11 -o Debug::pkgProblemResolver=yes
 
    if [ "$RUN_TYPE" = "coverage" ]; then
-      sudo apt-get install -y lcov ruby
+      sudo apt-get install -y lcov ruby valgrind
       sudo gem install coveralls-lcov
    fi
 

diff --git a/ci/test.sh b/ci/test.sh
@@ -6,6 +6,9 @@ set -x
 if [ "$RUN_TYPE" = "test" ]; then
    cd $(dirname "$0")/../build/tests
    exec ctest -j3 --output-on-failure && ../libraries/vendor/mira/test/mira_test
+elif [ "$RUN_TYPE" = "coverage" ]; then
+   cd $(dirname "$0")/../build/tests
+   exec valgrind --error-exitcode=1 --leak-check=yes ./koinos_mempool_tests
 fi
 
 if ! [[ -z $BUILD_DOCKER ]]; then