configure tests as optional in CMake and adjust GitHub Actions workflows

farukeryilmaz · Jan 8, 2024 · 88bc5cb · 88bc5cb
1 parent 77ceb31
commit 88bc5cb
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 7 deletions.
diff --git a/.github/workflows/platform_gnu_linux.yml b/.github/workflows/platform_gnu_linux.yml
@@ -37,7 +37,7 @@ jobs:
         with:
           submodules: 'true'
       - name: Configure
-        run: cmake -S . -B build
+        run: cmake -S . -B build -DBYTEPACK_BUILD_TESTS=ON
       - name: Build
         run: cmake --build build
       - name: Run tests

diff --git a/.github/workflows/platform_macos.yml b/.github/workflows/platform_macos.yml
@@ -18,7 +18,7 @@ jobs:
         with:
           submodules: 'true'
       - name: Configure
-        run: cmake -S . -B build
+        run: cmake -S . -B build -DBYTEPACK_BUILD_TESTS=ON
       - name: Build
         run: cmake --build build
       - name: Run tests

diff --git a/.github/workflows/platform_windows.yml b/.github/workflows/platform_windows.yml
@@ -15,7 +15,7 @@ jobs:
         with:
           submodules: 'true'
       - name: Configure
-        run: cmake -S . -B build -DCMAKE_CXX_STANDARD=20
+        run: cmake -S . -B build -DCMAKE_CXX_STANDARD=20 -DBYTEPACK_BUILD_TESTS=ON
       - name: Build
         run: cmake --build build --config Release
       - name: Run tests

diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -15,7 +15,11 @@ endif()
 add_library(bytepack INTERFACE)
 target_include_directories(bytepack INTERFACE ${PROJECT_SOURCE_DIR}/include)
 
-add_subdirectory(extern/Catch2)
-
-enable_testing()
-add_subdirectory(test)
+# Option for building tests
+option(BYTEPACK_BUILD_TESTS "Build the BytePack tests" OFF)
+if(BYTEPACK_BUILD_TESTS)
+    message("Build the BytePack tests")
+    add_subdirectory(extern/Catch2)
+    enable_testing()
+    add_subdirectory(test)
+endif()