Filter devices based on online status in ViCare integration (home-assistant#157287)

CFenner · web-flow · commit 4654d6de876b · 2025-11-26T18:00:52.000+01:00
diff --git a/homeassistant/components/vicare/__init__.py b/homeassistant/components/vicare/__init__.py
@@ -76,6 +76,7 @@ def setup_vicare_api(hass: HomeAssistant, entry: ViCareConfigEntry) -> PyViCare:
     devices = [
         ViCareDevice(config=device_config, api=get_device(entry, device_config))
         for device_config in device_config_list
+        if bool(device_config.isOnline())
     ]
     return ViCareData(client=client, devices=devices)
 
diff --git a/tests/components/vicare/conftest.py b/tests/components/vicare/conftest.py
@@ -42,7 +42,7 @@ def __init__(self, fixtures: list[Fixture]) -> None:
                     "Vitovalor"
                     if fixture.data_file.endswith("VitoValor.json")
                     else f"model{idx}",
-                    "online",
+                    "Online",
                 )
             )
 

Original file line number	Diff line number	Diff line change
`@@ -76,6 +76,7 @@ def setup_vicare_api(hass: HomeAssistant, entry: ViCareConfigEntry) -> PyViCare:`
`76`	`76`	`devices = [`
`77`	`77`	`ViCareDevice(config=device_config, api=get_device(entry, device_config))`
`78`	`78`	`for device_config in device_config_list`
	`79`	`+ if bool(device_config.isOnline())`
`79`	`80`	`]`
`80`	`81`	`return ViCareData(client=client, devices=devices)`
`81`	`82`
Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ def __init__(self, fixtures: list[Fixture]) -> None:`
`42`	`42`	`"Vitovalor"`
`43`	`43`	`if fixture.data_file.endswith("VitoValor.json")`
`44`	`44`	`else f"model{idx}",`
`45`		`- "online",`
	`45`	`+ "Online",`
`46`	`46`	`)`
`47`	`47`	`)`
`48`	`48`