2023-11-25 13:50:44 +00:00
|
|
|
"""Test climate intents."""
|
2024-02-07 21:13:42 +00:00
|
|
|
|
2023-11-25 13:50:44 +00:00
|
|
|
from collections.abc import Generator
|
|
|
|
from unittest.mock import patch
|
|
|
|
|
|
|
|
import pytest
|
|
|
|
|
|
|
|
from homeassistant.components.climate import (
|
|
|
|
DOMAIN,
|
|
|
|
ClimateEntity,
|
|
|
|
HVACMode,
|
|
|
|
intent as climate_intent,
|
|
|
|
)
|
|
|
|
from homeassistant.config_entries import ConfigEntry, ConfigFlow
|
|
|
|
from homeassistant.const import Platform, UnitOfTemperature
|
|
|
|
from homeassistant.core import HomeAssistant
|
|
|
|
from homeassistant.helpers import area_registry as ar, entity_registry as er, intent
|
|
|
|
from homeassistant.helpers.entity_platform import AddEntitiesCallback
|
|
|
|
|
|
|
|
from tests.common import (
|
|
|
|
MockConfigEntry,
|
|
|
|
MockModule,
|
|
|
|
MockPlatform,
|
|
|
|
mock_config_flow,
|
|
|
|
mock_integration,
|
|
|
|
mock_platform,
|
|
|
|
)
|
|
|
|
|
|
|
|
TEST_DOMAIN = "test"
|
|
|
|
|
|
|
|
|
|
|
|
class MockFlow(ConfigFlow):
|
|
|
|
"""Test flow."""
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.fixture(autouse=True)
|
|
|
|
def config_flow_fixture(hass: HomeAssistant) -> Generator[None, None, None]:
|
|
|
|
"""Mock config flow."""
|
|
|
|
mock_platform(hass, f"{TEST_DOMAIN}.config_flow")
|
|
|
|
|
|
|
|
with mock_config_flow(TEST_DOMAIN, MockFlow):
|
|
|
|
yield
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.fixture(autouse=True)
|
|
|
|
def mock_setup_integration(hass: HomeAssistant) -> None:
|
|
|
|
"""Fixture to set up a mock integration."""
|
|
|
|
|
|
|
|
async def async_setup_entry_init(
|
|
|
|
hass: HomeAssistant, config_entry: ConfigEntry
|
|
|
|
) -> bool:
|
|
|
|
"""Set up test config entry."""
|
Ensure config entries are not unloaded while their platforms are setting up (#118767)
* Report non-awaited/non-locked config entry platform forwards
Its currently possible for config entries to be reloaded while their platforms
are being forwarded if platform forwards are not awaited or done after the
config entry is setup since the lock will not be held in this case.
In https://developers.home-assistant.io/blog/2022/07/08/config_entry_forwards
we advised to await platform forwards to ensure this does not happen, however
for sleeping devices and late discovered devices, platform forwards may happen
later.
If config platform forwards are happening during setup, they should be awaited
If config entry platform forwards are not happening during setup, instead
async_late_forward_entry_setups should be used which will hold the lock to
prevent the config entry from being unloaded while its platforms are being
setup
* Report non-awaited/non-locked config entry platform forwards
Its currently possible for config entries to be reloaded while their platforms
are being forwarded if platform forwards are not awaited or done after the
config entry is setup since the lock will not be held in this case.
In https://developers.home-assistant.io/blog/2022/07/08/config_entry_forwards
we advised to await platform forwards to ensure this does not happen, however
for sleeping devices and late discovered devices, platform forwards may happen
later.
If config platform forwards are happening during setup, they should be awaited
If config entry platform forwards are not happening during setup, instead
async_late_forward_entry_setups should be used which will hold the lock to
prevent the config entry from being unloaded while its platforms are being
setup
* run with error on to find them
* cert_exp, hold lock
* cert_exp, hold lock
* shelly async_late_forward_entry_setups
* compact
* compact
* found another
* patch up mobileapp
* patch up hue tests
* patch up smartthings
* fix mqtt
* fix esphome
* zwave_js
* mqtt
* rework
* fixes
* fix mocking
* fix mocking
* do not call async_forward_entry_setup directly
* docstrings
* docstrings
* docstrings
* add comments
* doc strings
* fixed all in core, turn off strict
* coverage
* coverage
* missing
* coverage
2024-06-05 01:34:39 +00:00
|
|
|
await hass.config_entries.async_forward_entry_setups(config_entry, [DOMAIN])
|
2023-11-25 13:50:44 +00:00
|
|
|
return True
|
|
|
|
|
|
|
|
async def async_unload_entry_init(
|
|
|
|
hass: HomeAssistant,
|
|
|
|
config_entry: ConfigEntry,
|
|
|
|
) -> bool:
|
|
|
|
await hass.config_entries.async_unload_platforms(config_entry, [Platform.TODO])
|
|
|
|
return True
|
|
|
|
|
|
|
|
mock_platform(hass, f"{TEST_DOMAIN}.config_flow")
|
|
|
|
mock_integration(
|
|
|
|
hass,
|
|
|
|
MockModule(
|
|
|
|
TEST_DOMAIN,
|
|
|
|
async_setup_entry=async_setup_entry_init,
|
|
|
|
async_unload_entry=async_unload_entry_init,
|
|
|
|
),
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
|
|
async def create_mock_platform(
|
|
|
|
hass: HomeAssistant,
|
|
|
|
entities: list[ClimateEntity],
|
|
|
|
) -> MockConfigEntry:
|
|
|
|
"""Create a todo platform with the specified entities."""
|
|
|
|
|
|
|
|
async def async_setup_entry_platform(
|
|
|
|
hass: HomeAssistant,
|
|
|
|
config_entry: ConfigEntry,
|
|
|
|
async_add_entities: AddEntitiesCallback,
|
|
|
|
) -> None:
|
|
|
|
"""Set up test event platform via config entry."""
|
|
|
|
async_add_entities(entities)
|
|
|
|
|
|
|
|
mock_platform(
|
|
|
|
hass,
|
|
|
|
f"{TEST_DOMAIN}.{DOMAIN}",
|
|
|
|
MockPlatform(async_setup_entry=async_setup_entry_platform),
|
|
|
|
)
|
|
|
|
|
|
|
|
config_entry = MockConfigEntry(domain=TEST_DOMAIN)
|
|
|
|
config_entry.add_to_hass(hass)
|
|
|
|
assert await hass.config_entries.async_setup(config_entry.entry_id)
|
|
|
|
await hass.async_block_till_done()
|
|
|
|
|
|
|
|
return config_entry
|
|
|
|
|
|
|
|
|
|
|
|
class MockClimateEntity(ClimateEntity):
|
|
|
|
"""Mock Climate device to use in tests."""
|
|
|
|
|
|
|
|
_attr_temperature_unit = UnitOfTemperature.CELSIUS
|
|
|
|
_attr_hvac_mode = HVACMode.OFF
|
|
|
|
_attr_hvac_modes = [HVACMode.OFF, HVACMode.HEAT]
|
|
|
|
|
|
|
|
|
|
|
|
async def test_get_temperature(
|
|
|
|
hass: HomeAssistant,
|
|
|
|
area_registry: ar.AreaRegistry,
|
|
|
|
entity_registry: er.EntityRegistry,
|
|
|
|
) -> None:
|
|
|
|
"""Test HassClimateGetTemperature intent."""
|
|
|
|
await climate_intent.async_setup_intents(hass)
|
|
|
|
|
|
|
|
climate_1 = MockClimateEntity()
|
|
|
|
climate_1._attr_name = "Climate 1"
|
|
|
|
climate_1._attr_unique_id = "1234"
|
|
|
|
climate_1._attr_current_temperature = 10.0
|
|
|
|
entity_registry.async_get_or_create(
|
|
|
|
DOMAIN, "test", "1234", suggested_object_id="climate_1"
|
|
|
|
)
|
|
|
|
|
|
|
|
climate_2 = MockClimateEntity()
|
|
|
|
climate_2._attr_name = "Climate 2"
|
|
|
|
climate_2._attr_unique_id = "5678"
|
|
|
|
climate_2._attr_current_temperature = 22.0
|
|
|
|
entity_registry.async_get_or_create(
|
|
|
|
DOMAIN, "test", "5678", suggested_object_id="climate_2"
|
|
|
|
)
|
|
|
|
|
|
|
|
await create_mock_platform(hass, [climate_1, climate_2])
|
|
|
|
|
|
|
|
# Add climate entities to different areas:
|
|
|
|
# climate_1 => living room
|
|
|
|
# climate_2 => bedroom
|
2024-02-07 21:13:42 +00:00
|
|
|
# nothing in office
|
2023-11-25 13:50:44 +00:00
|
|
|
living_room_area = area_registry.async_create(name="Living Room")
|
|
|
|
bedroom_area = area_registry.async_create(name="Bedroom")
|
2024-02-07 21:13:42 +00:00
|
|
|
office_area = area_registry.async_create(name="Office")
|
2023-11-25 13:50:44 +00:00
|
|
|
|
|
|
|
entity_registry.async_update_entity(
|
|
|
|
climate_1.entity_id, area_id=living_room_area.id
|
|
|
|
)
|
|
|
|
entity_registry.async_update_entity(climate_2.entity_id, area_id=bedroom_area.id)
|
|
|
|
|
|
|
|
# First climate entity will be selected (no area)
|
|
|
|
response = await intent.async_handle(
|
|
|
|
hass, "test", climate_intent.INTENT_GET_TEMPERATURE, {}
|
|
|
|
)
|
|
|
|
assert response.response_type == intent.IntentResponseType.QUERY_ANSWER
|
|
|
|
assert len(response.matched_states) == 1
|
|
|
|
assert response.matched_states[0].entity_id == climate_1.entity_id
|
|
|
|
state = response.matched_states[0]
|
|
|
|
assert state.attributes["current_temperature"] == 10.0
|
|
|
|
|
2023-12-13 03:54:15 +00:00
|
|
|
# Select by area (climate_2)
|
2023-11-25 13:50:44 +00:00
|
|
|
response = await intent.async_handle(
|
|
|
|
hass,
|
|
|
|
"test",
|
|
|
|
climate_intent.INTENT_GET_TEMPERATURE,
|
2024-02-07 21:13:42 +00:00
|
|
|
{"area": {"value": bedroom_area.name}},
|
2023-11-25 13:50:44 +00:00
|
|
|
)
|
|
|
|
assert response.response_type == intent.IntentResponseType.QUERY_ANSWER
|
|
|
|
assert len(response.matched_states) == 1
|
|
|
|
assert response.matched_states[0].entity_id == climate_2.entity_id
|
|
|
|
state = response.matched_states[0]
|
|
|
|
assert state.attributes["current_temperature"] == 22.0
|
|
|
|
|
2023-12-13 03:54:15 +00:00
|
|
|
# Select by name (climate_2)
|
|
|
|
response = await intent.async_handle(
|
|
|
|
hass,
|
|
|
|
"test",
|
|
|
|
climate_intent.INTENT_GET_TEMPERATURE,
|
|
|
|
{"name": {"value": "Climate 2"}},
|
|
|
|
)
|
|
|
|
assert response.response_type == intent.IntentResponseType.QUERY_ANSWER
|
|
|
|
assert len(response.matched_states) == 1
|
|
|
|
assert response.matched_states[0].entity_id == climate_2.entity_id
|
|
|
|
state = response.matched_states[0]
|
|
|
|
assert state.attributes["current_temperature"] == 22.0
|
|
|
|
|
2024-02-07 21:13:42 +00:00
|
|
|
# Check area with no climate entities
|
2024-05-08 02:01:03 +00:00
|
|
|
with pytest.raises(intent.MatchFailedError) as error:
|
2024-02-07 21:13:42 +00:00
|
|
|
response = await intent.async_handle(
|
|
|
|
hass,
|
|
|
|
"test",
|
|
|
|
climate_intent.INTENT_GET_TEMPERATURE,
|
|
|
|
{"area": {"value": office_area.name}},
|
|
|
|
)
|
|
|
|
|
|
|
|
# Exception should contain details of what we tried to match
|
2024-05-08 02:01:03 +00:00
|
|
|
assert isinstance(error.value, intent.MatchFailedError)
|
|
|
|
assert error.value.result.no_match_reason == intent.MatchFailedReason.AREA
|
|
|
|
constraints = error.value.constraints
|
|
|
|
assert constraints.name is None
|
|
|
|
assert constraints.area_name == office_area.name
|
|
|
|
assert constraints.domains == {DOMAIN}
|
|
|
|
assert constraints.device_classes is None
|
2024-02-07 21:13:42 +00:00
|
|
|
|
|
|
|
# Check wrong name
|
2024-05-08 02:01:03 +00:00
|
|
|
with pytest.raises(intent.MatchFailedError) as error:
|
2024-02-07 21:13:42 +00:00
|
|
|
response = await intent.async_handle(
|
|
|
|
hass,
|
|
|
|
"test",
|
|
|
|
climate_intent.INTENT_GET_TEMPERATURE,
|
|
|
|
{"name": {"value": "Does not exist"}},
|
|
|
|
)
|
|
|
|
|
2024-05-08 02:01:03 +00:00
|
|
|
assert isinstance(error.value, intent.MatchFailedError)
|
|
|
|
assert error.value.result.no_match_reason == intent.MatchFailedReason.NAME
|
|
|
|
constraints = error.value.constraints
|
|
|
|
assert constraints.name == "Does not exist"
|
|
|
|
assert constraints.area_name is None
|
|
|
|
assert constraints.domains == {DOMAIN}
|
|
|
|
assert constraints.device_classes is None
|
2024-02-07 21:13:42 +00:00
|
|
|
|
|
|
|
# Check wrong name with area
|
2024-05-08 02:01:03 +00:00
|
|
|
with pytest.raises(intent.MatchFailedError) as error:
|
2024-02-07 21:13:42 +00:00
|
|
|
response = await intent.async_handle(
|
|
|
|
hass,
|
|
|
|
"test",
|
|
|
|
climate_intent.INTENT_GET_TEMPERATURE,
|
|
|
|
{"name": {"value": "Climate 1"}, "area": {"value": bedroom_area.name}},
|
|
|
|
)
|
|
|
|
|
2024-05-08 02:01:03 +00:00
|
|
|
assert isinstance(error.value, intent.MatchFailedError)
|
|
|
|
assert error.value.result.no_match_reason == intent.MatchFailedReason.AREA
|
|
|
|
constraints = error.value.constraints
|
|
|
|
assert constraints.name == "Climate 1"
|
|
|
|
assert constraints.area_name == bedroom_area.name
|
|
|
|
assert constraints.domains == {DOMAIN}
|
|
|
|
assert constraints.device_classes is None
|
2024-02-07 21:13:42 +00:00
|
|
|
|
2023-11-25 13:50:44 +00:00
|
|
|
|
|
|
|
async def test_get_temperature_no_entities(
|
|
|
|
hass: HomeAssistant,
|
|
|
|
) -> None:
|
|
|
|
"""Test HassClimateGetTemperature intent with no climate entities."""
|
|
|
|
await climate_intent.async_setup_intents(hass)
|
|
|
|
|
|
|
|
await create_mock_platform(hass, [])
|
|
|
|
|
|
|
|
with pytest.raises(intent.IntentHandleError):
|
|
|
|
await intent.async_handle(
|
|
|
|
hass, "test", climate_intent.INTENT_GET_TEMPERATURE, {}
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
|
|
async def test_get_temperature_no_state(
|
|
|
|
hass: HomeAssistant,
|
|
|
|
area_registry: ar.AreaRegistry,
|
|
|
|
entity_registry: er.EntityRegistry,
|
|
|
|
) -> None:
|
|
|
|
"""Test HassClimateGetTemperature intent when states are missing."""
|
|
|
|
await climate_intent.async_setup_intents(hass)
|
|
|
|
|
|
|
|
climate_1 = MockClimateEntity()
|
|
|
|
climate_1._attr_name = "Climate 1"
|
|
|
|
climate_1._attr_unique_id = "1234"
|
|
|
|
entity_registry.async_get_or_create(
|
|
|
|
DOMAIN, "test", "1234", suggested_object_id="climate_1"
|
|
|
|
)
|
|
|
|
|
|
|
|
await create_mock_platform(hass, [climate_1])
|
|
|
|
|
|
|
|
living_room_area = area_registry.async_create(name="Living Room")
|
|
|
|
entity_registry.async_update_entity(
|
|
|
|
climate_1.entity_id, area_id=living_room_area.id
|
|
|
|
)
|
|
|
|
|
2024-02-07 21:13:42 +00:00
|
|
|
with (
|
|
|
|
patch("homeassistant.core.StateMachine.get", return_value=None),
|
|
|
|
pytest.raises(intent.IntentHandleError),
|
2023-11-25 13:50:44 +00:00
|
|
|
):
|
|
|
|
await intent.async_handle(
|
|
|
|
hass, "test", climate_intent.INTENT_GET_TEMPERATURE, {}
|
|
|
|
)
|
|
|
|
|
2024-02-07 21:13:42 +00:00
|
|
|
with (
|
|
|
|
patch("homeassistant.core.StateMachine.async_all", return_value=[]),
|
2024-05-08 02:01:03 +00:00
|
|
|
pytest.raises(intent.MatchFailedError) as error,
|
2024-02-07 21:13:42 +00:00
|
|
|
):
|
2023-11-25 13:50:44 +00:00
|
|
|
await intent.async_handle(
|
|
|
|
hass,
|
|
|
|
"test",
|
|
|
|
climate_intent.INTENT_GET_TEMPERATURE,
|
|
|
|
{"area": {"value": "Living Room"}},
|
|
|
|
)
|
2024-02-07 21:13:42 +00:00
|
|
|
|
|
|
|
# Exception should contain details of what we tried to match
|
2024-05-08 02:01:03 +00:00
|
|
|
assert isinstance(error.value, intent.MatchFailedError)
|
|
|
|
assert error.value.result.no_match_reason == intent.MatchFailedReason.AREA
|
|
|
|
constraints = error.value.constraints
|
|
|
|
assert constraints.name is None
|
|
|
|
assert constraints.area_name == "Living Room"
|
|
|
|
assert constraints.domains == {DOMAIN}
|
|
|
|
assert constraints.device_classes is None
|