-
Notifications
You must be signed in to change notification settings - Fork 20
Closed as not planned
Closed as not planned
Copy link
Labels
part:power-distributionpart:testsAffects the unit, integration and performance (benchmarks) testsAffects the unit, integration and performance (benchmarks) teststype:bugSomething isn't workingSomething isn't working
Milestone
Description
What happened?
A test if failing when testing with the latest supported dependency versions:
$ nox -R -s pytest_max -- -vvsx tests/actor/test_battery_status.py::TestBatteryStatusRecovery::test_critical_error
...
# --- inverter warning error (keeps working) ---
await self._send_healthy_battery(mock_microgrid)
await self._send_warning_error_inverter(mock_microgrid)
> assert await recv_timeout(status_receiver, timeout=0.1) is _Timeout
E assert <Status.NOT_WORKING: 0> is _Timeout
tests/actor/test_battery_status.py:917: AssertionError
------------------------------------------------------------------------------------ Captured log call -------------------------------------------------------------------------------------
WARNING frequenz.sdk.actor.power_distributing._battery_status:_battery_status.py:272 Battery 9 stopped sending data, last timestamp: 2023-12-08 22:18:17.408082+00:00
WARNING frequenz.sdk.actor.power_distributing._battery_status:_battery_status.py:281 Inverter 8 stopped sending data, last timestamp: 2023-12-08 22:18:17.408038+00:00
WARNING frequenz.sdk.actor.power_distributing._battery_status:_battery_status.py:432 Component 9 has critical error: code: ERROR_CODE_BLOCK_ERROR
level: ERROR_LEVEL_CRITICAL
...Testing with -s pytest_min passes.
What did you expect instead?
Test passes with the latest dependency versions.
Affected version(s)
v0.25.1
Affected part(s)
Battery power distribution (part:power-distribution), Unit, integration and performance tests (part:tests)
Extra information
It is not clear to me if we want to investigate this, as this version will go out of support soon, and the currently running programs should be able to stick to the working versions of the dependencies.
At the time of writing, this is the diff between the pip freeze for pytest_min and pytest_max:
--- /tmp/freeze.min.txt 2023-12-08 23:20:54.649865044 +0100
+++ /tmp/freeze.max.txt 2023-12-08 23:21:11.920065594 +0100
@@ -1 +1 @@
-anyio==3.7.1
+anyio==4.1.0
@@ -13,3 +13,3 @@
-frequenz-api-common==0.3.0
-frequenz-api-microgrid==0.15.1
-frequenz-channels==0.16.0
+frequenz-api-common==0.3.1
+frequenz-api-microgrid==0.15.2
+frequenz-channels==0.16.1
@@ -20 +20 @@
-google-api-python-client==2.71.0
+google-api-python-client==2.110.0
@@ -24,2 +24,2 @@
-grpcio==1.54.2
-grpcio-tools==1.54.2
+grpcio==1.60.0
+grpcio-tools==1.60.0
@@ -38 +38 @@
-networkx==2.8
+networkx==3.2.1
@@ -40 +40 @@
-numpy==1.24.2
+numpy==1.26.2
@@ -46 +46 @@
-protobuf==4.21.6
+protobuf==4.25.1
@@ -49 +49 @@
-pydantic==1.9.0
+pydantic==1.10.13
@@ -65,2 +65,2 @@
-tqdm==4.38.0
-typing_extensions==4.4.0
+tqdm==4.66.1
+typing_extensions==4.8.0
@@ -71 +71 @@
-watchfiles==0.15.0
+watchfiles==0.19.0Metadata
Metadata
Assignees
Labels
part:power-distributionpart:testsAffects the unit, integration and performance (benchmarks) testsAffects the unit, integration and performance (benchmarks) teststype:bugSomething isn't workingSomething isn't working
Type
Projects
Status
Done