gilles/versatile_thermostat
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

FIX some testus. Some others are still KO

Browse Source
This commit is contained in:
Jean-Marc Collin
2024-03-03 20:35:38 +00:00
parent 156d19666c
commit 15c02e9722
6 changed files with 124 additions and 109 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
custom_components/versatile_thermostat/base_thermostat.py
View File

@@ -2719,3 +2719,6 @@ class BaseThermostat(ClimateEntity, RestoreEntity):
) )
else: else:
_LOGGER.debug("No preset_modes") _LOGGER.debug("No preset_modes")
if self._motion_on:
self._attr_preset_modes.append(PRESET_ACTIVITY)

15
tests/test_auto_fan_mode.py
View File

@@ -211,13 +211,14 @@ async def test_over_climate_auto_fan_mode_turbo_activation(
"custom_components.versatile_thermostat.underlyings.UnderlyingClimate.find_underlying_climate", "custom_components.versatile_thermostat.underlyings.UnderlyingClimate.find_underlying_climate",
return_value=fake_underlying_climate, return_value=fake_underlying_climate,
): ):
entry.add_to_hass(hass) entity = await create_thermostat(hass, entry, "climate.theoverclimatemockname")
await hass.config_entries.async_setup(entry.entry_id) # entry.add_to_hass(hass)
assert entry.state is ConfigEntryState.LOADED # await hass.config_entries.async_setup(entry.entry_id)
# assert entry.state is ConfigEntryState.LOADED
entity: ThermostatOverClimate = search_entity( #
hass, "climate.theoverclimatemockname", "climate" # entity: ThermostatOverClimate = search_entity(
) # hass, "climate.theoverclimatemockname", "climate"
# )
assert entity assert entity
assert isinstance(entity, ThermostatOverClimate) assert isinstance(entity, ThermostatOverClimate)

50
tests/test_auto_regulation.py
View File

@@ -52,18 +52,19 @@ async def test_over_climate_regulation(
"custom_components.versatile_thermostat.underlyings.UnderlyingClimate.find_underlying_climate", "custom_components.versatile_thermostat.underlyings.UnderlyingClimate.find_underlying_climate",
return_value=fake_underlying_climate, return_value=fake_underlying_climate,
): ):
entry.add_to_hass(hass) entity = await create_thermostat(hass, entry, "climate.theoverclimatemockname")
await hass.config_entries.async_setup(entry.entry_id) # entry.add_to_hass(hass)
assert entry.state is ConfigEntryState.LOADED # await hass.config_entries.async_setup(entry.entry_id)
# assert entry.state is ConfigEntryState.LOADED
def find_my_entity(entity_id) -> ClimateEntity: #
"""Find my new entity""" # def find_my_entity(entity_id) -> ClimateEntity:
component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN] # """Find my new entity"""
for entity in component.entities: # component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN]
if entity.entity_id == entity_id: # for entity in component.entities:
return entity # if entity.entity_id == entity_id:
# return entity
entity: ThermostatOverClimate = find_my_entity("climate.theoverclimatemockname") #
# entity: ThermostatOverClimate = find_my_entity("climate.theoverclimatemockname")
assert entity assert entity
assert isinstance(entity, ThermostatOverClimate) assert isinstance(entity, ThermostatOverClimate)
@@ -161,18 +162,19 @@ async def test_over_climate_regulation_ac_mode(
"custom_components.versatile_thermostat.underlyings.UnderlyingClimate.find_underlying_climate", "custom_components.versatile_thermostat.underlyings.UnderlyingClimate.find_underlying_climate",
return_value=fake_underlying_climate, return_value=fake_underlying_climate,
): ):
entry.add_to_hass(hass) entity = await create_thermostat(hass, entry, "climate.theoverclimatemockname")
await hass.config_entries.async_setup(entry.entry_id) # entry.add_to_hass(hass)
assert entry.state is ConfigEntryState.LOADED # await hass.config_entries.async_setup(entry.entry_id)
# assert entry.state is ConfigEntryState.LOADED
def find_my_entity(entity_id) -> ClimateEntity: #
"""Find my new entity""" # def find_my_entity(entity_id) -> ClimateEntity:
component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN] # """Find my new entity"""
for entity in component.entities: # component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN]
if entity.entity_id == entity_id: # for entity in component.entities:
return entity # if entity.entity_id == entity_id:
# return entity
entity: ThermostatOverClimate = find_my_entity("climate.theoverclimatemockname") #
# entity: ThermostatOverClimate = find_my_entity("climate.theoverclimatemockname")
assert entity assert entity
assert isinstance(entity, ThermostatOverClimate) assert isinstance(entity, ThermostatOverClimate)

75
tests/test_bugs.py
View File

@@ -380,18 +380,19 @@ async def test_bug_82(
"custom_components.versatile_thermostat.underlyings.UnderlyingClimate.find_underlying_climate", "custom_components.versatile_thermostat.underlyings.UnderlyingClimate.find_underlying_climate",
return_value=fake_underlying_climate, return_value=fake_underlying_climate,
) as mock_find_climate: ) as mock_find_climate:
entry.add_to_hass(hass) entity = await create_thermostat(hass, entry, "climate.theoverclimatemockname")
await hass.config_entries.async_setup(entry.entry_id) # entry.add_to_hass(hass)
assert entry.state is ConfigEntryState.LOADED # await hass.config_entries.async_setup(entry.entry_id)
# assert entry.state is ConfigEntryState.LOADED
def find_my_entity(entity_id) -> ClimateEntity: #
"""Find my new entity""" # def find_my_entity(entity_id) -> ClimateEntity:
component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN] # """Find my new entity"""
for entity in component.entities: # component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN]
if entity.entity_id == entity_id: # for entity in component.entities:
return entity # if entity.entity_id == entity_id:
# return entity
entity = find_my_entity("climate.theoverclimatemockname") #
# entity = find_my_entity("climate.theoverclimatemockname")
assert entity assert entity
@@ -490,18 +491,19 @@ async def test_bug_101(
) as mock_find_climate, patch( ) as mock_find_climate, patch(
"custom_components.versatile_thermostat.underlyings.UnderlyingClimate.set_hvac_mode" "custom_components.versatile_thermostat.underlyings.UnderlyingClimate.set_hvac_mode"
) as mock_underlying_set_hvac_mode: ) as mock_underlying_set_hvac_mode:
entry.add_to_hass(hass) entity = await create_thermostat(hass, entry, "climate.theoverclimatemockname")
await hass.config_entries.async_setup(entry.entry_id) # entry.add_to_hass(hass)
assert entry.state is ConfigEntryState.LOADED # await hass.config_entries.async_setup(entry.entry_id)
# assert entry.state is ConfigEntryState.LOADED
def find_my_entity(entity_id) -> ClimateEntity: #
"""Find my new entity""" # def find_my_entity(entity_id) -> ClimateEntity:
component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN] # """Find my new entity"""
for entity in component.entities: # component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN]
if entity.entity_id == entity_id: # for entity in component.entities:
return entity # if entity.entity_id == entity_id:
# return entity
entity = find_my_entity("climate.theoverclimatemockname") #
# entity = find_my_entity("climate.theoverclimatemockname")
assert entity assert entity
@@ -606,18 +608,19 @@ async def test_bug_272(
), patch( ), patch(
"homeassistant.core.ServiceRegistry.async_call" "homeassistant.core.ServiceRegistry.async_call"
) as mock_service_call: ) as mock_service_call:
entry.add_to_hass(hass) entity = await create_thermostat(hass, entry, "climate.theoverclimatemockname")
await hass.config_entries.async_setup(entry.entry_id) # entry.add_to_hass(hass)
assert entry.state is ConfigEntryState.LOADED # await hass.config_entries.async_setup(entry.entry_id)
# assert entry.state is ConfigEntryState.LOADED
def find_my_entity(entity_id) -> ClimateEntity: #
"""Find my new entity""" # def find_my_entity(entity_id) -> ClimateEntity:
component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN] # """Find my new entity"""
for entity in component.entities: # component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN]
if entity.entity_id == entity_id: # for entity in component.entities:
return entity # if entity.entity_id == entity_id:
# return entity
entity = find_my_entity("climate.theoverclimatemockname") #
# entity = find_my_entity("climate.theoverclimatemockname")
assert entity assert entity

15
tests/test_central_config.py
View File

@@ -80,13 +80,16 @@ async def test_add_a_central_config(hass: HomeAssistant, skip_hass_states_is_sta
}, },
) )
central_config_entry.add_to_hass(hass) entity = await create_thermostat(
await hass.config_entries.async_setup(central_config_entry.entry_id) hass, central_config_entry, "climate.thecentralconfigmockname"
assert central_config_entry.state is ConfigEntryState.LOADED
entity: ThermostatOverClimate = search_entity(
hass, "climate.thecentralconfigmockname", "climate"
) )
# central_config_entry.add_to_hass(hass)
# await hass.config_entries.async_setup(central_config_entry.entry_id)
# assert central_config_entry.state is ConfigEntryState.LOADED
#
# entity: ThermostatOverClimate = search_entity(
# hass, "climate.thecentralconfigmockname", "climate"
# )
assert entity is None assert entity is None

75
tests/test_start.py
View File

@@ -38,18 +38,19 @@ async def test_over_switch_full_start(hass: HomeAssistant, skip_hass_states_is_s
with patch( with patch(
"custom_components.versatile_thermostat.base_thermostat.BaseThermostat.send_event" "custom_components.versatile_thermostat.base_thermostat.BaseThermostat.send_event"
) as mock_send_event: ) as mock_send_event:
entry.add_to_hass(hass) entity = await create_thermostat(hass, entry, "climate.theoverswitchmockname")
await hass.config_entries.async_setup(entry.entry_id) # entry.add_to_hass(hass)
assert entry.state is ConfigEntryState.LOADED # await hass.config_entries.async_setup(entry.entry_id)
# assert entry.state is ConfigEntryState.LOADED
def find_my_entity(entity_id) -> ClimateEntity: #
"""Find my new entity""" # def find_my_entity(entity_id) -> ClimateEntity:
component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN] # """Find my new entity"""
for entity in component.entities: # component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN]
if entity.entity_id == entity_id: # for entity in component.entities:
return entity # if entity.entity_id == entity_id:
# return entity
entity: BaseThermostat = find_my_entity("climate.theoverswitchmockname") #
# entity: BaseThermostat = find_my_entity("climate.theoverswitchmockname")
assert entity assert entity
assert isinstance(entity, ThermostatOverSwitch) assert isinstance(entity, ThermostatOverSwitch)
@@ -108,18 +109,19 @@ async def test_over_climate_full_start(hass: HomeAssistant, skip_hass_states_is_
"custom_components.versatile_thermostat.underlyings.UnderlyingClimate.find_underlying_climate", "custom_components.versatile_thermostat.underlyings.UnderlyingClimate.find_underlying_climate",
return_value=fake_underlying_climate, return_value=fake_underlying_climate,
) as mock_find_climate: ) as mock_find_climate:
entry.add_to_hass(hass) entity = await create_thermostat(hass, entry, "climate.theoverclimatemockname")
await hass.config_entries.async_setup(entry.entry_id) # entry.add_to_hass(hass)
assert entry.state is ConfigEntryState.LOADED # await hass.config_entries.async_setup(entry.entry_id)
# assert entry.state is ConfigEntryState.LOADED
def find_my_entity(entity_id) -> ClimateEntity: #
"""Find my new entity""" # def find_my_entity(entity_id) -> ClimateEntity:
component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN] # """Find my new entity"""
for entity in component.entities: # component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN]
if entity.entity_id == entity_id: # for entity in component.entities:
return entity # if entity.entity_id == entity_id:
# return entity
entity = find_my_entity("climate.theoverclimatemockname") #
# entity = find_my_entity("climate.theoverclimatemockname")
assert entity assert entity
assert isinstance(entity, ThermostatOverClimate) assert isinstance(entity, ThermostatOverClimate)
@@ -174,18 +176,19 @@ async def test_over_4switch_full_start(hass: HomeAssistant, skip_hass_states_is_
with patch( with patch(
"custom_components.versatile_thermostat.base_thermostat.BaseThermostat.send_event" "custom_components.versatile_thermostat.base_thermostat.BaseThermostat.send_event"
) as mock_send_event: ) as mock_send_event:
entry.add_to_hass(hass) entity = await create_thermostat(hass, entry, "climate.theover4switchmockname")
await hass.config_entries.async_setup(entry.entry_id) # entry.add_to_hass(hass)
assert entry.state is ConfigEntryState.LOADED # await hass.config_entries.async_setup(entry.entry_id)
# assert entry.state is ConfigEntryState.LOADED
def find_my_entity(entity_id) -> ClimateEntity: #
"""Find my new entity""" # def find_my_entity(entity_id) -> ClimateEntity:
component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN] # """Find my new entity"""
for entity in component.entities: # component: EntityComponent[ClimateEntity] = hass.data[CLIMATE_DOMAIN]
if entity.entity_id == entity_id: # for entity in component.entities:
return entity # if entity.entity_id == entity_id:
# return entity
entity: BaseThermostat = find_my_entity("climate.theover4switchmockname") #
# entity: BaseThermostat = find_my_entity("climate.theover4switchmockname")
assert entity assert entity