Use standard BUILD_TESTING option with defensive check

Changed from BUILD_MOLLERDB_TESTS to BUILD_TESTING with CMAKE_PROJECT_NAME check to match sqlpp23 pattern. This prevents tests from being built when mollerdb is used as a subproject.

Co-authored-by: wdconinc <4656391+wdconinc@users.noreply.github.com>

Author: Copilot

.github/workflows/build.yml

@@ -132,7 +132,7 @@ jobs:
       run: |
         mkdir -p build
         cd build
-        cmake .. -DBUILD_MOLLERDB_TESTS=ON
+        cmake .. -DBUILD_TESTING=ON
         cmake --build . --target test_database
         ctest --output-on-failure -V
       env:

AGENTS.md

@@ -80,7 +80,7 @@ make
 **Building with Tests:**
 ```bash
 mkdir build && cd build
-cmake .. -DBUILD_MOLLERDB_TESTS=ON
+cmake .. -DBUILD_TESTING=ON
 cmake --build .
 ctest --output-on-failure
 ```
@@ -230,7 +230,7 @@ pytest tests/python/ -v
 **C++ Tests (Google Test via CTest):**
 ```bash
 mkdir build && cd build
-cmake .. -DBUILD_MOLLERDB_TESTS=ON
+cmake .. -DBUILD_TESTING=ON
 cmake --build .
 ctest --output-on-failure
 ```

CMakeLists.txt

@@ -68,10 +68,10 @@ set(BUILD_TESTING OFF CACHE BOOL "Build tests" FORCE)
 add_subdirectory(thirdparty/sqlpp23)
 
 # --- Testing ---
-# Enable testing for this project (separate from sqlpp23's BUILD_TESTING)
-option(BUILD_MOLLERDB_TESTS "Build mollerdb tests" ON)
+# Enable testing for this project (only when mollerdb is the top-level project)
+option(BUILD_TESTING "Build mollerdb tests" ON)
 
-if(BUILD_MOLLERDB_TESTS)
+if(CMAKE_PROJECT_NAME STREQUAL PROJECT_NAME AND BUILD_TESTING)
     enable_testing()
 
     # Fetch Google Test

docs/README.md

@@ -239,7 +239,7 @@ The C++ tests use Google Test and are built with CMake:
 ```bash
 # Build with tests enabled
 mkdir build && cd build
-cmake .. -DBUILD_MOLLERDB_TESTS=ON
+cmake .. -DBUILD_TESTING=ON
 cmake --build .
 
 # Run all C++ tests

tests/README.md

@@ -26,7 +26,7 @@ Build and run C++ tests using CMake and CTest:
 ```bash
 # From repository root
 mkdir build && cd build
-cmake .. -DBUILD_MOLLERDB_TESTS=ON
+cmake .. -DBUILD_TESTING=ON
 cmake --build .
 ctest --output-on-failure
 ```