Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve subscription coverage #681

Merged
merged 14 commits into from
Jun 19, 2020
Merged

Improve subscription coverage #681

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
e9f82da
Add bad argument tests for subscription.c
Blast545 Jun 11, 2020
8b00583
Add missing rcl_reset_error()
Blast545 Jun 12, 2020
621d3a3
Add test failing cyclone
Blast545 Jun 12, 2020
525d460
Disable test not working for rmw_cyclone_cpp
Blast545 Jun 12, 2020
86b4547
Add simple bad alloc test
Blast545 Jun 12, 2020
31d6684
Add tests enclave_name
Blast545 Jun 12, 2020
2e0f073
Revert "Add tests enclave_name"
Blast545 Jun 16, 2020
e145923
Move init/fini tests
Blast545 Jun 17, 2020
375dbf2
Add auxiliar testing class
Blast545 Jun 17, 2020
0446918
Reformat into smaller tests
Blast545 Jun 17, 2020
1a99fb0
Add checks ini/fini rmw messages
Blast545 Jun 17, 2020
62e6111
Add issue report for disabled test
Blast545 Jun 17, 2020
fa88846
Add assert for msg init
Blast545 Jun 17, 2020
3eb7cb7
Add bad_allocator test
Blast545 Jun 18, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions rcl/test/rcl/test_subscription.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -638,7 +638,6 @@ TEST_F(CLASSNAME(TestSubscriptionFixture, RMW_IMPLEMENTATION), test_subscription
ret = rcl_subscription_init(&subscription, this->node_ptr, ts, topic, &subscription_options);
EXPECT_EQ(RCL_RET_ALREADY_INIT, ret) << rcl_get_error_string().str;
rcl_reset_error();

EXPECT_EQ(
RCL_RET_INVALID_ARGUMENT,
rcl_take_sequence(&subscription, seq_size + 1, &messages, &message_infos, nullptr));
Expand All @@ -647,10 +646,15 @@ TEST_F(CLASSNAME(TestSubscriptionFixture, RMW_IMPLEMENTATION), test_subscription
RCL_RET_INVALID_ARGUMENT,
rcl_take_sequence(&subscription, seq_size, &messages, &message_infos_short, nullptr));
rcl_reset_error();

// This test fails for rmw_cyclonedds_cpp function rmw_take_sequence
Blast545 marked this conversation as resolved.
Show resolved Hide resolved
/*
EXPECT_EQ(
RCL_RET_SUBSCRIPTION_TAKE_FAILED,
rcl_take_sequence(&subscription, seq_size, &messages, &message_infos, nullptr));
rcl_reset_error();
*/

Blast545 marked this conversation as resolved.
Show resolved Hide resolved
EXPECT_EQ(
RCL_RET_SUBSCRIPTION_TAKE_FAILED, rcl_take(&subscription, &msg, &message_info, nullptr));
rcl_reset_error();
Expand All @@ -663,12 +667,10 @@ TEST_F(CLASSNAME(TestSubscriptionFixture, RMW_IMPLEMENTATION), test_subscription
ASSERT_FALSE(rcl_node_is_valid_except_context(&invalid_node));
EXPECT_EQ(nullptr, rcl_node_get_rmw_handle(&invalid_node));

/*
EXPECT_EQ(RCL_RET_NODE_INVALID, rcl_subscription_fini(&subscription, nullptr));
rcl_reset_error();
EXPECT_EQ(RCL_RET_NODE_INVALID, rcl_subscription_fini(&subscription, &invalid_node));
rcl_reset_error();
*/

ret = rcl_subscription_fini(&subscription, this->node_ptr);
EXPECT_EQ(RCL_RET_OK, ret) << rcl_get_error_string().str;
Expand Down