Skip to content

Commit e57e131

Browse files
committed
Wait for stale subscriber cleanup in the mixed subscribers test
Signed-off-by: Carlos Agüero <caguero@honurobotics.com>
1 parent b53a8da commit e57e131

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

test/integration/twoProcsPubSub.cc

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -586,8 +586,9 @@ TEST(twoProcPubSub, PubSubTwoProcsMixedSubscribers)
586586
auto pub = node.Advertise<msgs::Vector3d>(g_topic);
587587
EXPECT_TRUE(pub);
588588

589-
// No subscribers yet.
590-
EXPECT_FALSE(pub.HasConnections());
589+
// No subscribers yet. Wait in case a remote subscriber from a previous
590+
// test is still being cleaned up.
591+
EXPECT_TRUE(transport::waitUntil([&pub]{return !pub.HasConnections();}));
591592

592593
auto pi = gz::utils::Subprocess(
593594
{test_executables::kTwoProcsPubSubMixedSubscribers, partition});

0 commit comments

Comments
 (0)