Merge pull request #517 from plugwise/schedule_fix

Bugfix for core 103251

Author: CoMPaTech

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## Versions from 0.40 and up
 
+## v0.44.1
+
+- Bugfix: ensure selecting schedule works correctly
+
 ## v0.44.0
 
 - New Feature: implement device_registry cleaning, solution to [Core Issue #102926](https://github.com/home-assistant/core/issues/102926)

custom_components/plugwise/manifest.json

@@ -9,5 +9,5 @@
   "iot_class": "local_polling",
   "loggers": ["plugwise"],
   "requirements": ["plugwise==0.34.4"],
-  "version": "0.44.0"
+  "version": "0.44.1"
 }

custom_components/plugwise/select.py

@@ -46,7 +46,7 @@ class PlugwiseSelectEntityDescription(
         key="select_schedule",
         translation_key="thermostat_schedule",
         icon="mdi:calendar-clock",
-        command=lambda api, loc, opt: api.set_schedule_state(loc, opt, STATE_ON),
+        command=lambda api, loc, opt: api.set_schedule_state(loc, STATE_ON, opt),
         options_key="available_schedules",
     ),
     PlugwiseSelectEntityDescription(

tests/components/plugwise/test_select.py

@@ -38,7 +38,9 @@ async def test_adam_change_select_entity(
 
     assert mock_smile_adam.set_schedule_state.call_count == 1
     mock_smile_adam.set_schedule_state.assert_called_with(
-        "c50f167537524366a5af7aa3942feb1e", "Badkamer Schema", "on"
+        "c50f167537524366a5af7aa3942feb1e",
+        "on",
+        "Badkamer Schema",
     )