FIX #159 - Doesn't send target temp if VTherm is off

custom_components/versatile_thermostat/config_flow.py

@@ -364,7 +364,7 @@ class VersatileThermostatBaseConfigFlow(FlowHandler):
             schema = schema_ac_or_not
         elif user_input and user_input.get(CONF_USE_PRESETS_CENTRAL_CONFIG) is False:
             next_step = self.async_step_spec_presets
-            schema = schema_ac_or_not
+            schema = STEP_PRESETS_DATA_SCHEMA
 
         return await self.generic_step("presets", schema, user_input, next_step)
 

custom_components/versatile_thermostat/thermostat_climate.py

@@ -136,6 +136,11 @@ class ThermostatOverClimate(BaseThermostat):
 
     async def _send_regulated_temperature(self, force=False):
         """Sends the regulated temperature to all underlying"""
+
+        if self.hvac_mode == HVACMode.OFF:
+            _LOGGER.debug("%s - don't send regulated temperature cause VTherm is off ")
+            return
+
         _LOGGER.info(
             "%s - Calling ThermostatClimate._send_regulated_temperature force=%s",
             self,

tests/test_bugs.py

@@ -629,13 +629,18 @@ async def test_bug_272(
         assert mock_service_call.call_count == 0
 
         # Set the hvac_mode to HEAT
-        entity.async_set_hvac_mode(HVACMode.HEAT)
+        await entity.async_set_hvac_mode(HVACMode.HEAT)
 
         # In the accepted interval
         await entity.async_set_temperature(temperature=17)
-        assert mock_service_call.call_count == 1
+        assert mock_service_call.call_count == 2
         mock_service_call.assert_has_calls(
             [
+                call.async_call(
+                    "climate",
+                    SERVICE_SET_HVAC_MODE,
+                    {"entity_id": "climate.mock_climate", "hvac_mode": HVACMode.HEAT},
+                ),
                 call.async_call(
                     "climate",
                     SERVICE_SET_TEMPERATURE,