Consistently add <cmath> include with define for windows (#3061) (#3065)

Author: mergify[bot]

controller_manager/test/test_spawner_unspawner.cpp

@@ -14,6 +14,10 @@
 
 #include <gtest/gtest.h>
 
+#ifndef _USE_MATH_DEFINES
+#define _USE_MATH_DEFINES
+#endif
+#include <cmath>
 #include <cstdlib>
 #include <memory>
 #include <string>

hardware_interface/test/test_component_parser.cpp

@@ -13,17 +13,17 @@
 // limitations under the License.
 
 #include <gmock/gmock.h>
+#ifndef _USE_MATH_DEFINES
+#define _USE_MATH_DEFINES
+#endif
+#include <cmath>
 #include <string>
 
 #include "hardware_interface/component_parser.hpp"
 #include "hardware_interface/types/hardware_interface_type_values.hpp"
 #include "ros2_control_test_assets/components_urdfs.hpp"
 #include "ros2_control_test_assets/descriptions.hpp"
 
-#ifdef _WIN32
-#define M_PI 3.1415926535897932384626433832795
-#endif
-
 using namespace ::testing;  // NOLINT
 
 class TestComponentParser : public Test

hardware_interface_testing/test/test_resource_manager.cpp

@@ -14,9 +14,13 @@
 
 // Authors: Karsten Knese, Denis Stogl
 
+#ifndef _USE_MATH_DEFINES
+#define _USE_MATH_DEFINES
+#endif
 #include "test_resource_manager.hpp"
 
 #include <algorithm>
+#include <cmath>
 #include <memory>
 #include <string>
 #include <unordered_map>