Skip to content

Commit 1ae0b91

Browse files
committed
Modify based on reviews
Signed-off-by: Ada-King <[email protected]>
1 parent 3f34e28 commit 1ae0b91

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

rcl/test/rcl/test_node.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -750,9 +750,9 @@ TEST_F(CLASSNAME(TestNodeFixture, RMW_IMPLEMENTATION), test_rcl_node_options) {
750750

751751
EXPECT_TRUE(default_options.use_global_arguments);
752752
EXPECT_TRUE(default_options.enable_rosout);
753+
EXPECT_EQ(rcl_qos_profile_rosout_default, default_options.rosout_qos);
753754
EXPECT_TRUE(rcutils_allocator_is_valid(&(default_options.allocator)));
754755

755-
EXPECT_EQ(rcl_qos_profile_rosout_default, default_options.rosout_qos);
756756
EXPECT_EQ(RCL_RET_INVALID_ARGUMENT, rcl_node_options_copy(nullptr, &default_options));
757757
EXPECT_EQ(RCL_RET_INVALID_ARGUMENT, rcl_node_options_copy(&default_options, nullptr));
758758
EXPECT_EQ(RCL_RET_INVALID_ARGUMENT, rcl_node_options_copy(&default_options, &default_options));
@@ -765,11 +765,11 @@ TEST_F(CLASSNAME(TestNodeFixture, RMW_IMPLEMENTATION), test_rcl_node_options) {
765765
rcl_parse_arguments(argc, argv, default_options.allocator, &(default_options.arguments)));
766766
default_options.use_global_arguments = false;
767767
default_options.enable_rosout = false;
768-
default_options.rosout_qos = rcl_qos_profile_rosout_default;
768+
default_options.rosout_qos = rmw_qos_profile_default;
769769
EXPECT_EQ(RCL_RET_OK, rcl_node_options_copy(&default_options, &not_ini_options));
770770
EXPECT_FALSE(not_ini_options.use_global_arguments);
771771
EXPECT_FALSE(not_ini_options.enable_rosout);
772-
EXPECT_EQ(rcl_qos_profile_rosout_default, not_ini_options.rosout_qos);
772+
EXPECT_EQ(default_options.rosout_qos, not_ini_options.rosout_qos);
773773
EXPECT_EQ(
774774
rcl_arguments_get_count_unparsed(&(default_options.arguments)),
775775
rcl_arguments_get_count_unparsed(&(not_ini_options.arguments)));

0 commit comments

Comments
 (0)