Refactor CMakeLists.txt to remove include_directories for Catch2 and streamline include handling

Author: ahojukka5

tests/CMakeLists.txt

@@ -4,7 +4,6 @@
 # https://github.com/catchorg/Catch2/blob/devel/docs/cmake-integration.md#top
 
 find_package(Catch2 REQUIRED)
-include_directories(${Catch2_INCLUDE_DIRS})  # Add Catch2 include directories
 
 add_executable(openpfc-tests
                runtests.cpp
@@ -16,16 +15,28 @@ add_executable(openpfc-tests
                test_discrete_field.cpp
                test_field_modifier.cpp
                test_ic_constant.cpp
+               test_boundary_conditions.cpp
                test_model.cpp
                test_multi_index.cpp
                test_simulator.cpp
                test_time.cpp)
 
+# Link against OpenPFC and Catch2
 target_link_libraries(openpfc-tests PRIVATE OpenPFC Catch2::Catch2)
 
+# Attach Catch2 include dirs manually for compile_commands.json
+get_target_property(_catch2_include_dirs Catch2::Catch2 INTERFACE_INCLUDE_DIRECTORIES)
+message(STATUS "Catch2 includes: ${_catch2_include_dirs}")
+target_include_directories(openpfc-tests PRIVATE ${_catch2_include_dirs})
+foreach(dir IN LISTS _catch2_include_dirs)
+  target_compile_options(openpfc-tests PRIVATE "-isystem" "${dir}")
+endforeach()
+
 if(OpenPFC_ENABLE_CODE_COVERAGE)
-  target_compile_options(openpfc-tests PUBLIC --coverage)
-  target_link_options(openpfc-tests PUBLIC --coverage)
+  # Enable code coverage if OpenPFC_ENABLE_CODE_COVERAGE is set
+  message(STATUS "Enabling code coverage for tests")
+  target_compile_options(openpfc-tests PRIVATE --coverage)
+  target_link_options(openpfc-tests PRIVATE --coverage)
 endif()
 
 # Disable automatic test discovery during the build