Skip to content

Commit b6238de

Browse files
mergify[bot]Yuchen966ahcorde
authored
correct test function descriptions (backport #2970) (#2994)
* correct test function descriptions (#2970) Signed-off-by: Yuchen Liu <[email protected]> (cherry picked from commit 354413c) Signed-off-by: Alejandro Hernandez Cordero <[email protected]> Co-authored-by: Yuchen966 <[email protected]> Co-authored-by: Alejandro Hernandez Cordero <[email protected]>
1 parent 36d0aee commit b6238de

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
@@ -430,7 +430,7 @@ TEST(TestIntraProcessManager, add_pub_sub) {
430430
- Remove the first subscription from ipm and add a new one.
431431
- Publishes a unique_ptr message with a subscription not requesting ownership.
432432
- The received message is expected to be the same, the first subscription do not receive it.
433-
- Publishes a shared_ptr message with a subscription not requesting ownership.
433+
- Publishes a unique_ptr message with a subscription not requesting ownership.
434434
- The received message is expected to be the same.
435435
*/
436436
TEST(TestIntraProcessManager, single_subscription) {
@@ -482,9 +482,9 @@ TEST(TestIntraProcessManager, single_subscription) {
482482
- One is expected to receive the published message, while the other will receive a copy.
483483
- Publishes a unique_ptr message with 2 subscriptions not requesting ownership.
484484
- Both received messages are expected to be the same as the published one.
485-
- Publishes a shared_ptr message with 2 subscriptions requesting ownership.
485+
- Publishes a unique_ptr message with 2 subscriptions requesting ownership.
486486
- Both received messages are expected to be a copy of the published one.
487-
- Publishes a shared_ptr message with 2 subscriptions not requesting ownership.
487+
- Publishes a unique_ptr message with 2 subscriptions not requesting ownership.
488488
- Both received messages are expected to be the same as the published one.
489489
*/
490490
TEST(TestIntraProcessManager, multiple_subscriptions_same_type) {
@@ -589,9 +589,9 @@ TEST(TestIntraProcessManager, multiple_subscriptions_same_type) {
589589
- The 2 subscriptions not requesting ownership are expected to both receive the same copy
590590
of the message, one of the subscription requesting ownership is expected to receive a
591591
different copy, while the last is expected to receive the published message.
592-
- Publishes a shared_ptr message with 1 subscription requesting ownership and 1 not.
593-
- The subscription requesting ownership is expected to receive a copy of the message, while
594-
the other is expected to receive the published message
592+
- Publishes a unique_ptr message with 1 subscription requesting ownership and 1 not.
593+
- The subscription requesting ownership is expected to receive the published message, while
594+
the other is expected to receive a copy of the published message
595595
*/
596596
TEST(TestIntraProcessManager, multiple_subscriptions_different_type) {
597597
using IntraProcessManagerT = rclcpp::experimental::IntraProcessManager;

0 commit comments

Comments
 (0)