Fixes from review.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
ros2 · Jan 19, 2021 · 9ff8d5b · 9ff8d5b
1 parent 4662418
commit 9ff8d5b
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/rclcpp/test/rclcpp/test_executor.cpp b/rclcpp/test/rclcpp/test_executor.cpp
@@ -122,14 +122,14 @@ TEST_F(TestExecutor, constructor_bad_guard_condition_init) {
   auto mock = mocking_utils::patch_and_return(
     "lib:rclcpp", rcl_guard_condition_init, RCL_RET_ERROR);
   EXPECT_THROW(
-    std::unique_ptr<DummyExecutor>(new DummyExecutor()),
+    static_cast<void>(std::make_unique<DummyExecutor>()),
     rclcpp::exceptions::RCLError);
 }
 
 TEST_F(TestExecutor, constructor_bad_wait_set_init) {
   auto mock = mocking_utils::patch_and_return("lib:rclcpp", rcl_wait_set_init, RCL_RET_ERROR);
   RCLCPP_EXPECT_THROW_EQ(
-    std::unique_ptr<DummyExecutor>(new DummyExecutor()),
+    static_cast<void>(std::make_unique<DummyExecutor>()),
     std::runtime_error("Failed to create wait set in Executor constructor: error not set"));
 }