Merge pull request #11 from NordSecurity/kristupas/add-valgrind-tests

Add Valgrind tests

.github/workflows/cpp.yml

@@ -23,7 +23,7 @@ jobs:
   test-bindings:
     runs-on: ubuntu-latest
     container:
-      image: ghcr.io/nordsecurity/uniffi-bindgen-cpp-test-runner:v0.1.0
+      image: ghcr.io/nordsecurity/uniffi-bindgen-cpp-test-runner:v0.2.0
     steps:
     - uses: actions/checkout@v3
     - name: Test bindings

Dockerfile

@@ -2,6 +2,6 @@ FROM gcc:10-bullseye
 
 LABEL org.opencontainers.image.source=https://github.com/NordSecurity/uniffi-bindgen-cpp
 
-RUN apt-get update && apt-get install -y --no-install-recommends cmake curl && apt-get clean
+RUN apt-get update && apt-get install -y --no-install-recommends cmake curl valgrind && apt-get clean
 
 RUN curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y --default-toolchain=1.72

cpp-tests/CMakeLists.txt

@@ -20,6 +20,14 @@ target_compile_definitions(${TEST_NAME}-test PRIVATE UNIFFI_BINDING_DIR="${BINDI
 
 add_test(NAME ${TEST_NAME}-test COMMAND ${TEST_NAME}-test)
 
+add_test(NAME ${TEST_NAME}-test-memcheck
+  COMMAND valgrind
+    --error-exitcode=1
+    --tool=memcheck
+    --leak-check=full
+    --errors-for-leak-kinds=definite
+    --show-leak-kinds=definite $<TARGET_FILE:${TEST_NAME}-test>)
+
 add_dependencies(${TEST_NAME}-test bindings)
 
 list(APPEND BINDING_FILES ${BINDINGS_SRC_DIR}/${TEST_NAME}.cpp)

docker.sh

@@ -5,4 +5,4 @@ docker run \
     -ti --rm \
     --volume $PWD:/mounted_workdir \
     --workdir /mounted_workdir \
-    ghcr.io/nordsecurity/uniffi-bindgen-cpp-test-runner:v0.1.0 bash
+    ghcr.io/nordsecurity/uniffi-bindgen-cpp-test-runner:v0.2.0 bash

test_bindings.sh

@@ -3,6 +3,6 @@ set -euxo pipefail
 
 mkdir -p cpp-tests/build
 cd cpp-tests/build
-cmake ..
+cmake -DCMAKE_BUILD_TYPE=Debug ..
 make
-make test
+CTEST_OUTPUT_ON_FAILURE=1 make test