fix(enumAvailableDevices): exception when enumerating

Author: FrogTheFrog

src/windows/win_display_device_general.cpp

@@ -42,7 +42,8 @@ namespace display_device {
     }
 
     for (const auto &[device_id, data] : source_data) {
-      const auto source_id_index {data.m_active_source.value_or(0)};
+      // In case we have no active source, we will take the first available source id
+      const auto source_id_index {data.m_active_source.value_or(data.m_source_id_to_path_index.begin()->first)};
       const auto &best_path {display_data->m_paths.at(data.m_source_id_to_path_index.at(source_id_index))};
       const auto friendly_name {m_w_api->getFriendlyName(best_path)};
       const bool is_active {win_utils::isActive(best_path)};

tests/unit/windows/utils/mock_win_api_layer.cpp

@@ -11,7 +11,7 @@ namespace {
       data.m_paths.back().flags = DISPLAYCONFIG_PATH_ACTIVE;
       data.m_paths.back().sourceInfo.sourceModeInfoIdx = data.m_modes.size();
       data.m_paths.back().sourceInfo.adapterId = {1, 1};
-      data.m_paths.back().sourceInfo.id = 0;
+      data.m_paths.back().sourceInfo.id = 1;
       data.m_paths.back().targetInfo.targetAvailable = TRUE;
       data.m_paths.back().targetInfo.refreshRate = {120, 1};
 
@@ -29,7 +29,7 @@ namespace {
       data.m_paths.back().flags = DISPLAYCONFIG_PATH_ACTIVE;
       data.m_paths.back().sourceInfo.sourceModeInfoIdx = data.m_modes.size();
       data.m_paths.back().sourceInfo.adapterId = {2, 2};
-      data.m_paths.back().sourceInfo.id = 0;
+      data.m_paths.back().sourceInfo.id = 2;
       data.m_paths.back().targetInfo.targetAvailable = TRUE;
       data.m_paths.back().targetInfo.refreshRate = {119995, 1000};
 
@@ -45,7 +45,7 @@ namespace {
         data.m_paths.back().flags = DISPLAYCONFIG_PATH_ACTIVE;
         data.m_paths.back().sourceInfo.sourceModeInfoIdx = data.m_modes.size();
         data.m_paths.back().sourceInfo.adapterId = {3, 3};
-        data.m_paths.back().sourceInfo.id = 0;
+        data.m_paths.back().sourceInfo.id = 3;
         data.m_paths.back().targetInfo.targetAvailable = TRUE;
         data.m_paths.back().targetInfo.refreshRate = {60, 1};
 
@@ -64,7 +64,7 @@ namespace {
       data.m_paths.back().flags = DISPLAYCONFIG_PATH_ACTIVE;
       data.m_paths.back().sourceInfo.sourceModeInfoIdx = data.m_modes.size();
       data.m_paths.back().sourceInfo.adapterId = {4, 4};
-      data.m_paths.back().sourceInfo.id = 0;
+      data.m_paths.back().sourceInfo.id = 4;
       data.m_paths.back().targetInfo.targetAvailable = TRUE;
       data.m_paths.back().targetInfo.refreshRate = {90, 1};