Skip to content

Commit 115f882

Browse files
committed
test: Make _filter_logs() take the name optionally
Most of the times we just want to check the _resampling logs. Signed-off-by: Leandro Lucarella <[email protected]>
1 parent b147ad7 commit 115f882

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

tests/timeseries/test_resampling.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -115,10 +115,7 @@ async def test_resampler_config_len_warn(
115115
)
116116
assert config.initial_buffer_len == init_len
117117
# Ignore errors produced by wrongly finalized gRPC server in unrelated tests
118-
assert _filter_logs(
119-
caplog.record_tuples,
120-
logger_name="frequenz.sdk.timeseries._resampling",
121-
) == [
118+
assert _filter_logs(caplog.record_tuples) == [
122119
(
123120
"frequenz.sdk.timeseries._resampling",
124121
logging.WARNING,
@@ -161,7 +158,6 @@ async def test_helper_buffer_too_big(
161158
# Ignore errors produced by wrongly finalized gRPC server in unrelated tests
162159
assert _filter_logs(
163160
caplog.record_tuples,
164-
logger_name="frequenz.sdk.timeseries._resampling",
165161
) == [
166162
(
167163
"frequenz.sdk.timeseries._resampling",
@@ -890,7 +886,7 @@ def _get_buffer_len(resampler: Resampler, source_recvr: Source) -> int:
890886
def _filter_logs(
891887
record_tuples: list[tuple[str, int, str]],
892888
*,
893-
logger_name: str,
889+
logger_name: str = "frequenz.sdk.timeseries._resampling",
894890
logger_level: int | None = None,
895891
) -> list[tuple[str, int, str]]:
896892
return [

0 commit comments

Comments
 (0)