Skip to content

Commit 8e185ce

Browse files
committed
Fix analytics after supervisor info change
1 parent 1d7efbf commit 8e185ce

2 files changed

Lines changed: 12 additions & 7 deletions

File tree

homeassistant/components/analytics/analytics.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -338,6 +338,7 @@ async def send_analytics(self, _: datetime | None = None) -> None:
338338

339339
hass = self._hass
340340
supervisor_info = None
341+
addons_info: dict[str, Any] | None = None
341342
operating_system_info: dict[str, Any] = {}
342343

343344
if self._data.uuid is None:
@@ -347,6 +348,7 @@ async def send_analytics(self, _: datetime | None = None) -> None:
347348
if self.supervisor:
348349
supervisor_info = hassio.get_supervisor_info(hass)
349350
operating_system_info = hassio.get_os_info(hass) or {}
351+
addons_info = hassio.get_addons_info(hass) or {}
350352

351353
system_info = await async_get_system_info(hass)
352354
integrations = []
@@ -419,13 +421,10 @@ async def send_analytics(self, _: datetime | None = None) -> None:
419421

420422
integrations.append(integration.domain)
421423

422-
if supervisor_info is not None:
424+
if addons_info is not None:
423425
supervisor_client = hassio.get_supervisor_client(hass)
424426
installed_addons = await asyncio.gather(
425-
*(
426-
supervisor_client.addons.addon_info(addon[ATTR_SLUG])
427-
for addon in supervisor_info[ATTR_ADDONS]
428-
)
427+
*(supervisor_client.addons.addon_info(slug) for slug in addons_info)
429428
)
430429
addons.extend(
431430
{

tests/components/analytics/test_analytics.py

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -359,10 +359,13 @@ async def test_send_usage_with_supervisor(
359359
"healthy": True,
360360
"supported": True,
361361
"arch": "amd64",
362-
"addons": [{"slug": "test_addon"}],
363362
}
364363
),
365364
),
365+
patch(
366+
"homeassistant.components.hassio.get_addons_info",
367+
side_effect=Mock(return_value={"test_addon": {}}),
368+
),
366369
patch(
367370
"homeassistant.components.hassio.get_os_info",
368371
side_effect=Mock(return_value={}),
@@ -578,10 +581,13 @@ async def test_send_statistics_with_supervisor(
578581
"healthy": True,
579582
"supported": True,
580583
"arch": "amd64",
581-
"addons": [{"slug": "test_addon"}],
582584
}
583585
),
584586
),
587+
patch(
588+
"homeassistant.components.hassio.get_addons_info",
589+
side_effect=Mock(return_value={"test_addon": {}}),
590+
),
585591
patch(
586592
"homeassistant.components.hassio.get_os_info",
587593
side_effect=Mock(return_value={}),

0 commit comments

Comments
 (0)