Add climate temp ranges support for Airzone Cloud (#115025)
airzone_cloud: climate: add temperature ranges support Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
This commit is contained in:
parent
14fcf7be8e
commit
3774d8ed54
2 changed files with 73 additions and 26 deletions
|
@ -11,11 +11,14 @@ from aioairzone_cloud.const import (
|
||||||
API_PARAMS,
|
API_PARAMS,
|
||||||
API_POWER,
|
API_POWER,
|
||||||
API_SETPOINT,
|
API_SETPOINT,
|
||||||
|
API_SP_AIR_COOL,
|
||||||
|
API_SP_AIR_HEAT,
|
||||||
API_SPEED_CONF,
|
API_SPEED_CONF,
|
||||||
API_UNITS,
|
API_UNITS,
|
||||||
API_VALUE,
|
API_VALUE,
|
||||||
AZD_ACTION,
|
AZD_ACTION,
|
||||||
AZD_AIDOOS,
|
AZD_AIDOOS,
|
||||||
|
AZD_DOUBLE_SET_POINT,
|
||||||
AZD_GROUPS,
|
AZD_GROUPS,
|
||||||
AZD_HUMIDITY,
|
AZD_HUMIDITY,
|
||||||
AZD_INSTALLATIONS,
|
AZD_INSTALLATIONS,
|
||||||
|
@ -29,6 +32,8 @@ from aioairzone_cloud.const import (
|
||||||
AZD_SPEEDS,
|
AZD_SPEEDS,
|
||||||
AZD_TEMP,
|
AZD_TEMP,
|
||||||
AZD_TEMP_SET,
|
AZD_TEMP_SET,
|
||||||
|
AZD_TEMP_SET_COOL_AIR,
|
||||||
|
AZD_TEMP_SET_HOT_AIR,
|
||||||
AZD_TEMP_SET_MAX,
|
AZD_TEMP_SET_MAX,
|
||||||
AZD_TEMP_SET_MIN,
|
AZD_TEMP_SET_MIN,
|
||||||
AZD_TEMP_STEP,
|
AZD_TEMP_STEP,
|
||||||
|
@ -37,6 +42,8 @@ from aioairzone_cloud.const import (
|
||||||
|
|
||||||
from homeassistant.components.climate import (
|
from homeassistant.components.climate import (
|
||||||
ATTR_HVAC_MODE,
|
ATTR_HVAC_MODE,
|
||||||
|
ATTR_TARGET_TEMP_HIGH,
|
||||||
|
ATTR_TARGET_TEMP_LOW,
|
||||||
FAN_AUTO,
|
FAN_AUTO,
|
||||||
FAN_HIGH,
|
FAN_HIGH,
|
||||||
FAN_LOW,
|
FAN_LOW,
|
||||||
|
@ -171,6 +178,21 @@ class AirzoneClimate(AirzoneEntity, ClimateEntity):
|
||||||
_attr_temperature_unit = UnitOfTemperature.CELSIUS
|
_attr_temperature_unit = UnitOfTemperature.CELSIUS
|
||||||
_enable_turn_on_off_backwards_compatibility = False
|
_enable_turn_on_off_backwards_compatibility = False
|
||||||
|
|
||||||
|
def _init_attributes(self) -> None:
|
||||||
|
"""Init common climate device attributes."""
|
||||||
|
self._attr_target_temperature_step = self.get_airzone_value(AZD_TEMP_STEP)
|
||||||
|
|
||||||
|
self._attr_hvac_modes = [
|
||||||
|
HVAC_MODE_LIB_TO_HASS[mode] for mode in self.get_airzone_value(AZD_MODES)
|
||||||
|
]
|
||||||
|
if HVACMode.OFF not in self._attr_hvac_modes:
|
||||||
|
self._attr_hvac_modes += [HVACMode.OFF]
|
||||||
|
|
||||||
|
if self.get_airzone_value(AZD_DOUBLE_SET_POINT):
|
||||||
|
self._attr_supported_features |= (
|
||||||
|
ClimateEntityFeature.TARGET_TEMPERATURE_RANGE
|
||||||
|
)
|
||||||
|
|
||||||
@callback
|
@callback
|
||||||
def _handle_coordinator_update(self) -> None:
|
def _handle_coordinator_update(self) -> None:
|
||||||
"""Update attributes when the coordinator updates."""
|
"""Update attributes when the coordinator updates."""
|
||||||
|
@ -193,7 +215,15 @@ class AirzoneClimate(AirzoneEntity, ClimateEntity):
|
||||||
self._attr_hvac_mode = HVACMode.OFF
|
self._attr_hvac_mode = HVACMode.OFF
|
||||||
self._attr_max_temp = self.get_airzone_value(AZD_TEMP_SET_MAX)
|
self._attr_max_temp = self.get_airzone_value(AZD_TEMP_SET_MAX)
|
||||||
self._attr_min_temp = self.get_airzone_value(AZD_TEMP_SET_MIN)
|
self._attr_min_temp = self.get_airzone_value(AZD_TEMP_SET_MIN)
|
||||||
self._attr_target_temperature = self.get_airzone_value(AZD_TEMP_SET)
|
if self.supported_features & ClimateEntityFeature.TARGET_TEMPERATURE_RANGE:
|
||||||
|
self._attr_target_temperature_high = self.get_airzone_value(
|
||||||
|
AZD_TEMP_SET_COOL_AIR
|
||||||
|
)
|
||||||
|
self._attr_target_temperature_low = self.get_airzone_value(
|
||||||
|
AZD_TEMP_SET_HOT_AIR
|
||||||
|
)
|
||||||
|
else:
|
||||||
|
self._attr_target_temperature = self.get_airzone_value(AZD_TEMP_SET)
|
||||||
|
|
||||||
|
|
||||||
class AirzoneDeviceClimate(AirzoneClimate):
|
class AirzoneDeviceClimate(AirzoneClimate):
|
||||||
|
@ -233,6 +263,19 @@ class AirzoneDeviceClimate(AirzoneClimate):
|
||||||
API_UNITS: TemperatureUnit.CELSIUS.value,
|
API_UNITS: TemperatureUnit.CELSIUS.value,
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
if ATTR_TARGET_TEMP_LOW in kwargs and ATTR_TARGET_TEMP_HIGH in kwargs:
|
||||||
|
params[API_SP_AIR_COOL] = {
|
||||||
|
API_VALUE: kwargs[ATTR_TARGET_TEMP_HIGH],
|
||||||
|
API_OPTS: {
|
||||||
|
API_UNITS: TemperatureUnit.CELSIUS.value,
|
||||||
|
},
|
||||||
|
}
|
||||||
|
params[API_SP_AIR_HEAT] = {
|
||||||
|
API_VALUE: kwargs[ATTR_TARGET_TEMP_LOW],
|
||||||
|
API_OPTS: {
|
||||||
|
API_UNITS: TemperatureUnit.CELSIUS.value,
|
||||||
|
},
|
||||||
|
}
|
||||||
await self._async_update_params(params)
|
await self._async_update_params(params)
|
||||||
|
|
||||||
if ATTR_HVAC_MODE in kwargs:
|
if ATTR_HVAC_MODE in kwargs:
|
||||||
|
@ -311,12 +354,7 @@ class AirzoneAidooClimate(AirzoneAidooEntity, AirzoneDeviceClimate):
|
||||||
super().__init__(coordinator, aidoo_id, aidoo_data)
|
super().__init__(coordinator, aidoo_id, aidoo_data)
|
||||||
|
|
||||||
self._attr_unique_id = aidoo_id
|
self._attr_unique_id = aidoo_id
|
||||||
self._attr_target_temperature_step = self.get_airzone_value(AZD_TEMP_STEP)
|
self._init_attributes()
|
||||||
self._attr_hvac_modes = [
|
|
||||||
HVAC_MODE_LIB_TO_HASS[mode] for mode in self.get_airzone_value(AZD_MODES)
|
|
||||||
]
|
|
||||||
if HVACMode.OFF not in self._attr_hvac_modes:
|
|
||||||
self._attr_hvac_modes += [HVACMode.OFF]
|
|
||||||
if (
|
if (
|
||||||
self.get_airzone_value(AZD_SPEED) is not None
|
self.get_airzone_value(AZD_SPEED) is not None
|
||||||
and self.get_airzone_value(AZD_SPEEDS) is not None
|
and self.get_airzone_value(AZD_SPEEDS) is not None
|
||||||
|
@ -402,12 +440,7 @@ class AirzoneGroupClimate(AirzoneGroupEntity, AirzoneDeviceGroupClimate):
|
||||||
super().__init__(coordinator, group_id, group_data)
|
super().__init__(coordinator, group_id, group_data)
|
||||||
|
|
||||||
self._attr_unique_id = group_id
|
self._attr_unique_id = group_id
|
||||||
self._attr_target_temperature_step = self.get_airzone_value(AZD_TEMP_STEP)
|
self._init_attributes()
|
||||||
self._attr_hvac_modes = [
|
|
||||||
HVAC_MODE_LIB_TO_HASS[mode] for mode in self.get_airzone_value(AZD_MODES)
|
|
||||||
]
|
|
||||||
if HVACMode.OFF not in self._attr_hvac_modes:
|
|
||||||
self._attr_hvac_modes += [HVACMode.OFF]
|
|
||||||
|
|
||||||
self._async_update_attrs()
|
self._async_update_attrs()
|
||||||
|
|
||||||
|
@ -425,12 +458,7 @@ class AirzoneInstallationClimate(AirzoneInstallationEntity, AirzoneDeviceGroupCl
|
||||||
super().__init__(coordinator, inst_id, inst_data)
|
super().__init__(coordinator, inst_id, inst_data)
|
||||||
|
|
||||||
self._attr_unique_id = inst_id
|
self._attr_unique_id = inst_id
|
||||||
self._attr_target_temperature_step = self.get_airzone_value(AZD_TEMP_STEP)
|
self._init_attributes()
|
||||||
self._attr_hvac_modes = [
|
|
||||||
HVAC_MODE_LIB_TO_HASS[mode] for mode in self.get_airzone_value(AZD_MODES)
|
|
||||||
]
|
|
||||||
if HVACMode.OFF not in self._attr_hvac_modes:
|
|
||||||
self._attr_hvac_modes += [HVACMode.OFF]
|
|
||||||
|
|
||||||
self._async_update_attrs()
|
self._async_update_attrs()
|
||||||
|
|
||||||
|
@ -448,12 +476,7 @@ class AirzoneZoneClimate(AirzoneZoneEntity, AirzoneDeviceClimate):
|
||||||
super().__init__(coordinator, system_zone_id, zone_data)
|
super().__init__(coordinator, system_zone_id, zone_data)
|
||||||
|
|
||||||
self._attr_unique_id = system_zone_id
|
self._attr_unique_id = system_zone_id
|
||||||
self._attr_target_temperature_step = self.get_airzone_value(AZD_TEMP_STEP)
|
self._init_attributes()
|
||||||
self._attr_hvac_modes = [
|
|
||||||
HVAC_MODE_LIB_TO_HASS[mode] for mode in self.get_airzone_value(AZD_MODES)
|
|
||||||
]
|
|
||||||
if HVACMode.OFF not in self._attr_hvac_modes:
|
|
||||||
self._attr_hvac_modes += [HVACMode.OFF]
|
|
||||||
|
|
||||||
self._async_update_attrs()
|
self._async_update_attrs()
|
||||||
|
|
||||||
|
|
|
@ -16,6 +16,8 @@ from homeassistant.components.climate import (
|
||||||
ATTR_HVAC_MODES,
|
ATTR_HVAC_MODES,
|
||||||
ATTR_MAX_TEMP,
|
ATTR_MAX_TEMP,
|
||||||
ATTR_MIN_TEMP,
|
ATTR_MIN_TEMP,
|
||||||
|
ATTR_TARGET_TEMP_HIGH,
|
||||||
|
ATTR_TARGET_TEMP_LOW,
|
||||||
ATTR_TARGET_TEMP_STEP,
|
ATTR_TARGET_TEMP_STEP,
|
||||||
DOMAIN as CLIMATE_DOMAIN,
|
DOMAIN as CLIMATE_DOMAIN,
|
||||||
FAN_AUTO,
|
FAN_AUTO,
|
||||||
|
@ -95,7 +97,8 @@ async def test_airzone_create_climates(hass: HomeAssistant) -> None:
|
||||||
assert state.attributes[ATTR_MAX_TEMP] == 30
|
assert state.attributes[ATTR_MAX_TEMP] == 30
|
||||||
assert state.attributes[ATTR_MIN_TEMP] == 15
|
assert state.attributes[ATTR_MIN_TEMP] == 15
|
||||||
assert state.attributes[ATTR_TARGET_TEMP_STEP] == API_DEFAULT_TEMP_STEP
|
assert state.attributes[ATTR_TARGET_TEMP_STEP] == API_DEFAULT_TEMP_STEP
|
||||||
assert state.attributes[ATTR_TEMPERATURE] == 22.0
|
assert state.attributes.get(ATTR_TARGET_TEMP_HIGH) == 22.0
|
||||||
|
assert state.attributes.get(ATTR_TARGET_TEMP_LOW) == 18.0
|
||||||
|
|
||||||
# Groups
|
# Groups
|
||||||
state = hass.states.get("climate.group")
|
state = hass.states.get("climate.group")
|
||||||
|
@ -576,6 +579,27 @@ async def test_airzone_climate_set_temp(hass: HomeAssistant) -> None:
|
||||||
assert state.state == HVACMode.HEAT
|
assert state.state == HVACMode.HEAT
|
||||||
assert state.attributes[ATTR_TEMPERATURE] == 20.5
|
assert state.attributes[ATTR_TEMPERATURE] == 20.5
|
||||||
|
|
||||||
|
# Aidoo Pro with Double Setpoint
|
||||||
|
with patch(
|
||||||
|
"homeassistant.components.airzone_cloud.AirzoneCloudApi.api_patch_device",
|
||||||
|
return_value=None,
|
||||||
|
):
|
||||||
|
await hass.services.async_call(
|
||||||
|
CLIMATE_DOMAIN,
|
||||||
|
SERVICE_SET_TEMPERATURE,
|
||||||
|
{
|
||||||
|
ATTR_ENTITY_ID: "climate.bron_pro",
|
||||||
|
ATTR_TARGET_TEMP_HIGH: 25.0,
|
||||||
|
ATTR_TARGET_TEMP_LOW: 20.0,
|
||||||
|
},
|
||||||
|
blocking=True,
|
||||||
|
)
|
||||||
|
|
||||||
|
state = hass.states.get("climate.bron_pro")
|
||||||
|
assert state.state == HVACMode.HEAT
|
||||||
|
assert state.attributes.get(ATTR_TARGET_TEMP_HIGH) == 25.0
|
||||||
|
assert state.attributes.get(ATTR_TARGET_TEMP_LOW) == 20.0
|
||||||
|
|
||||||
|
|
||||||
async def test_airzone_climate_set_temp_error(hass: HomeAssistant) -> None:
|
async def test_airzone_climate_set_temp_error(hass: HomeAssistant) -> None:
|
||||||
"""Test error when setting the target temperature."""
|
"""Test error when setting the target temperature."""
|
||||||
|
|
Loading…
Add table
Reference in a new issue