Allow to remove devices in Sensibo (#101890)
parent
148087a1c9
commit
264eef8dd8
|
@ -3,9 +3,12 @@ from __future__ import annotations
|
|||
|
||||
from pysensibo.exceptions import AuthenticationError
|
||||
|
||||
from homeassistant.components.climate import DOMAIN as CLIMATE_DOMAIN
|
||||
from homeassistant.config_entries import ConfigEntry
|
||||
from homeassistant.const import CONF_API_KEY
|
||||
from homeassistant.core import HomeAssistant
|
||||
from homeassistant.helpers import entity_registry as er
|
||||
from homeassistant.helpers.device_registry import DeviceEntry
|
||||
|
||||
from .const import DOMAIN, LOGGER, PLATFORMS
|
||||
from .coordinator import SensiboDataUpdateCoordinator
|
||||
|
@ -53,3 +56,17 @@ async def async_migrate_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
|
|||
)
|
||||
|
||||
return True
|
||||
|
||||
|
||||
async def async_remove_config_entry_device(
|
||||
hass: HomeAssistant, entry: ConfigEntry, device: DeviceEntry
|
||||
) -> bool:
|
||||
"""Remove Sensibo config entry from a device."""
|
||||
entity_registry = er.async_get(hass)
|
||||
for identifier in device.identifiers:
|
||||
if identifier[0] == DOMAIN and entity_registry.async_get_entity_id(
|
||||
CLIMATE_DOMAIN, DOMAIN, identifier[1]
|
||||
):
|
||||
return False
|
||||
|
||||
return True
|
||||
|
|
|
@ -8,12 +8,15 @@ from pysensibo.model import SensiboData
|
|||
from homeassistant import config_entries
|
||||
from homeassistant.components.sensibo.const import DOMAIN
|
||||
from homeassistant.components.sensibo.util import NoUsernameError
|
||||
from homeassistant.config_entries import SOURCE_USER
|
||||
from homeassistant.config_entries import SOURCE_USER, ConfigEntry
|
||||
from homeassistant.core import HomeAssistant
|
||||
from homeassistant.helpers import device_registry as dr, entity_registry as er
|
||||
from homeassistant.setup import async_setup_component
|
||||
|
||||
from . import ENTRY_CONFIG
|
||||
|
||||
from tests.common import MockConfigEntry
|
||||
from tests.typing import WebSocketGenerator
|
||||
|
||||
|
||||
async def test_setup_entry(hass: HomeAssistant, get_data: SensiboData) -> None:
|
||||
|
@ -131,3 +134,48 @@ async def test_unload_entry(hass: HomeAssistant, get_data: SensiboData) -> None:
|
|||
assert await hass.config_entries.async_unload(entry.entry_id)
|
||||
await hass.async_block_till_done()
|
||||
assert entry.state is config_entries.ConfigEntryState.NOT_LOADED
|
||||
|
||||
|
||||
async def remove_device(ws_client, device_id, config_entry_id):
|
||||
"""Remove config entry from a device."""
|
||||
await ws_client.send_json(
|
||||
{
|
||||
"id": 5,
|
||||
"type": "config/device_registry/remove_config_entry",
|
||||
"config_entry_id": config_entry_id,
|
||||
"device_id": device_id,
|
||||
}
|
||||
)
|
||||
response = await ws_client.receive_json()
|
||||
return response["success"]
|
||||
|
||||
|
||||
async def test_device_remove_devices(
|
||||
hass: HomeAssistant,
|
||||
load_int: ConfigEntry,
|
||||
hass_ws_client: WebSocketGenerator,
|
||||
) -> None:
|
||||
"""Test we can only remove a device that no longer exists."""
|
||||
assert await async_setup_component(hass, "config", {})
|
||||
registry: er.EntityRegistry = er.async_get(hass)
|
||||
entity = registry.entities["climate.hallway"]
|
||||
|
||||
device_registry = dr.async_get(hass)
|
||||
device_entry = device_registry.async_get(entity.device_id)
|
||||
assert (
|
||||
await remove_device(
|
||||
await hass_ws_client(hass), device_entry.id, load_int.entry_id
|
||||
)
|
||||
is False
|
||||
)
|
||||
|
||||
dead_device_entry = device_registry.async_get_or_create(
|
||||
config_entry_id=load_int.entry_id,
|
||||
identifiers={(DOMAIN, "remove-device-id")},
|
||||
)
|
||||
assert (
|
||||
await remove_device(
|
||||
await hass_ws_client(hass), dead_device_entry.id, load_int.entry_id
|
||||
)
|
||||
is True
|
||||
)
|
||||
|
|
Loading…
Reference in New Issue