Merge pull request #2577 from mavlink/pr-wait-camera-list

system_tests: make camera discovery more robust

Author: julianoes

src/system_tests/camera_format_storage.cpp

@@ -55,7 +55,7 @@ TEST(SystemTest, CameraFormatStorage)
 
     auto camera = Camera{system};
 
-    std::this_thread::sleep_for(std::chrono::milliseconds(500));
+    std::this_thread::sleep_for(std::chrono::milliseconds(1500));
     // We expect to find one camera.
     ASSERT_EQ(camera.camera_list().cameras.size(), 1);
 

src/system_tests/camera_list_cameras.cpp

@@ -56,7 +56,7 @@ TEST(SystemTest, CameraListCameras)
     });
 
     // We have to wait until camera is found.
-    std::this_thread::sleep_for(std::chrono::milliseconds(1000));
+    std::this_thread::sleep_for(std::chrono::milliseconds(1500));
 
     EXPECT_EQ(camera.camera_list().cameras.size(), 1);
     EXPECT_TRUE(found_camera);

src/system_tests/camera_mode.cpp

@@ -64,7 +64,7 @@ TEST(SystemTest, CameraMode)
 
     auto camera = Camera{system};
 
-    std::this_thread::sleep_for(std::chrono::milliseconds(500));
+    std::this_thread::sleep_for(std::chrono::milliseconds(1500));
     // We expect to find one camera.
     ASSERT_EQ(camera.camera_list().cameras.size(), 1);
 

src/system_tests/camera_storage.cpp

@@ -54,7 +54,7 @@ TEST(SystemTest, CameraStorage)
 
     auto camera = Camera{system};
 
-    std::this_thread::sleep_for(std::chrono::milliseconds(500));
+    std::this_thread::sleep_for(std::chrono::milliseconds(1500));
     EXPECT_EQ(camera.camera_list().cameras.size(), 1);
 
     std::this_thread::sleep_for(std::chrono::milliseconds(500));
@@ -77,4 +77,4 @@ TEST(SystemTest, CameraStorage)
         got_storage = true;
     }
     EXPECT_TRUE(got_storage);
-}
+}

src/system_tests/camera_take_photo.cpp

@@ -63,7 +63,7 @@ TEST(SystemTest, CameraTakePhoto)
 
     auto camera = Camera{system};
 
-    std::this_thread::sleep_for(std::chrono::milliseconds(500));
+    std::this_thread::sleep_for(std::chrono::milliseconds(1500));
     // We expect to find one camera.
     ASSERT_EQ(camera.camera_list().cameras.size(), 1);
 

src/system_tests/camera_take_photo_interval.cpp

@@ -59,7 +59,7 @@ TEST(SystemTest, CameraTakePhotoInterval)
 
     auto camera = Camera{system};
 
-    std::this_thread::sleep_for(std::chrono::milliseconds(500));
+    std::this_thread::sleep_for(std::chrono::milliseconds(1500));
     // We expect to find one camera.
     ASSERT_EQ(camera.camera_list().cameras.size(), 1);
 

src/system_tests/camera_video_stream.cpp

@@ -56,7 +56,7 @@ TEST(SystemTest, CameraVideoStreamSettings)
 
     auto camera = Camera{system};
 
-    std::this_thread::sleep_for(std::chrono::milliseconds(500));
+    std::this_thread::sleep_for(std::chrono::milliseconds(1500));
     auto prom_got_info = std::promise<void>();
     auto fut_got_info = prom_got_info.get_future();
     camera.subscribe_video_stream_info([&](const Camera::VideoStreamUpdate& update) {