Skip to content

Commit 7081ee7

Browse files
committed
test: poll with backoff for transient add_request failures
Replace the no-delay 3-attempt retry loop with the poll_until_condition helper so transient platform-side None results have time to clear between attempts.
1 parent 62e00bd commit 7081ee7

1 file changed

Lines changed: 2 additions & 5 deletions

File tree

tests/integration/test_request_queue.py

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -743,11 +743,8 @@ async def test_request_deduplication_edge_cases(request_queue_apify: RequestQueu
743743

744744
results = list[bool]()
745745
for url, expected_duplicate in urls_and_deduplication_expectations:
746-
# add_request may transiently return None due to platform-side issues, retry up to 3 times.
747-
for _ in range(3):
748-
result = await rq.add_request(url)
749-
if result is not None:
750-
break # Successfully added.
746+
# add_request may transiently return None due to platform-side issues, poll until it succeeds.
747+
result = await poll_until_condition(lambda url=url: rq.add_request(url), lambda result: result is not None)
751748

752749
assert result is not None
753750
results.append(result.was_already_present)

0 commit comments

Comments
 (0)