Fix warnings in power distributing actor tests

The issue was caused by missing calls to the cleanup() method
of the mock microgrid, resulting in warnings during test execution.
The warnings occurred because the actor background service
attempted to cancel tasks when the event loop was already
closed upon test completion.

Signed-off-by: Daniel Zullo <[email protected]>

Author: daniel-zullo-frequenz

tests/actor/power_distributing/test_power_distributing.py

@@ -150,6 +150,8 @@ async def test_power_distributor_one_user(self, mocker: MockerFixture) -> None:
         assert result.excess_power.isclose(Power.from_watts(200.0))
         assert result.request == request
 
+        await mockgrid.cleanup()
+
     async def test_power_distributor_exclusion_bounds(
         self, mocker: MockerFixture
     ) -> None:
@@ -242,6 +244,8 @@ async def test_power_distributor_exclusion_bounds(
             assert result.bounds == PowerBounds(-1000, -600, 600, 1000)
             assert result.request == request
 
+        await mockgrid.cleanup()
+
     # pylint: disable=too-many-locals
     async def test_two_batteries_one_inverters(self, mocker: MockerFixture) -> None:
         """Test if power distribution works with two batteries connected to one inverter."""
@@ -312,6 +316,8 @@ async def test_two_batteries_one_inverters(self, mocker: MockerFixture) -> None:
             assert result.excess_power.isclose(Power.from_watts(700.0))
             assert result.request == request
 
+        await mockgrid.cleanup()
+
     async def test_two_batteries_one_broken_one_inverters(
         self, mocker: MockerFixture
     ) -> None:
@@ -393,6 +399,8 @@ async def test_two_batteries_one_broken_one_inverters(
                 result.msg == "No data for at least one of the given batteries {9, 19}"
             )
 
+        await mockgrid.cleanup()
+
     async def test_battery_two_inverters(self, mocker: MockerFixture) -> None:
         """Test if power distribution works with two inverters for one battery."""
         gen = GraphGenerator()
@@ -464,6 +472,8 @@ async def test_battery_two_inverters(self, mocker: MockerFixture) -> None:
             assert result.excess_power.isclose(Power.from_watts(200.0))
             assert result.request == request
 
+        await mockgrid.cleanup()
+
     async def test_two_batteries_three_inverters(self, mocker: MockerFixture) -> None:
         """Test if power distribution works with two batteries connected to three inverters."""
         gen = GraphGenerator()
@@ -540,6 +550,8 @@ async def test_two_batteries_three_inverters(self, mocker: MockerFixture) -> Non
             assert result.excess_power.isclose(Power.from_watts(200.0))
             assert result.request == request
 
+        await mockgrid.cleanup()
+
     async def test_two_batteries_one_inverter_different_exclusion_bounds_2(
         self, mocker: MockerFixture
     ) -> None:
@@ -625,6 +637,8 @@ async def test_two_batteries_one_inverter_different_exclusion_bounds_2(
             assert result.request == request
             assert result.bounds == PowerBounds(-1000, -500, 500, 1000)
 
+        await mockgrid.cleanup()
+
     async def test_two_batteries_one_inverter_different_exclusion_bounds(
         self, mocker: MockerFixture
     ) -> None:
@@ -712,6 +726,8 @@ async def test_two_batteries_one_inverter_different_exclusion_bounds(
             # each inverter is bounded at 500
             assert result.bounds == PowerBounds(-500, -400, 400, 500)
 
+        await mockgrid.cleanup()
+
     async def test_connected_but_not_requested_batteries(
         self, mocker: MockerFixture
     ) -> None:
@@ -785,6 +801,8 @@ async def test_connected_but_not_requested_batteries(
             )
             assert err_msg is not None
 
+        await mockgrid.cleanup()
+
     async def test_battery_soc_nan(self, mocker: MockerFixture) -> None:
         """Test if battery with SoC==NaN is not used."""
         mockgrid = MockMicrogrid(grid_meter=False)
@@ -847,6 +865,8 @@ async def test_battery_soc_nan(self, mocker: MockerFixture) -> None:
         assert result.excess_power.isclose(Power.from_watts(700.0))
         assert result.request == request
 
+        await mockgrid.cleanup()
+
     async def test_battery_capacity_nan(self, mocker: MockerFixture) -> None:
         """Test battery with capacity set to NaN is not used."""
         mockgrid = MockMicrogrid(grid_meter=False)
@@ -902,6 +922,8 @@ async def test_battery_capacity_nan(self, mocker: MockerFixture) -> None:
         assert result.excess_power.isclose(Power.from_watts(700.0))
         assert result.request == request
 
+        await mockgrid.cleanup()
+
     async def test_battery_power_bounds_nan(self, mocker: MockerFixture) -> None:
         """Test battery with power bounds set to NaN is not used."""
         mockgrid = MockMicrogrid(grid_meter=False)
@@ -972,6 +994,8 @@ async def test_battery_power_bounds_nan(self, mocker: MockerFixture) -> None:
         assert result.excess_power.isclose(Power.from_watts(200.0))
         assert result.request == request
 
+        await mockgrid.cleanup()
+
     async def test_power_distributor_invalid_battery_id(
         self, mocker: MockerFixture
     ) -> None:
@@ -1017,6 +1041,8 @@ async def test_power_distributor_invalid_battery_id(
         err_msg = re.search(r"No battery 100, available batteries:", result.msg)
         assert err_msg is not None
 
+        await mockgrid.cleanup()
+
     async def test_power_distributor_one_user_adjust_power_consume(
         self, mocker: MockerFixture
     ) -> None:
@@ -1067,6 +1093,8 @@ async def test_power_distributor_one_user_adjust_power_consume(
         assert result.request == request
         assert result.bounds.inclusion_upper == 1000
 
+        await mockgrid.cleanup()
+
     async def test_power_distributor_one_user_adjust_power_supply(
         self, mocker: MockerFixture
     ) -> None:
@@ -1117,6 +1145,8 @@ async def test_power_distributor_one_user_adjust_power_supply(
         assert result.request == request
         assert result.bounds.inclusion_lower == -1000
 
+        await mockgrid.cleanup()
+
     async def test_power_distributor_one_user_adjust_power_success(
         self, mocker: MockerFixture
     ) -> None:
@@ -1167,6 +1197,8 @@ async def test_power_distributor_one_user_adjust_power_success(
         assert result.excess_power.isclose(Power.zero(), abs_tol=1e-9)
         assert result.request == request
 
+        await mockgrid.cleanup()
+
     async def test_not_all_batteries_are_working(self, mocker: MockerFixture) -> None:
         """Test if power distribution works if not all batteries are working."""
         mockgrid = MockMicrogrid(grid_meter=False)
@@ -1215,3 +1247,5 @@ async def test_not_all_batteries_are_working(self, mocker: MockerFixture) -> Non
             assert result.excess_power.isclose(Power.from_watts(700.0))
             assert result.succeeded_power.isclose(Power.from_watts(500.0))
             assert result.request == request
+
+        await mockgrid.cleanup()