Fix deprecated sub callbacks in test_rclcpp

Signed-off-by: Abrar Rahman Protyasha <[email protected]>
ros2 · Aug 20, 2021 · 1f68dea · 1f68dea
1 parent b21f960
commit 1f68dea
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/test_rclcpp/test/test_intra_process.cpp b/test_rclcpp/test/test_intra_process.cpp
@@ -58,7 +58,7 @@ TEST_F(CLASSNAME(test_intra_process_within_one_node, RMW_IMPLEMENTATION), nomina
   int counter = 0;
   auto callback =
     [&counter](
-    const test_rclcpp::msg::UInt32::SharedPtr msg,
+    const test_rclcpp::msg::UInt32::ConstSharedPtr msg,
     const rclcpp::MessageInfo & message_info
     ) -> void
     {

diff --git a/test_rclcpp/test/test_multithreaded.cpp b/test_rclcpp/test/test_multithreaded.cpp
@@ -294,7 +294,7 @@ static inline void multi_access_publisher(bool intra_process)
   // callback groups?
 
   std::atomic_uint subscription_counter(0);
-  auto sub_callback = [&subscription_counter](const test_rclcpp::msg::UInt32::SharedPtr msg)
+  auto sub_callback = [&subscription_counter](const test_rclcpp::msg::UInt32::ConstSharedPtr msg)
     {
       ++subscription_counter;
       printf("Subscription callback %u\n", subscription_counter.load());

diff --git a/test_rclcpp/test/test_repeated_publisher_subscriber.cpp b/test_rclcpp/test/test_repeated_publisher_subscriber.cpp
@@ -51,7 +51,7 @@ TEST_F(
   auto node = rclcpp::Node::make_shared("test_repeated_publisher_subscriber");
 
   auto callback =
-    [](const test_rclcpp::msg::UInt32::SharedPtr) -> void
+    [](const test_rclcpp::msg::UInt32::ConstSharedPtr) -> void
     {
     };
 

diff --git a/test_rclcpp/test/test_subscription.cpp b/test_rclcpp/test/test_subscription.cpp
@@ -81,7 +81,7 @@ TEST_F(CLASSNAME(test_subscription, RMW_IMPLEMENTATION), subscription_and_spinni
   std::shared_future<void> sub_called_future(sub_called.get_future());
   auto fail_after_timeout = 5s;
   auto callback =
-    [&counter, &sub_called](const test_rclcpp::msg::UInt32::SharedPtr msg) -> void
+    [&counter, &sub_called](const test_rclcpp::msg::UInt32::ConstSharedPtr msg) -> void
     {
       ++counter;
       printf("  callback() %d with message data %u\n", counter, msg->data);

diff --git a/test_rclcpp/test/test_timeout_subscriber.cpp b/test_rclcpp/test/test_timeout_subscriber.cpp
@@ -28,7 +28,7 @@
 # define CLASSNAME(NAME, SUFFIX) NAME
 #endif
 
-void callback(const test_rclcpp::msg::UInt32::SharedPtr /*msg*/)
+void callback(const test_rclcpp::msg::UInt32::ConstSharedPtr /*msg*/)
 {
   throw std::runtime_error("The subscriber received a message but there should be no publisher!");
 }