Make flaky VisualAt test more verbose (#174)

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

test/integration/camera.cc

@@ -227,30 +227,44 @@ void CameraTest::VisualAt(const std::string &_renderEngine)
   camera->SetHFOV(IGN_PI / 2);
   root->AddChild(camera);
 
-  // render a frame
-  camera->Update();
+  // render a few frames
+  for (auto i = 0; i < 30; ++i)
+  {
+    camera->Update();
+  }
+
+  EXPECT_EQ(800u, camera->ImageWidth());
+  EXPECT_EQ(600u, camera->ImageHeight());
 
   for (auto x = 0u; x < camera->ImageWidth(); x = x + 100)
   {
     auto vis = camera->VisualAt(math::Vector2i(x, camera->ImageHeight() / 2));
 
     if (x <= 100)
     {
-     EXPECT_EQ(nullptr, vis);
+      EXPECT_EQ(nullptr, vis);
     }
     else if (x > 100 && x <= 300)
     {
-      ASSERT_NE(nullptr, vis);
-      EXPECT_EQ("sphere", vis->Name());
+      // Don't end test here on failure, this condition is flaky
+      EXPECT_NE(nullptr, vis) << x;
+      if (vis)
+      {
+        EXPECT_EQ("sphere", vis->Name());
+      }
     }
     else if (x > 300 && x <= 400)
     {
       EXPECT_EQ(nullptr, vis);
     }
     else if (x > 400 && x <= 700)
     {
-      ASSERT_NE(nullptr, vis);
-      EXPECT_EQ("box", vis->Name());
+      // Don't end test here on failure, this condition is flaky
+      EXPECT_NE(nullptr, vis) << x;
+      if (vis)
+      {
+        EXPECT_EQ("box", vis->Name());
+      }
     }
     else
     {