Send requests to PowerDistributor even if target power hasn't changed

This is required until
#293 is
done.

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

Author: shsms

src/frequenz/sdk/actor/_power_managing/_power_managing_actor.py

@@ -181,7 +181,19 @@ async def _run(self) -> None:
                 proposal = selected.value
                 if proposal.battery_ids not in self._bound_tracker_tasks:
                     self._add_bounds_tracker(proposal.battery_ids)
-                await self._send_updated_target_power(proposal.battery_ids, proposal)
+
+                # TODO: must_send=True forces a new request to # pylint: disable=fixme
+                # be sent to the PowerDistributor, even if there's no change in power.
+                #
+                # This is needed because requests would expire in the microgrid service
+                # otherwise.
+                #
+                # This can be removed as soon as
+                # https://github.com/frequenz-floss/frequenz-sdk-python/issues/293 is
+                # implemented.
+                await self._send_updated_target_power(
+                    proposal.battery_ids, proposal, must_send=True
+                )
                 await self._send_reports(proposal.battery_ids)
 
             elif selected_from(selected, self._bounds_subscription_receiver):

tests/actor/_power_managing/test_matryoshka.py

@@ -31,6 +31,7 @@ def test_tgt_power(
         power: float | None,
         bounds: tuple[float | None, float | None],
         expected: float | None,
+        must_send: bool = False,
     ) -> None:
         nonlocal call_count
         call_count += 1
@@ -47,6 +48,7 @@ def test_tgt_power(
                 priority=priority,
             ),
             system_bounds,
+            must_send,
         )
         assert tgt_power == (
             Power.from_watts(expected) if expected is not None else None
@@ -72,6 +74,9 @@ def test_bounds(
     test_bounds(priority=1, expected_power=25.0, expected_bounds=(25.0, 50.0))
 
     test_tgt_power(priority=1, power=20.0, bounds=(20.0, 50.0), expected=None)
+    test_tgt_power(
+        priority=1, power=20.0, bounds=(20.0, 50.0), expected=25.0, must_send=True
+    )
     test_bounds(priority=1, expected_power=25.0, expected_bounds=(25.0, 50.0))
 
     test_tgt_power(priority=3, power=10.0, bounds=(10.0, 15.0), expected=15.0)