Skip to content

Commit ac913e7

Browse files
committed
Remove unused class FakeSelect
Signed-off-by: Jack <[email protected]>
1 parent 996b40e commit ac913e7

File tree

2 files changed

+2
-34
lines changed

2 files changed

+2
-34
lines changed

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -198,11 +198,11 @@ def __init__( # pylint: disable=too-many-arguments
198198

199199
self._battery: _ComponentStreamStatus = _ComponentStreamStatus(
200200
battery_id,
201-
data_recv_timer=Timer.timeout(timedelta(max_data_age_sec)),
201+
data_recv_timer=Timer.timeout(timedelta(seconds=max_data_age_sec)),
202202
)
203203
self._inverter: _ComponentStreamStatus = _ComponentStreamStatus(
204204
inverter_id,
205-
data_recv_timer=Timer.timeout(timedelta(max_data_age_sec)),
205+
data_recv_timer=Timer.timeout(timedelta(seconds=max_data_age_sec)),
206206
)
207207

208208
# Select needs receivers that can be get in async way only.

tests/actor/test_battery_status.py

Lines changed: 0 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -154,38 +154,6 @@ class Message(Generic[T]):
154154
inner: T
155155

156156

157-
class FakeSelect:
158-
"""Helper class to mock Select object used in BatteryStatusTracker"""
159-
160-
# This is just number of Select instance attributes.
161-
def __init__( # pylint: disable=too-many-arguments
162-
self,
163-
battery: Optional[BatteryData] = None,
164-
inverter: Optional[InverterData] = None,
165-
set_power_result: Optional[SetPowerResult] = None,
166-
battery_timer_flag: bool = False,
167-
inverter_timer_flag: bool = False,
168-
) -> None:
169-
"""Create FakeSelect instance
170-
171-
Args:
172-
battery: Expected battery message. Defaults to None.
173-
inverter: Expected inverter message. Defaults to None.
174-
battery_timer_flag: If true the battery data timer will be set to indicate
175-
that no messages have been received for a while.
176-
inverter_timer_flag: If true the inverter data timer will be set to indicate
177-
that no messages have been received for a while.
178-
set_power_result: Expected SetPowerResult message. Defaults to None.
179-
"""
180-
self.battery = None if battery is None else Message(battery)
181-
self.inverter = None if inverter is None else Message(inverter)
182-
self.inverter_timer = inverter_timer_flag
183-
self.battery_timer = battery_timer_flag
184-
self.set_power_result = (
185-
None if set_power_result is None else Message(set_power_result)
186-
)
187-
188-
189157
BATTERY_ID = 106
190158
INVERTER_ID = 105
191159

0 commit comments

Comments
 (0)