Fix EVCharger status identification function

Earlier it was not consider all possible operational states.

Signed-off-by: Sahas Subramanian <[email protected]>

Author: shsms

src/frequenz/sdk/actor/power_distributing/_component_status/_ev_charger_status_tracker.py

@@ -75,14 +75,17 @@ def start(self) -> None:
         self._tasks.add(asyncio.create_task(self._run()))
 
     def _is_working(self, ev_data: EVChargerData) -> bool:
-        """Return whether the given data indicates that the component is working."""
-        return (
-            ev_data.cable_state == EVChargerCableState.EV_PLUGGED
-            and ev_data.component_state
-            in (
-                EVChargerComponentState.READY,
-                EVChargerComponentState.CHARGING,
-            )
+        """Return whether the given EV charger can be assigned power.
+
+        This is True when an EV is connected and the charger is in a healthy state.
+        """
+        return ev_data.cable_state in (
+            EVChargerCableState.EV_PLUGGED,
+            EVChargerCableState.EV_LOCKED,
+        ) and ev_data.component_state in (
+            EVChargerComponentState.READY,
+            EVChargerComponentState.CHARGING,
+            EVChargerComponentState.DISCHARGING,
         )
 
     def _is_stale(self, ev_data: EVChargerData) -> bool:

src/frequenz/sdk/microgrid/component/_component_data.py

@@ -478,9 +478,10 @@ def is_ev_connected(self) -> bool:
             When the charger is not in an error state, whether an EV is connected to
                 the charger.
         """
-        return self.component_state not in (
-            EVChargerComponentState.AUTHORIZATION_REJECTED,
-            EVChargerComponentState.ERROR,
+        return self.component_state in (
+            EVChargerComponentState.READY,
+            EVChargerComponentState.CHARGING,
+            EVChargerComponentState.DISCHARGING,
         ) and self.cable_state in (
             EVChargerCableState.EV_LOCKED,
             EVChargerCableState.EV_PLUGGED,