👩‍🌾 Make depth camera tests more robust (#897) (#1257)

Signed-off-by: Louise Poubel <[email protected]>

Author: chapulina

test/integration/depth_camera.cc

@@ -35,6 +35,7 @@
 
 using namespace ignition;
 using namespace gazebo;
+using namespace std::chrono_literals;
 
 /// \brief Test DepthCameraTest system
 class DepthCameraTest : public InternalFixture<::testing::Test>
@@ -81,14 +82,15 @@ TEST_F(DepthCameraTest, IGN_UTILS_TEST_DISABLED_ON_MAC(DepthCameraBox))
   size_t iters100 = 100u;
   server.Run(true, iters100, false);
 
-  ignition::common::Time waitTime = ignition::common::Time(0.001);
-  int i = 0;
-  while (i < 300)
+  int sleep{0};
+  int maxSleep{30};
+  while (depthBuffer == nullptr && sleep < maxSleep)
   {
-    ignition::common::Time::Sleep(waitTime);
-    i++;
+    std::this_thread::sleep_for(100ms);
+    sleep++;
   }
-  EXPECT_NE(depthBuffer, nullptr);
+  EXPECT_LT(sleep, maxSleep);
+  ASSERT_NE(depthBuffer, nullptr);
 
   // Take into account box of 1 m on each side and 0.05 cm sensor offset
   double expectedRangeAtMidPointBox1 = 2.45;

test/integration/pose_publisher_system.cc

@@ -339,9 +339,9 @@ TEST_F(PosePublisherTest, UpdateFrequency)
   std::size_t nExpMessages = 100;
   // Wait for 100 messages to be received
   bool received = false;
-  for (int sleep = 0; sleep < 300; ++sleep)
+  for (int sleep = 0; sleep < 30; ++sleep)
   {
-    std::this_thread::sleep_for(std::chrono::milliseconds(10));
+    std::this_thread::sleep_for(std::chrono::milliseconds(100));
 
     {
       std::lock_guard<std::mutex> lock(mutex);
@@ -657,9 +657,9 @@ TEST_F(PosePublisherTest, StaticPoseUpdateFrequency)
   std::size_t nExpMessages = 100;
   // Wait for 100 messages to be received
   bool received = false;
-  for (int sleep = 0; sleep < 300; ++sleep)
+  for (int sleep = 0; sleep < 30; ++sleep)
   {
-    std::this_thread::sleep_for(std::chrono::milliseconds(10));
+    std::this_thread::sleep_for(std::chrono::milliseconds(100));
 
     {
       std::lock_guard<std::mutex> lock(mutex);
@@ -717,9 +717,9 @@ TEST_F(PosePublisherTest, NestedModelLoadPlugin)
 
   // Wait for messages to be received
   int sleep = 0;
-  while (poseMsgs.empty() && sleep++ < 300)
+  while (poseMsgs.empty() && sleep++ < 30)
   {
-    std::this_thread::sleep_for(std::chrono::milliseconds(10));
+    std::this_thread::sleep_for(std::chrono::milliseconds(100));
   }
 
   EXPECT_TRUE(!poseMsgs.empty());

test/integration/rgbd_camera.cc

@@ -35,6 +35,7 @@
 
 using namespace ignition;
 using namespace gazebo;
+using namespace std::chrono_literals;
 
 /// \brief Test RgbdCameraTest system
 class RgbdCameraTest : public InternalFixture<::testing::Test>
@@ -81,13 +82,14 @@ TEST_F(RgbdCameraTest, IGN_UTILS_TEST_DISABLED_ON_MAC(RgbdCameraBox))
   size_t iters100 = 100u;
   server.Run(true, iters100, false);
 
-  ignition::common::Time waitTime = ignition::common::Time(0.001);
-  int i = 0;
-  while (nullptr == depthBuffer && i < 500)
+  int sleep{0};
+  int maxSleep{30};
+  while (depthBuffer == nullptr && sleep < maxSleep)
   {
-    ignition::common::Time::Sleep(waitTime);
-    i++;
+    std::this_thread::sleep_for(100ms);
+    sleep++;
   }
+  EXPECT_LT(sleep, maxSleep);
   ASSERT_NE(depthBuffer, nullptr);
 
   // Take into account box of 1 m on each side and 0.05 cm sensor offset

test/integration/scene_broadcaster_system.cc

@@ -78,7 +78,7 @@ TEST_P(SceneBroadcasterTest, PoseInfo)
   server.Run(true, 1, false);
 
   unsigned int sleep{0u};
-  unsigned int maxSleep{10u};
+  unsigned int maxSleep{30u};
   // cppcheck-suppress unmatchedSuppression
   // cppcheck-suppress knownConditionTrueFalse
   while (!received && sleep++ < maxSleep)
@@ -464,7 +464,7 @@ TEST_P(SceneBroadcasterTest, State)
 
   // Run server
   unsigned int sleep{0u};
-  unsigned int maxSleep{10u};
+  unsigned int maxSleep{30u};
   while (!received && sleep++ < maxSleep)
   {
     IGN_SLEEP_MS(100);
@@ -588,7 +588,7 @@ TEST_P(SceneBroadcasterTest, StateStatic)
 
   // Run server
   unsigned int sleep{0u};
-  unsigned int maxSleep{10u};
+  unsigned int maxSleep{30u};
   while (!received && sleep++ < maxSleep)
   {
     IGN_SLEEP_MS(100);