feat: fetch vehicle profiles inside get_vehicles (flag-gated), drop VehicleManager wiring

Author: blka

hyundai_kia_connect_api/ApiImplType1.py

@@ -195,6 +195,8 @@ def get_vehicles(self, token: Token) -> list[Vehicle]:
                 ccu_ccs2_protocol_support=entry["ccuCCS2ProtocolSupport"],
             )
             result.append(vehicle)
+        if self.supports_vehicle_profile:
+            self._fetch_vehicle_profiles(token, result)
         return result
 
     def _fetch_vehicle_profiles(self, token: Token, vehicles: list[Vehicle]) -> None:

hyundai_kia_connect_api/VehicleManager.py

@@ -127,7 +127,6 @@ def initialize_vehicles(self):
                 "Vehicles already initialized, this will re-initialize and cause data loss mapping errors"
             )
         vehicles = self.api.get_vehicles(self.token)
-        self.api._fetch_vehicle_profiles(self.token, vehicles)
         for vehicle in vehicles:
             vehicle.supports_window_control = self.api.supports_window_control
             self.vehicles[vehicle.id] = vehicle

tests/test_vehicle_profile.py

@@ -186,3 +186,50 @@ def test_capability_properties_work_with_mapped_profile(self, eu_api):
         assert v.air_purifier_supported is False
         assert v.is_left_hand_drive is True
         assert v.ev_alarm_supported is False
+
+
+class TestGetVehiclesProfileFetch:
+    """get_vehicles fetches profiles only when supports_vehicle_profile is set."""
+
+    def _make_api(self, supported: bool):
+        from hyundai_kia_connect_api.ApiImplType1 import ApiImplType1
+
+        api = object.__new__(ApiImplType1)
+        api.supports_vehicle_profile = supported
+        api.SPA_API_URL = "https://example.test/api/v1/spa/"
+        return api
+
+    def test_get_vehicles_fetches_profiles_when_supported(self):
+        import hyundai_kia_connect_api.ApiImplType1 as type1_mod
+        from hyundai_kia_connect_api.ApiImplType1 import ApiImplType1
+        from unittest.mock import patch, MagicMock
+
+        api = self._make_api(supported=True)
+        resp = MagicMock()
+        resp.json.return_value = {"resMsg": {"vehicles": []}}
+        with (
+            patch.object(ApiImplType1, "_get_authenticated_headers", return_value={}),
+            patch.object(type1_mod, "_check_response_for_errors", return_value=None),
+            patch.object(type1_mod.requests, "get", return_value=resp),
+            patch.object(ApiImplType1, "_fetch_vehicle_profiles") as fetch,
+        ):
+            result = ApiImplType1.get_vehicles(api, token=MagicMock())
+            assert result == []
+            fetch.assert_called_once()
+
+    def test_get_vehicles_skips_profiles_when_not_supported(self):
+        import hyundai_kia_connect_api.ApiImplType1 as type1_mod
+        from hyundai_kia_connect_api.ApiImplType1 import ApiImplType1
+        from unittest.mock import patch, MagicMock
+
+        api = self._make_api(supported=False)
+        resp = MagicMock()
+        resp.json.return_value = {"resMsg": {"vehicles": []}}
+        with (
+            patch.object(ApiImplType1, "_get_authenticated_headers", return_value={}),
+            patch.object(type1_mod, "_check_response_for_errors", return_value=None),
+            patch.object(type1_mod.requests, "get", return_value=resp),
+            patch.object(ApiImplType1, "_fetch_vehicle_profiles") as fetch,
+        ):
+            ApiImplType1.get_vehicles(api, token=MagicMock())
+            fetch.assert_not_called()