Skip to content

Commit f79f57f

Browse files
correct test function descriptions (#2970) (#2993)
(cherry picked from commit 354413c) Signed-off-by: Yuchen Liu <lyuchen9696@gmail.com> Co-authored-by: Yuchen966 <70603129+Yuchen966@users.noreply.github.com>
1 parent ed3f937 commit f79f57f

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

rclcpp/test/rclcpp/test_intra_process_manager.cpp

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -517,7 +517,7 @@ TEST(TestIntraProcessManager, add_pub_sub) {
517517
* - Remove the first subscription from ipm and add a new one.
518518
* - Publishes a unique_ptr message with a subscription not requesting ownership.
519519
* - The received message is expected to be the same, the first subscription do not receive it.
520-
* - Publishes a shared_ptr message with a subscription not requesting ownership.
520+
* - Publishes a unique_ptr message with a subscription not requesting ownership.
521521
* - The received message is expected to be the same.
522522
*/
523523
TEST(TestIntraProcessManager, single_subscription) {
@@ -569,9 +569,9 @@ TEST(TestIntraProcessManager, single_subscription) {
569569
* - One is expected to receive the published message, while the other will receive a copy.
570570
* - Publishes a unique_ptr message with 2 subscriptions not requesting ownership.
571571
* - Both received messages are expected to be the same as the published one.
572-
* - Publishes a shared_ptr message with 2 subscriptions requesting ownership.
572+
* - Publishes a unique_ptr message with 2 subscriptions requesting ownership.
573573
* - Both received messages are expected to be a copy of the published one.
574-
* - Publishes a shared_ptr message with 2 subscriptions not requesting ownership.
574+
* - Publishes a unique_ptr message with 2 subscriptions not requesting ownership.
575575
* - Both received messages are expected to be the same as the published one.
576576
*/
577577
TEST(TestIntraProcessManager, multiple_subscriptions_same_type) {
@@ -676,9 +676,9 @@ TEST(TestIntraProcessManager, multiple_subscriptions_same_type) {
676676
* - The 2 subscriptions not requesting ownership are expected to both receive the same copy
677677
* of the message, one of the subscription requesting ownership is expected to receive a
678678
* different copy, while the last is expected to receive the published message.
679-
* - Publishes a shared_ptr message with 1 subscription requesting ownership and 1 not.
680-
* - The subscription requesting ownership is expected to receive a copy of the message, while
681-
* the other is expected to receive the published message
679+
* - Publishes a unique_ptr message with 1 subscription requesting ownership and 1 not.
680+
* - The subscription requesting ownership is expected to receive the published message, while
681+
* the other is expected to receive a copy of the message
682682
*/
683683
TEST(TestIntraProcessManager, multiple_subscriptions_different_type) {
684684
using IntraProcessManagerT = rclcpp::experimental::IntraProcessManager;

0 commit comments

Comments
 (0)