Use assert isinstance() instead of cast()

Author: bouwew

plugwise/helper.py

@@ -78,8 +78,6 @@
     version_to_model,
 )
 
-# from typing import cast
-
 
 def check_model(name: str | None, vendor_name: str | None) -> str | None:
     """Model checking before using version_to_model."""
@@ -863,13 +861,10 @@ def _appliance_measurements(
                         data["select_dhw_mode"] = appl_p_loc.text
                     case _ as meas_rn if meas_rn in BINARY_SENSORS:
                         bs_key = cast(BinarySensorType, meas_rn)
-                        bs_value = cast(
-                            bool,
-                            format_measure(
-                                appl_p_loc.text,
-                                getattr(attrs, ATTR_UNIT_OF_MEASUREMENT),
-                            ),
+                        bs_value = format_measure(
+                            appl_p_loc.text, getattr(attrs, ATTR_UNIT_OF_MEASUREMENT)
                         )
+                        assert isinstance(bs_value, bool)
                         data["binary_sensors"][bs_key] = bs_value
                     case _ as meas_rn if meas_rn in SENSORS:
                         s_key = cast(SensorType, meas_rn)
@@ -893,22 +888,17 @@ def _appliance_measurements(
                             ]
                     case _ as meas_rn if meas_rn in SWITCHES:
                         sw_key = cast(SwitchType, meas_rn)
-                        sw_value = cast(
-                            bool,
-                            format_measure(
-                                appl_p_loc.text,
-                                getattr(attrs, ATTR_UNIT_OF_MEASUREMENT),
-                            ),
+                        value = format_measure(
+                            appl_p_loc.text, getattr(attrs, ATTR_UNIT_OF_MEASUREMENT)
                         )
+                        sw_value = cast(bool, value)
                         data["switches"][sw_key] = sw_value
                     case "c_heating_state":
-                        data["c_heating_state"] = cast(
-                            bool,
-                            format_measure(
-                                appl_p_loc.text,
-                                getattr(attrs, ATTR_UNIT_OF_MEASUREMENT),
-                            ),
+                        value = format_measure(
+                            appl_p_loc.text, getattr(attrs, ATTR_UNIT_OF_MEASUREMENT)
                         )
+                        assert isinstance(value, bool)
+                        data["c_heating_state"] = value
                     case "elga_status_code":
                         data["elga_status_code"] = int(appl_p_loc.text)