Skip to content

Fix #381 unstable play_end_to_end test #396

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

Merged
mabelzhang merged 14 commits into from
May 12, 2020
Merged

Fix #381 unstable play_end_to_end test #396

Changes from 7 commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
471c582
decrease subscription expected msgs
mabelzhang Apr 30, 2020
4ea33bc
change workflow to test this branch
mabelzhang Apr 30, 2020
5e8b85d
change expected number too
mabelzhang Apr 30, 2020
1de1daa
linter
mabelzhang Apr 30, 2020
2aff04b
change workflow to autotest feature branch
mabelzhang Apr 30, 2020
668a988
revert workflow
mabelzhang Apr 30, 2020
93caf12
run workflow more frequently to test stability
mabelzhang Apr 30, 2020
5db6558
revert github workflow
mabelzhang Apr 30, 2020
cd63428
cleanup
mabelzhang Apr 30, 2020
f78c2e3
define constants
mabelzhang Apr 30, 2020
9f7e783
Merge remote-tracking branch 'origin/master' into fix_unstable_play_test
mabelzhang May 7, 2020
d3f4383
Merge remote-tracking branch 'upstream/master' into fix_unstable_play…
mabelzhang May 7, 2020
8412d8a
disable test on windows
mabelzhang May 8, 2020
08972e2
disable play_filters_by_topic on windows
mabelzhang May 8, 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
18 changes: 10 additions & 8 deletions rosbag2_tests/test/rosbag2_tests/test_rosbag2_play_end_to_end.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,9 @@ TEST_F(PlayEndToEndTestFixture, play_fails_gracefully_if_needed_coverter_plugin_

TEST_F(PlayEndToEndTestFixture, play_filters_by_topic) {
// Play a specific topic
sub_->add_subscription<test_msgs::msg::BasicTypes>("/test_topic", 3);
// Due to a problem related to the subscriber, we play many (3) messages but make the subscriber
// node spin only until 2 have arrived. Hence the 2 as `launch_subscriber()` argument.
sub_->add_subscription<test_msgs::msg::BasicTypes>("/test_topic", 2);
sub_->add_subscription<test_msgs::msg::Arrays>("/array_topic", 0);

auto subscription_future = sub_->spin_subscriptions();
Expand All @@ -136,13 +138,13 @@ TEST_F(PlayEndToEndTestFixture, play_filters_by_topic) {

EXPECT_THAT(exit_code, Eq(EXIT_SUCCESS));

EXPECT_THAT(primitive_messages, SizeIs(Ge(3u)));
EXPECT_THAT(primitive_messages, SizeIs(Ge(2u)));
EXPECT_THAT(array_messages, SizeIs(Ge(0u)));

// Play a different topic
sub_ = std::make_unique<SubscriptionManager>();
sub_->add_subscription<test_msgs::msg::BasicTypes>("/test_topic", 0);
sub_->add_subscription<test_msgs::msg::Arrays>("/array_topic", 2);
sub_->add_subscription<test_msgs::msg::Arrays>("/array_topic", 1);

subscription_future = sub_->spin_subscriptions();

Expand All @@ -158,12 +160,12 @@ TEST_F(PlayEndToEndTestFixture, play_filters_by_topic) {
EXPECT_THAT(exit_code, Eq(EXIT_SUCCESS));

EXPECT_THAT(primitive_messages, SizeIs(Ge(0u)));
EXPECT_THAT(array_messages, SizeIs(Ge(2u)));
EXPECT_THAT(array_messages, SizeIs(Ge(1u)));

// Play all topics
sub_ = std::make_unique<SubscriptionManager>();
sub_->add_subscription<test_msgs::msg::BasicTypes>("/test_topic", 3);
sub_->add_subscription<test_msgs::msg::Arrays>("/array_topic", 2);
sub_->add_subscription<test_msgs::msg::BasicTypes>("/test_topic", 2);
sub_->add_subscription<test_msgs::msg::Arrays>("/array_topic", 1);

subscription_future = sub_->spin_subscriptions();

Expand All @@ -178,8 +180,8 @@ TEST_F(PlayEndToEndTestFixture, play_filters_by_topic) {

EXPECT_THAT(exit_code, Eq(EXIT_SUCCESS));

EXPECT_THAT(primitive_messages, SizeIs(Ge(3u)));
EXPECT_THAT(array_messages, SizeIs(Ge(2u)));
EXPECT_THAT(primitive_messages, SizeIs(Ge(2u)));
EXPECT_THAT(array_messages, SizeIs(Ge(1u)));

// Play a non-existent topic
sub_ = std::make_unique<SubscriptionManager>();
Expand Down