Skip to content

Commit

Permalink
Fix lint error in test_subscription.cpp (#469)
Browse files Browse the repository at this point in the history
Co-Authored-By: Jacob Perron <jacob@openrobotics.org>

Signed-off-by: y-okumura-isp <y-okumura@isp.co.jp>
  • Loading branch information
y-okumura-isp committed Feb 10, 2020
1 parent 3173e87 commit 65f40ce
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions rcl/test/rcl/test_subscription.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,8 @@ wait_for_subscription_to_be_ready(

/* Test subscription init, fini and is_valid functions
*/
TEST_F(CLASSNAME(TestSubscriptionFixture, RMW_IMPLEMENTATION), test_subscription_init_fini_and_is_valid) {
TEST_F(CLASSNAME(TestSubscriptionFixture, RMW_IMPLEMENTATION),
test_subscription_init_fini_and_is_valid) {
rcl_ret_t ret;

const rosidl_message_type_support_t * ts =
Expand Down Expand Up @@ -162,7 +163,7 @@ TEST_F(CLASSNAME(TestSubscriptionFixture, RMW_IMPLEMENTATION), test_subscription
});

rcl_subscription_options_t subscription_options = rcl_subscription_get_default_options();
rcl_subscription_t subscription = rcl_get_zero_initialized_subscription();
rcl_subscription_t subscription = rcl_get_zero_initialized_subscription();
ret = rcl_subscription_init(&subscription, this->node_ptr, ts, topic, &subscription_options);
ASSERT_EQ(RCL_RET_OK, ret) << rcl_get_error_string().str;
OSRF_TESTING_TOOLS_CPP_SCOPE_EXIT(
Expand Down Expand Up @@ -192,7 +193,7 @@ TEST_F(CLASSNAME(TestSubscriptionFixture, RMW_IMPLEMENTATION), test_subscription
test_msgs__msg__BasicTypes__init(&msg);
OSRF_TESTING_TOOLS_CPP_SCOPE_EXIT(
{
test_msgs__msg__BasicTypes__fini(&msg);
test_msgs__msg__BasicTypes__fini(&msg);
});
ret = rcl_take(&subscription, &msg, nullptr, nullptr);
ASSERT_EQ(RCL_RET_OK, ret) << rcl_get_error_string().str;
Expand Down

0 comments on commit 65f40ce

Please sign in to comment.