Commit bb68787
authored
test: raise default poll_until_condition timeout to deflake integration tests (#868)
The `test_request_queue_list_and_lock_head` integration test flaked on a
loaded Python 3.14 CI runner: after adding 5 requests, the queue head
stayed empty past the 5s poll ceiling (`assert 0 == 5`). The Apify
platform's listing endpoints are eventually consistent, so the 5s
default in `poll_until_condition` was too tight under parallel load.
Raise the default timeout from 5s to 30s, and drop the now-redundant
explicit `timeout=30` overrides in the request-queue and run tests. The
timeout is only a ceiling, so happy paths are unaffected.
Failing CI run:
https://github.com/apify/apify-client-python/actions/runs/27765226958/job/821499972761 parent 417d570 commit bb68787
3 files changed
Lines changed: 4 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
134 | 134 | | |
135 | 135 | | |
136 | 136 | | |
137 | | - | |
| 137 | + | |
138 | 138 | | |
139 | 139 | | |
140 | 140 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
526 | 526 | | |
527 | 527 | | |
528 | 528 | | |
529 | | - | |
| 529 | + | |
530 | 530 | | |
531 | 531 | | |
532 | 532 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
295 | 295 | | |
296 | 296 | | |
297 | 297 | | |
298 | | - | |
| 298 | + | |
299 | 299 | | |
300 | 300 | | |
301 | 301 | | |
| |||
339 | 339 | | |
340 | 340 | | |
341 | 341 | | |
342 | | - | |
| 342 | + | |
343 | 343 | | |
344 | 344 | | |
345 | 345 | | |
| |||
0 commit comments