Change get_components_data to get data for working batteries only

This method is also used for preprocessing the request to check if
request is correct before adding it to the main task queue.
In methods: `_get_upper_bound` and `get_lower_bound`.

Signed-off-by: ela-kotulska-frequenz <[email protected]>

Author: ela-kotulska-frequenz

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

@@ -266,7 +266,7 @@ async def run(self) -> None:
 
             try:
                 pairs_data: List[InvBatPair] = self._get_components_data(
-                    self._all_battery_status.get_working_batteries(request.batteries)
+                    request.batteries
                 )
             except KeyError as err:
                 await user.channel.send(Error(request, str(err)))
@@ -548,8 +548,8 @@ def _get_components_data(self, batteries: Set[int]) -> List[InvBatPair]:
             Pairs of battery and adjacent inverter data.
         """
         pairs_data: List[InvBatPair] = []
-
-        for battery_id in batteries:
+        working_batteries = self._all_battery_status.get_working_batteries(batteries)
+        for battery_id in working_batteries:
             if battery_id not in self._battery_receivers:
                 raise KeyError(
                     f"No battery {battery_id}, "