Fix CMake test flags: always apply MSVC /Zm500 heap flag

Separate MSVC compiler heap size flags from warning flags. The /Zm500 flag
now always applies to MSVC test builds (not tied to WARNINGS_AS_ERRORS),
ensuring C1060 'compiler out of heap space' is prevented in all builds,
including CI.

Previously, test flags were set but never applied to targets. Now they are
applied via target_compile_options() to each test executable.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

Author: jll63

test/CMakeLists.txt

@@ -20,24 +20,29 @@ if (NOT TARGET tests)
     set_property(TARGET tests PROPERTY FOLDER Dependencies)
 endif()
 
-# Replicate error flags from Jamfile
+# MSVC compiler heap size (always applied to avoid C1060 "out of heap space")
+if (CMAKE_CXX_COMPILER_ID STREQUAL "MSVC" OR CMAKE_CXX_COMPILER_FRONTEND_VARIANT MATCHES "MSVC")
+    set(BOOST_OPENMETHOD_MSVC_HEAP_FLAGS "/Zm500")
+endif()
+
+# Warning flags (applied only when WARNINGS_AS_ERRORS is enabled)
 if (BOOST_OPENMETHOD_WARNINGS_AS_ERRORS)
     if (CMAKE_CXX_COMPILER_ID MATCHES "GNU")
         if (CMAKE_CXX_COMPILER_VERSION VERSION_GREATER_EQUAL 7)
-            set(BOOST_OPENMETHOD_TEST_FLAGS "-Wall -Werror -Wno-unused-but-set-variable -Wno-maybe-uninitialized")
+            set(BOOST_OPENMETHOD_WARNING_FLAGS "-Wall -Werror -Wno-unused-but-set-variable -Wno-maybe-uninitialized")
         else()
-            set(BOOST_OPENMETHOD_TEST_FLAGS "-Wall -Werror -Wno-unused-but-set-variable")
+            set(BOOST_OPENMETHOD_WARNING_FLAGS "-Wall -Werror -Wno-unused-but-set-variable")
         endif()
     elseif (CMAKE_CXX_COMPILER_ID MATCHES "AppleClang")
-        set(BOOST_OPENMETHOD_TEST_FLAGS "-Wall -Werror")
+        set(BOOST_OPENMETHOD_WARNING_FLAGS "-Wall -Werror")
     elseif (CMAKE_CXX_COMPILER_ID MATCHES "Clang" AND NOT CMAKE_CXX_COMPILER_FRONTEND_VARIANT MATCHES "MSVC")
         if (CMAKE_CXX_COMPILER_VERSION VERSION_GREATER_EQUAL 13)
-            set(BOOST_OPENMETHOD_TEST_FLAGS "-Wall -Werror -Wno-unused-but-set-variable")
+            set(BOOST_OPENMETHOD_WARNING_FLAGS "-Wall -Werror -Wno-unused-but-set-variable")
         else()
-            set(BOOST_OPENMETHOD_TEST_FLAGS "-Wall -Werror")
+            set(BOOST_OPENMETHOD_WARNING_FLAGS "-Wall -Werror")
         endif()
     elseif (CMAKE_CXX_COMPILER_ID STREQUAL "MSVC" OR CMAKE_CXX_COMPILER_FRONTEND_VARIANT MATCHES "MSVC")
-        set(BOOST_OPENMETHOD_TEST_FLAGS "/W4 /WX /we4265 /wd4251 /Zm500")
+        set(BOOST_OPENMETHOD_WARNING_FLAGS "/W4 /WX /we4265 /wd4251")
     endif()
 
     # Print test configuration if running in CI
@@ -49,7 +54,8 @@ if (BOOST_OPENMETHOD_WARNINGS_AS_ERRORS)
         endif()
         message(STATUS "Boost.OpenMethod Tests - Platform: ${CMAKE_SYSTEM_NAME} / ${CMAKE_SYSTEM_VERSION}")
         message(STATUS "Boost.OpenMethod Tests - C++ standard: ${CMAKE_CXX_STANDARD}")
-        message(STATUS "Boost.OpenMethod Tests - Test error flags: ${BOOST_OPENMETHOD_TEST_FLAGS}")
+        message(STATUS "Boost.OpenMethod Tests - Test warning flags: ${BOOST_OPENMETHOD_WARNING_FLAGS}")
+        message(STATUS "Boost.OpenMethod Tests - Test heap flags: ${BOOST_OPENMETHOD_MSVC_HEAP_FLAGS}")
     endif()
 endif()
 
@@ -70,12 +76,28 @@ foreach(test_cpp ${test_cpp_files})
     set(test_target "boost_openmethod-${test}")
     add_executable(${test_target} EXCLUDE_FROM_ALL ${test_cpp})
     target_link_libraries(${test_target} PRIVATE Boost::openmethod Boost::unit_test_framework)
+    # Always apply MSVC heap flags to avoid C1060 "out of heap space"
+    if (BOOST_OPENMETHOD_MSVC_HEAP_FLAGS)
+        target_compile_options(${test_target} PRIVATE ${BOOST_OPENMETHOD_MSVC_HEAP_FLAGS})
+    endif()
+    # Apply warning flags only if WARNINGS_AS_ERRORS is enabled
+    if (BOOST_OPENMETHOD_WARNING_FLAGS)
+        target_compile_options(${test_target} PRIVATE ${BOOST_OPENMETHOD_WARNING_FLAGS})
+    endif()
     boost_openmethod_add_test(${test_target})
     add_dependencies(tests ${test_target})
 endforeach()
 
 add_executable(boost_openmethod-test_mix_release_debug EXCLUDE_FROM_ALL mix_release_debug/main.cpp mix_release_debug/lib.cpp)
 target_link_libraries(boost_openmethod-test_mix_release_debug PRIVATE Boost::openmethod Boost::unit_test_framework)
+# Always apply MSVC heap flags
+if (BOOST_OPENMETHOD_MSVC_HEAP_FLAGS)
+    target_compile_options(boost_openmethod-test_mix_release_debug PRIVATE ${BOOST_OPENMETHOD_MSVC_HEAP_FLAGS})
+endif()
+# Apply warning flags only if WARNINGS_AS_ERRORS is enabled
+if (BOOST_OPENMETHOD_WARNING_FLAGS)
+    target_compile_options(boost_openmethod-test_mix_release_debug PRIVATE ${BOOST_OPENMETHOD_WARNING_FLAGS})
+endif()
 boost_openmethod_add_test(boost_openmethod-test_mix_release_debug)
 add_dependencies(tests boost_openmethod-test_mix_release_debug)