COMP: Fix compiler errors on Windows (MSVC)

Signed-off-by: Michael Jackson <mike.jackson@bluequartz.net>

Author: imikejackson

CMakeLists.txt

@@ -4,7 +4,6 @@ cmake_policy(SET CMP0057 NEW) # ``IN_LIST will be interpreted as an operator whe
 cmake_policy(SET CMP0074 NEW) # ``find_package()`` uses ``<PackageName>_ROOT`` variables.
 cmake_policy(SET CMP0077 NEW) # ``option()`` honors normal variables.
 cmake_policy(SET CMP0080 NEW) # ``BundleUtilities`` cannot be included at configure time.
-cmake_policy(SET CMP0167 NEW) # Suppress FindBoost deprecation warning (retained for EbsdLib compatibility)
 
 # The ``FindPythonInterp`` and ``FindPythonLibs`` modules are removed.
 if(CMAKE_VERSION VERSION_GREATER "3.27.0")
@@ -16,6 +15,10 @@ if(CMAKE_VERSION VERSION_GREATER "3.28.0")
   cmake_policy(SET CMP0152 NEW)
 endif()
 
+if(CMAKE_VERSION VERSION_GREATER "3.29.0")
+  cmake_policy(SET CMP0167 NEW) # Suppress FindBoost deprecation warning (retained for EbsdLib compatibility)
+endif()
+
 include(${CMAKE_CURRENT_LIST_DIR}/cmake/Utility.cmake)
 
 include(CMakeDependentOption)

test/CMakeLists.txt

@@ -27,7 +27,7 @@ create_simplnx_plugin_unit_test(PLUGIN_NAME ${PLUGIN_NAME}
 # Realistic-data validation test (ComputeODFTest.cpp's [validation]-tagged
 # test) loads data/real_world_microtexture_data.dream3d through SimplnxCore's
 # ReadDREAM3DFilter, so we need its headers + link.
-target_link_libraries(${PLUGIN_NAME}UnitTest PRIVATE SimplnxCore)
+target_link_libraries(${PLUGIN_NAME}UnitTest PRIVATE SimplnxCore ${PLUGIN_NAME})
 target_include_directories(${PLUGIN_NAME}UnitTest PRIVATE
   ${simplnx_SOURCE_DIR}/src/Plugins/SimplnxCore/src
 )

test/MTRSimTest.cpp

@@ -30,6 +30,7 @@
 #include <filesystem>
 #include <fstream>
 #include <vector>
+#include <numbers>
 
 using namespace nx::core;
 using namespace nx::core::UnitTest;
@@ -202,8 +203,8 @@ TEST_CASE("MTRSim::MTRSimFilter: Execute wires simulation to output arrays", "[M
   REQUIRE(distinct >= 2);
 
   // Euler values finite and within Bunge bounds (interleaved 3/voxel).
-  constexpr float twoPi = 2.0f * static_cast<float>(M_PI);
-  constexpr float pi = static_cast<float>(M_PI);
+  constexpr float twoPi = 2.0f * std::numbers::pi_v<float>;
+
   const auto& eulerStore = eulers.getDataStoreRef();
   for(usize t = 0; t < expectedTuples; ++t)
   {
@@ -216,7 +217,7 @@ TEST_CASE("MTRSim::MTRSimFilter: Execute wires simulation to output arrays", "[M
     REQUIRE(phi1 >= 0.0f);
     REQUIRE(phi1 <= twoPi);
     REQUIRE(Phi >= 0.0f);
-    REQUIRE(Phi <= pi);
+    REQUIRE(Phi <= std::numbers::pi_v<float>);
     REQUIRE(phi2 >= 0.0f);
     REQUIRE(phi2 <= twoPi);
   }