Skip to content

Commit c29b896

Browse files
committed
Test data time alignment.
Naming.
1 parent b1338f1 commit c29b896

File tree

1 file changed

+11
-5
lines changed

1 file changed

+11
-5
lines changed

tests/unit/test_logging.py

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,12 +31,12 @@
3131
b'2025-05-13T07:26:14.132Z [apify] DEBUG c\n',
3232
b'2025-05-13T0', # Chunked log that got split in the marker
3333
b'7:26:14.132Z [apify] DEBUG d\n' # part 2
34-
b'2025-05-13T07:26:14.132Z [apify] DEB', # Chunked log that got split outside of marker
34+
b'2025-05-13T07:27:14.132Z [apify] DEB', # Chunked log that got split outside of marker
3535
b'UG e\n', # part 2
3636
# Already redirected message
37-
b'2025-05-16T12:08:50.123Z [apify.redirect-logger-4U1oAnKau6jpzjUuA] -> 2025-05-16T12:08:49.840Z ACTOR: Pulling\n',
37+
b'2025-05-13T07:28:14.132Z [apify.redirect-logger-4U1oAnKau6jpzjUuA] -> 2025-05-13T07:27:14.132Z ACTOR: Pulling\n',
3838
)
39-
_CHUNKED_MESSAGES_COUNT = 3
39+
_EXISTING_LOGS_BEFORE_REDIRECT_ATTACH = 3
4040

4141
_EXPECTED_MESSAGES_AND_LEVELS = (
4242
('2025-05-13T07:24:12.588Z ACTOR: Pulling Docker image of build.', logging.INFO),
@@ -135,7 +135,10 @@ def propagate_stream_logs() -> None:
135135

136136
@pytest.mark.parametrize(
137137
('log_from_start', 'expected_log_count'),
138-
[(True, len(_EXPECTED_MESSAGES_AND_LEVELS)), (False, len(_EXPECTED_MESSAGES_AND_LEVELS) - _CHUNKED_MESSAGES_COUNT)],
138+
[
139+
(True, len(_EXPECTED_MESSAGES_AND_LEVELS)),
140+
(False, len(_EXPECTED_MESSAGES_AND_LEVELS) - _EXISTING_LOGS_BEFORE_REDIRECT_ATTACH),
141+
],
139142
)
140143
@respx.mock
141144
async def test_redirected_logs_async(
@@ -171,7 +174,10 @@ async def test_redirected_logs_async(
171174

172175
@pytest.mark.parametrize(
173176
('log_from_start', 'expected_log_count'),
174-
[(True, len(_EXPECTED_MESSAGES_AND_LEVELS)), (False, len(_EXPECTED_MESSAGES_AND_LEVELS) - _CHUNKED_MESSAGES_COUNT)],
177+
[
178+
(True, len(_EXPECTED_MESSAGES_AND_LEVELS)),
179+
(False, len(_EXPECTED_MESSAGES_AND_LEVELS) - _EXISTING_LOGS_BEFORE_REDIRECT_ATTACH),
180+
],
175181
)
176182
@respx.mock
177183
def test_redirected_logs_sync(

0 commit comments

Comments
 (0)