More clean up

Author: bouwew

plugwise/__init__.py

@@ -409,7 +409,7 @@ async def set_temperature_offset(self, dev_id: str, offset: float) -> None:
             ) from exc  # pragma no cover
 
     async def set_switch_state(
-        self, appl_id: str, members: list[str] | None, model: str, state: str
+        self, appl_id: str, model: str, state: str
     ) -> bool:
         """Set the given State of the relevant Switch.
 
@@ -423,7 +423,7 @@ async def set_switch_state(
 
         try:
             return await self._smile_api.set_switch_state(
-                appl_id, members, model, state
+                appl_id, model, state
             )
         except ConnectionFailedError as exc:
             raise ConnectionFailedError(

plugwise/legacy/smile.py

@@ -216,7 +216,7 @@ async def set_schedule_state(
         await self.call_request(uri, method="put", data=data)
 
     async def set_switch_state(
-        self, appl_id: str, members: list[str] | None, model: str, state: str
+        self, appl_id: str, model: str, state: str
     ) -> bool:
         """Set the given state of the relevant switch.
 

plugwise/smile.py

@@ -394,7 +394,7 @@ def determine_contexts(self, loc_id: str, state: str, sched_id: str) -> str:
         return str(etree.tostring(contexts, encoding="unicode").rstrip())
 
     async def set_switch_state(
-        self, appl_id: str, members: list[str] | None, model: str, state: str
+        self, appl_id: str, model: str, state: str
     ) -> bool:
         """Set the given state of the relevant Switch.
 

tests/test_init.py

@@ -622,7 +622,7 @@ async def tinker_reboot(self, api, unhappy=False):
 
     @pytest.mark.asyncio
     async def tinker_switch(
-        self, api, dev_id=None, members=None, model="relay", unhappy=False
+        self, api, dev_id=None, model="relay", unhappy=False
     ):
         """Turn a Switch on and off to test functionality."""
         _LOGGER.info("Asserting modifying settings for switch devices:")
@@ -632,7 +632,7 @@ async def tinker_switch(
         for new_state in ["off", "on", "off"]:
             _LOGGER.info("- Switching %s", new_state)
             try:
-                result = await api.set_switch_state(dev_id, members, model, new_state)
+                result = await api.set_switch_state(dev_id, model, new_state)
                 if result == convert[new_state]:
                     tinker_switch_passed = True
                     _LOGGER.info("  + tinker_switch worked as intended")
@@ -653,14 +653,14 @@ async def tinker_switch(
 
     @pytest.mark.asyncio
     async def tinker_switch_bad_input(
-        self, api, dev_id=None, members=None, model="relay", unhappy=False
+        self, api, dev_id=None, model="relay", unhappy=False
     ):
         """Enter a wrong state as input to toggle a Switch."""
         _LOGGER.info("Test entering bad input set_switch_state:")
         _LOGGER.info("- Devices (%s):", dev_id)
         new_state = "false"
         try:
-            await api.set_switch_state(dev_id, members, model, new_state)
+            await api.set_switch_state(dev_id, model, new_state)
         except pw_exceptions.PlugwiseError:
             _LOGGER.info("  + failed input-check as expected")
             return True  # test is pass!