Fix for thermostat operating mode

[DannyDeGaspari]

Thermostat operating mode not handled correctly. The modes are : disabled, manual, automatic and sleep timer. Whenever the preset is changed (safe, night, day, comfort) the operating mode is forced to go to automatic mode. This fix retains the operating mode when the preset is changed.

@Cereal2nd, I would also like to add following parameters to be available in home assistant:

• thermostat operating mode (disabled, manual, automatic and sleep timer)
• module program mode (none, winter, summer, holiday)

But the Climate entity does not have such properties, what would be the best way to add these parameters ? Is adding 2 extra channels to the module a good idea ? One for the operating mode and one for the program mode.

[DannyDeGaspari]

Does every module that supports the modulestqtusmessage2 have the sleep timer value?

I'll check for all modules that are listed in this message.

Same question for the tempsensorstatusmessage.

I did only a bug fix in here and did not add new code or functionality.

Could you write a couple of simple Testcases that implement the changes in this PR?

I'll try, but I will have to figure out how to do this for this specific message.

[Cereal2nd]

I deed it's only the modulestatusmessage2 that needs checking

For the tests there are some good simple examples in there, the one for the modulename is a good example and is the same idea as this one.

[Cereal2nd]

thanks for adding the testcases, we are almost there.

About the question in hass, i think we should do these things with extra state attributes

[Cereal2nd]

Thanks for the help

[DannyDeGaspari]

About the question in hass, i think we should do these things with extra state attributes

Does this require changes on the Hass side ? If yes, can you help here or at least guide me into it ?

BTW: during my check on the ModuleStatusMessage I noticed there is still another message variant needed for the VMBGP4PIR module. I own such a module and will implement support for it.

[Cereal2nd]

Yes it requires changes on the Hass side.

So what attributes exactly and how can we get these from the current channel.

If I know this it's easy.

[DannyDeGaspari]

There are 2 attributes:

• "Operating mode": stored in channel._cstatus variable and get from
  def get_climate_mode(self) -> str: No set function defined yet. Should I create it ?
• "Selected program": only decoded in the message itself as a first step, further implementation needed.

[DannyDeGaspari]

@Cereal2nd, Just to let you know that I'm still continuing with further implementations of the VMBGP4PIR module, the thermostat run mode and selected program. But did not had too much spare time past week...