ci: enable asan in test

Zxilly · Sep 20, 2024 · 358a856 · 358a856
1 parent f85cd46
commit 358a856
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -93,8 +93,9 @@ jobs:
 
       - name: Run unit tests
         run: |
-          cmake -S . -B build -DUA2F_BUILD_TESTS=OFF -DUA2F_ENABLE_UCI=OFF
+          cmake -S . -B build -DUA2F_BUILD_TESTS=OFF -DUA2F_ENABLE_UCI=OFF -DUA2F_ENABLE_ASAN=ON
           cmake --build build
+          ./build/ua2f --version
           sudo python scripts/test.py ./build/ua2f
           
 

diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -62,8 +62,7 @@ if (DEFINED ENV{UA2F_ENABLE_ASAN})
     set(UA2F_ENABLE_ASAN $ENV{UA2F_ENABLE_ASAN})
     message(STATUS "UA2F_ENABLE_ASAN set from environment: ${UA2F_ENABLE_ASAN}")
 else ()
-    set(UA2F_ENABLE_ASAN OFF)
-    message(STATUS "UA2F_ENABLE_ASAN not set in environment, using default: OFF")
+    message(STATUS "UA2F_ENABLE_ASAN not set in environment")
 endif ()
 
 if (UA2F_ENABLE_ASAN)