1
0
mirror of https://github.com/home-assistant/core.git synced 2026-04-18 07:56:03 +01:00

Use runtime_data in openuv integration (#167029)

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This commit is contained in:
epenet
2026-04-01 11:34:10 +02:00
committed by GitHub
parent 3369dfece1
commit d5c7a04751
8 changed files with 40 additions and 36 deletions

View File

@@ -7,7 +7,6 @@ from typing import Any
from pyopenuv import Client from pyopenuv import Client
from homeassistant.config_entries import ConfigEntry
from homeassistant.const import ( from homeassistant.const import (
CONF_API_KEY, CONF_API_KEY,
CONF_BINARY_SENSORS, CONF_BINARY_SENSORS,
@@ -27,15 +26,18 @@ from .const import (
DATA_UV, DATA_UV,
DEFAULT_FROM_WINDOW, DEFAULT_FROM_WINDOW,
DEFAULT_TO_WINDOW, DEFAULT_TO_WINDOW,
DOMAIN,
LOGGER, LOGGER,
) )
from .coordinator import OpenUvCoordinator, OpenUvProtectionWindowCoordinator from .coordinator import (
OpenUvConfigEntry,
OpenUvCoordinator,
OpenUvProtectionWindowCoordinator,
)
PLATFORMS = [Platform.BINARY_SENSOR, Platform.SENSOR] PLATFORMS = [Platform.BINARY_SENSOR, Platform.SENSOR]
async def async_setup_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool: async def async_setup_entry(hass: HomeAssistant, entry: OpenUvConfigEntry) -> bool:
"""Set up OpenUV as config entry.""" """Set up OpenUV as config entry."""
websession = aiohttp_client.async_get_clientsession(hass) websession = aiohttp_client.async_get_clientsession(hass)
client = Client( client = Client(
@@ -78,24 +80,19 @@ async def async_setup_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
] ]
await asyncio.gather(*init_tasks) await asyncio.gather(*init_tasks)
hass.data.setdefault(DOMAIN, {}) entry.runtime_data = coordinators
hass.data[DOMAIN][entry.entry_id] = coordinators
await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS) await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS)
return True return True
async def async_unload_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool: async def async_unload_entry(hass: HomeAssistant, entry: OpenUvConfigEntry) -> bool:
"""Unload an OpenUV config entry.""" """Unload an OpenUV config entry."""
unload_ok = await hass.config_entries.async_unload_platforms(entry, PLATFORMS) return await hass.config_entries.async_unload_platforms(entry, PLATFORMS)
if unload_ok:
hass.data[DOMAIN].pop(entry.entry_id)
return unload_ok
async def async_migrate_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool: async def async_migrate_entry(hass: HomeAssistant, entry: OpenUvConfigEntry) -> bool:
"""Migrate the config entry upon new versions.""" """Migrate the config entry upon new versions."""
version = entry.version version = entry.version
data = {**entry.data} data = {**entry.data}

View File

@@ -4,13 +4,12 @@ from homeassistant.components.binary_sensor import (
BinarySensorEntity, BinarySensorEntity,
BinarySensorEntityDescription, BinarySensorEntityDescription,
) )
from homeassistant.config_entries import ConfigEntry
from homeassistant.core import HomeAssistant, callback from homeassistant.core import HomeAssistant, callback
from homeassistant.helpers.entity_platform import AddConfigEntryEntitiesCallback from homeassistant.helpers.entity_platform import AddConfigEntryEntitiesCallback
from homeassistant.util.dt import as_local from homeassistant.util.dt import as_local
from .const import DATA_PROTECTION_WINDOW, DOMAIN, LOGGER, TYPE_PROTECTION_WINDOW from .const import DATA_PROTECTION_WINDOW, LOGGER, TYPE_PROTECTION_WINDOW
from .coordinator import OpenUvCoordinator from .coordinator import OpenUvConfigEntry
from .entity import OpenUvEntity from .entity import OpenUvEntity
ATTR_PROTECTION_WINDOW_ENDING_TIME = "end_time" ATTR_PROTECTION_WINDOW_ENDING_TIME = "end_time"
@@ -26,12 +25,11 @@ BINARY_SENSOR_DESCRIPTION_PROTECTION_WINDOW = BinarySensorEntityDescription(
async def async_setup_entry( async def async_setup_entry(
hass: HomeAssistant, hass: HomeAssistant,
entry: ConfigEntry, entry: OpenUvConfigEntry,
async_add_entities: AddConfigEntryEntitiesCallback, async_add_entities: AddConfigEntryEntitiesCallback,
) -> None: ) -> None:
# Once we've successfully authenticated, we re-enable client request retries: """Set up OpenUV binary sensors for a config entry."""
"""Set up an OpenUV sensor based on a config entry.""" coordinators = entry.runtime_data
coordinators: dict[str, OpenUvCoordinator] = hass.data[DOMAIN][entry.entry_id]
async_add_entities( async_add_entities(
[ [

View File

@@ -10,7 +10,7 @@ from pyopenuv import Client
from pyopenuv.errors import OpenUvError from pyopenuv.errors import OpenUvError
import voluptuous as vol import voluptuous as vol
from homeassistant.config_entries import ConfigEntry, ConfigFlow, ConfigFlowResult from homeassistant.config_entries import ConfigFlow, ConfigFlowResult
from homeassistant.const import ( from homeassistant.const import (
CONF_API_KEY, CONF_API_KEY,
CONF_ELEVATION, CONF_ELEVATION,
@@ -31,6 +31,7 @@ from .const import (
DEFAULT_TO_WINDOW, DEFAULT_TO_WINDOW,
DOMAIN, DOMAIN,
) )
from .coordinator import OpenUvConfigEntry
STEP_REAUTH_SCHEMA = vol.Schema( STEP_REAUTH_SCHEMA = vol.Schema(
{ {
@@ -133,7 +134,9 @@ class OpenUvFlowHandler(ConfigFlow, domain=DOMAIN):
@staticmethod @staticmethod
@callback @callback
def async_get_options_flow(config_entry: ConfigEntry) -> SchemaOptionsFlowHandler: def async_get_options_flow(
config_entry: OpenUvConfigEntry,
) -> SchemaOptionsFlowHandler:
"""Define the config flow to handle options.""" """Define the config flow to handle options."""
return SchemaOptionsFlowHandler(config_entry, OPTIONS_FLOW) return SchemaOptionsFlowHandler(config_entry, OPTIONS_FLOW)

View File

@@ -20,18 +20,20 @@ from .const import LOGGER
DEFAULT_DEBOUNCER_COOLDOWN_SECONDS = 15 * 60 DEFAULT_DEBOUNCER_COOLDOWN_SECONDS = 15 * 60
type OpenUvConfigEntry = ConfigEntry[dict[str, OpenUvCoordinator]]
class OpenUvCoordinator(DataUpdateCoordinator[dict[str, Any]]): class OpenUvCoordinator(DataUpdateCoordinator[dict[str, Any]]):
"""Define an OpenUV data coordinator.""" """Define an OpenUV data coordinator."""
config_entry: ConfigEntry config_entry: OpenUvConfigEntry
update_method: Callable[[], Awaitable[dict[str, Any]]] update_method: Callable[[], Awaitable[dict[str, Any]]]
def __init__( def __init__(
self, self,
hass: HomeAssistant, hass: HomeAssistant,
*, *,
entry: ConfigEntry, entry: OpenUvConfigEntry,
name: str, name: str,
latitude: str, latitude: str,
longitude: str, longitude: str,

View File

@@ -5,7 +5,6 @@ from __future__ import annotations
from typing import Any from typing import Any
from homeassistant.components.diagnostics import async_redact_data from homeassistant.components.diagnostics import async_redact_data
from homeassistant.config_entries import ConfigEntry
from homeassistant.const import ( from homeassistant.const import (
CONF_API_KEY, CONF_API_KEY,
CONF_LATITUDE, CONF_LATITUDE,
@@ -14,8 +13,7 @@ from homeassistant.const import (
) )
from homeassistant.core import HomeAssistant from homeassistant.core import HomeAssistant
from .const import DOMAIN from .coordinator import OpenUvConfigEntry
from .coordinator import OpenUvCoordinator
CONF_COORDINATES = "coordinates" CONF_COORDINATES = "coordinates"
CONF_TITLE = "title" CONF_TITLE = "title"
@@ -31,10 +29,10 @@ TO_REDACT = {
async def async_get_config_entry_diagnostics( async def async_get_config_entry_diagnostics(
hass: HomeAssistant, entry: ConfigEntry hass: HomeAssistant, entry: OpenUvConfigEntry
) -> dict[str, Any]: ) -> dict[str, Any]:
"""Return diagnostics for a config entry.""" """Return diagnostics for a config entry."""
coordinators: dict[str, OpenUvCoordinator] = hass.data[DOMAIN][entry.entry_id] coordinators = entry.runtime_data
return async_redact_data( return async_redact_data(
{ {

View File

@@ -12,7 +12,6 @@ from homeassistant.components.sensor import (
SensorEntityDescription, SensorEntityDescription,
SensorStateClass, SensorStateClass,
) )
from homeassistant.config_entries import ConfigEntry
from homeassistant.const import UV_INDEX, UnitOfTime from homeassistant.const import UV_INDEX, UnitOfTime
from homeassistant.core import HomeAssistant from homeassistant.core import HomeAssistant
from homeassistant.helpers.entity_platform import AddConfigEntryEntitiesCallback from homeassistant.helpers.entity_platform import AddConfigEntryEntitiesCallback
@@ -20,7 +19,6 @@ from homeassistant.util.dt import as_local, parse_datetime
from .const import ( from .const import (
DATA_UV, DATA_UV,
DOMAIN,
TYPE_CURRENT_OZONE_LEVEL, TYPE_CURRENT_OZONE_LEVEL,
TYPE_CURRENT_UV_INDEX, TYPE_CURRENT_UV_INDEX,
TYPE_CURRENT_UV_LEVEL, TYPE_CURRENT_UV_LEVEL,
@@ -32,7 +30,7 @@ from .const import (
TYPE_SAFE_EXPOSURE_TIME_5, TYPE_SAFE_EXPOSURE_TIME_5,
TYPE_SAFE_EXPOSURE_TIME_6, TYPE_SAFE_EXPOSURE_TIME_6,
) )
from .coordinator import OpenUvCoordinator from .coordinator import OpenUvConfigEntry
from .entity import OpenUvEntity from .entity import OpenUvEntity
ATTR_MAX_UV_TIME = "time" ATTR_MAX_UV_TIME = "time"
@@ -167,11 +165,11 @@ SENSOR_DESCRIPTIONS = (
async def async_setup_entry( async def async_setup_entry(
hass: HomeAssistant, hass: HomeAssistant,
entry: ConfigEntry, entry: OpenUvConfigEntry,
async_add_entities: AddConfigEntryEntitiesCallback, async_add_entities: AddConfigEntryEntitiesCallback,
) -> None: ) -> None:
"""Set up a OpenUV sensor based on a config entry.""" """Set up a OpenUV sensor based on a config entry."""
coordinators: dict[str, OpenUvCoordinator] = hass.data[DOMAIN][entry.entry_id] coordinators = entry.runtime_data
async_add_entities( async_add_entities(
[ [

View File

@@ -7,7 +7,11 @@ from unittest.mock import AsyncMock, Mock, patch
import pytest import pytest
from homeassistant.components.openuv import CONF_FROM_WINDOW, CONF_TO_WINDOW, DOMAIN from homeassistant.components.openuv.const import (
CONF_FROM_WINDOW,
CONF_TO_WINDOW,
DOMAIN,
)
from homeassistant.const import ( from homeassistant.const import (
CONF_API_KEY, CONF_API_KEY,
CONF_ELEVATION, CONF_ELEVATION,

View File

@@ -6,7 +6,11 @@ from pyopenuv.errors import InvalidApiKeyError
import pytest import pytest
import voluptuous as vol import voluptuous as vol
from homeassistant.components.openuv import CONF_FROM_WINDOW, CONF_TO_WINDOW, DOMAIN from homeassistant.components.openuv.const import (
CONF_FROM_WINDOW,
CONF_TO_WINDOW,
DOMAIN,
)
from homeassistant.config_entries import SOURCE_USER from homeassistant.config_entries import SOURCE_USER
from homeassistant.const import ( from homeassistant.const import (
CONF_API_KEY, CONF_API_KEY,