Update all in [..., ...] constructs to using set()

Author: bouwew

plugwise/helper.py

@@ -214,7 +214,7 @@ def __init__(self) -> None:
         self._thermo_locs: dict[str, ThermoLoc] = {}
         ###################################################################
         # '_cooling_enabled' can refer to the state of the Elga heatpump
-        # connected to an Anna. For Elga, 'elga_status_code' in [8, 9]
+        # connected to an Anna. For Elga, 'elga_status_code' in (8, 9)
         # means cooling mode is available, next to heating mode.
         # 'elga_status_code' = 8 means cooling is active, 9 means idle.
         #
@@ -523,7 +523,7 @@ def _get_measurement_data(self, dev_id: str) -> DeviceData:
                     # Techneco Elga has cooling-capability
                     self._cooling_present = True
                     data["model"] = "Generic heater/cooler"
-                    self._cooling_enabled = data["elga_status_code"] in [8, 9]
+                    self._cooling_enabled = data["elga_status_code"] in (8, 9)
                     data["binary_sensors"]["cooling_state"] = self._cooling_active = (
                         data["elga_status_code"] == 8
                     )
@@ -590,7 +590,7 @@ def _appliance_measurements(
                         data["select_dhw_mode"] = appl_p_loc.text
                     case _ as measurement if measurement in BINARY_SENSORS:
                         bs_key = cast(BinarySensorType, measurement)
-                        bs_value = appl_p_loc.text in ["on", "true"]
+                        bs_value = appl_p_loc.text in ("on", "true")
                         data["binary_sensors"][bs_key] = bs_value
                     case _ as measurement if measurement in SENSORS:
                         s_key = cast(SensorType, measurement)
@@ -600,11 +600,11 @@ def _appliance_measurements(
                         data["sensors"][s_key] = s_value
                     case _ as measurement if measurement in SWITCHES:
                         sw_key = cast(SwitchType, measurement)
-                        sw_value = appl_p_loc.text in ["on", "true"]
+                        sw_value = appl_p_loc.text in ("on", "true")
                         data["switches"][sw_key] = sw_value
                     case _ as measurement if measurement in SPECIALS:
                         sp_key = cast(SpecialType, measurement)
-                        sp_value = appl_p_loc.text in ["on", "true"]
+                        sp_value = appl_p_loc.text in ("on", "true")
                         data[sp_key] = sp_value
                     case "elga_status_code":
                         data["elga_status_code"] = int(appl_p_loc.text)

plugwise/legacy/helper.py

@@ -344,7 +344,7 @@ def _appliance_measurements(
                 match measurement:
                     case _ as measurement if measurement in BINARY_SENSORS:
                         bs_key = cast(BinarySensorType, measurement)
-                        bs_value = appl_p_loc.text in ["on", "true"]
+                        bs_value = appl_p_loc.text in ("on", "true")
                         data["binary_sensors"][bs_key] = bs_value
                     case _ as measurement if measurement in SENSORS:
                         s_key = cast(SensorType, measurement)
@@ -354,11 +354,11 @@ def _appliance_measurements(
                         data["sensors"][s_key] = s_value
                     case _ as measurement if measurement in SWITCHES:
                         sw_key = cast(SwitchType, measurement)
-                        sw_value = appl_p_loc.text in ["on", "true"]
+                        sw_value = appl_p_loc.text in ("on", "true")
                         data["switches"][sw_key] = sw_value
                     case _ as measurement if measurement in SPECIALS:
                         sp_key = cast(SpecialType, measurement)
-                        sp_value = appl_p_loc.text in ["on", "true"]
+                        sp_value = appl_p_loc.text in ("on", "true")
                         data[sp_key] = sp_value
 
             i_locator = f'.//logs/interval_log[type="{measurement}"]/period/measurement'

plugwise/legacy/smile.py

@@ -188,7 +188,7 @@ async def set_schedule_state(self, _: str, state: str, name: str | None) -> None
         Determined from - DOMAIN_OBJECTS.
         Used in HA Core to set the hvac_mode: in practice switch between schedule on - off.
         """
-        if state not in ["on", "off"]:
+        if state not in ("on", "off"):
             raise PlugwiseError("Plugwise: invalid schedule state.")
 
         # Handle no schedule-name / Off-schedule provided

plugwise/smile.py

@@ -266,7 +266,7 @@ async def set_schedule_state(
         Used in HA Core to set the hvac_mode: in practice switch between schedule on - off.
         """
         # Input checking
-        if new_state not in ["on", "off"]:
+        if new_state not in ("on", "off"):
             raise PlugwiseError("Plugwise: invalid schedule state.")
 
         # Translate selection of Off-schedule-option to disabling the active schedule