Skip to content

Commit 241f75f

Browse files
committed
system_tests: deal with slow test runners
1 parent 5e19124 commit 241f75f

File tree

1 file changed

+19
-4
lines changed

1 file changed

+19
-4
lines changed

src/system_tests/camera_settings.cpp

Lines changed: 19 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -76,8 +76,15 @@ TEST(SystemTest, CameraSettings)
7676

7777
auto camera = Camera{system};
7878

79-
std::this_thread::sleep_for(std::chrono::milliseconds(500));
80-
EXPECT_EQ(camera.camera_list().cameras.size(), 1);
79+
// Wait for camera list to be populated before proceeding
80+
for (unsigned i = 0; i < 20; ++i) {
81+
std::this_thread::sleep_for(std::chrono::milliseconds(100));
82+
if (camera.camera_list().cameras.size() > 0) {
83+
break;
84+
}
85+
}
86+
87+
ASSERT_EQ(camera.camera_list().cameras.size(), 1);
8188

8289
std::pair<Camera::Result, std::vector<Camera::SettingOptions>> possible_setting_options;
8390

@@ -229,8 +236,16 @@ TEST(SystemTest, CameraSettingsAsync)
229236
auto system = fut.get();
230237

231238
auto camera = Camera{system};
232-
std::this_thread::sleep_for(std::chrono::milliseconds(500));
233-
EXPECT_EQ(camera.camera_list().cameras.size(), 1);
239+
240+
// Wait for camera list to be populated before proceeding
241+
for (unsigned i = 0; i < 20; ++i) {
242+
std::this_thread::sleep_for(std::chrono::milliseconds(100));
243+
if (camera.camera_list().cameras.size() > 0) {
244+
break;
245+
}
246+
}
247+
248+
ASSERT_EQ(camera.camera_list().cameras.size(), 1);
234249

235250
bool found_wb_temp = false;
236251
camera.subscribe_current_settings([&](const Camera::CurrentSettingsUpdate& update) {

0 commit comments

Comments
 (0)