Merge pull request #1 from mrsobakin/master

Add memory leaks test
bialger · Jan 17, 2024 · 763997b · 763997b
2 parents bf8e281 + 3e555e8
commit 763997b
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 2 deletions.
diff --git a/.github/workflows/ci_tests.yml b/.github/workflows/ci_tests.yml
@@ -34,7 +34,7 @@ jobs:
 
     steps:
       - uses: actions/checkout@v3
-      - name: Create CMake cash
+      - name: Create CMake cache
         run: |
           cmake -S . -B cmake-build
 
@@ -67,3 +67,26 @@ jobs:
           else
               ./cpp_tests_tests
           fi
+
+  memory-leaks:
+    name: Memory leaks
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+
+      - name: Install valgrind
+        run: |
+          sudo apt-get update && sudo apt-get -y install valgrind
+
+      - name: Create CMake cache
+        run: |
+          cmake -S . -B cmake-build
+
+      - name: Build tests target
+        run: |
+          cmake --build cmake-build --target cpp_tests_tests
+
+      - name: Run valgrind
+        working-directory: ./cmake-build/tests
+        run: |
+          valgrind --leak-check=full --track-origins=yes --error-exitcode=1 ./cpp_tests_tests
diff --git a/tests/main_test.cpp b/tests/main_test.cpp
@@ -12,4 +12,4 @@ TEST(BasicTestSuite, BasicTest1) {
   std::ostringstream out;
   out << "Hello, World!";
   ASSERT_EQ(out.str(), "Hello, World!");
-}
+}