fix(aluminum): respect parent project test settings in CMakeLists

- Only build aluminumTest when both ALUMINUM_ENABLE_TESTS and
  OpenPFC_BUILD_TESTS are enabled
- Use find_package(Catch2 QUIET) instead of REQUIRED to avoid failure
  when tests are disabled
- Change WARNING to STATUS for missing Catch2 message
- Fixes CI documentation workflow that builds with tests disabled

Author: ahojukka5

apps/aluminumNew/CMakeLists.txt

@@ -16,18 +16,19 @@ endif()
 add_executable(aluminumNew aluminumNew.cpp)
 target_link_libraries(aluminumNew OpenPFC nlohmann_json::nlohmann_json)
 
+# Only build tests if tests are enabled for the parent project
 option(ALUMINUM_ENABLE_TESTS "Enable building Aluminum tests" ON)
 
-if(ALUMINUM_ENABLE_TESTS)
-    find_package(Catch2 REQUIRED)
+if(ALUMINUM_ENABLE_TESTS AND OpenPFC_BUILD_TESTS)
+    find_package(Catch2 QUIET)
     if(TARGET Catch2::Catch2WithMain)
         add_executable(aluminumTest aluminumTest.cpp)
         target_link_libraries(aluminumTest OpenPFC nlohmann_json::nlohmann_json Catch2::Catch2WithMain)
         include(CTest)
         include(Catch)
         catch_discover_tests(aluminumTest)
     else()
-        message(WARNING "Aluminum: Catch2 not found, not building tests")
+        message(STATUS "Aluminum: Catch2 not found, not building tests")
     endif()
 endif()