fix: guard tests with BUILD_TESTING, remove doctest from cross-build deps

Co-authored-by: gfauredev <19304085+gfauredev@users.noreply.github.com>

Author: Copilot

CMakeLists.txt

@@ -58,6 +58,8 @@ add_custom_target(
   DEPENDS merge_coverage_data
   COMMENT "Rapport de couverture HTML généré: ${CMAKE_BINARY_DIR}/coverage")
 add_dependencies(run main)
-add_dependencies(tests LoggerTest)
-add_dependencies(tests integrationTest)
+if(BUILD_TESTING)
+  add_dependencies(tests LoggerTest)
+  add_dependencies(tests integrationTest)
+endif()
 add_dependencies(coverage merge_coverage_data)

flake.nix

@@ -65,6 +65,7 @@
                     clang-tools # Clang CLIs, including LSP
                     cmake-format # CMake formatter
                     cmake-language-server # Cmake LSP
+                    doctest # Testing framework (for local -DBUILD_TESTING=ON builds)
                     doxygen # Documentation generator
                     lldb # Clang debug adapter
                   ]

test/CMakeLists.txt

@@ -1,12 +1,14 @@
-find_package(doctest REQUIRED)
-include(CTest)
+if(BUILD_TESTING)
+  find_package(doctest REQUIRED)
+  include(CTest)
 
-add_executable(LoggerTest LoggerTest.cpp)
-target_include_directories(LoggerTest PRIVATE ${CMAKE_SOURCE_DIR}/include)
-target_link_libraries(LoggerTest PRIVATE doctest::doctest)
-add_test(NAME LoggerTest COMMAND LoggerTest)
+  add_executable(LoggerTest LoggerTest.cpp)
+  target_include_directories(LoggerTest PRIVATE ${CMAKE_SOURCE_DIR}/include)
+  target_link_libraries(LoggerTest PRIVATE doctest::doctest)
+  add_test(NAME LoggerTest COMMAND LoggerTest)
 
-add_executable(integrationTest integrationTest.cpp)
-target_include_directories(integrationTest PRIVATE ${CMAKE_SOURCE_DIR}/include)
-target_link_libraries(integrationTest PRIVATE doctest::doctest)
-add_test(NAME integrationTest COMMAND integrationTest)
+  add_executable(integrationTest integrationTest.cpp)
+  target_include_directories(integrationTest PRIVATE ${CMAKE_SOURCE_DIR}/include)
+  target_link_libraries(integrationTest PRIVATE doctest::doctest)
+  add_test(NAME integrationTest COMMAND integrationTest)
+endif()

ui.nix

@@ -1,7 +1,6 @@
 {
   cmake,
   clang,
-  doctest,
   engine,
   glfw,
   llvm,
@@ -32,7 +31,6 @@ stdenv.mkDerivation {
     pkg-config # Build tool
   ];
   buildInputs = [
-    doctest # Testing framework
     engine # SmartPianoEngine
     glfw # Raylib dependency
     libGL # GPU library