Skip to content

Commit d0650b1

Browse files
mbertrandclaude
andcommitted
Make backoff test deterministic by asserting delegated parameters
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent fc64752 commit d0650b1

1 file changed

Lines changed: 9 additions & 5 deletions

File tree

vector_search/tasks_test.py

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -865,11 +865,15 @@ def test_generate_embeddings_retries_on_unavailable(mocker):
865865
retry.assert_called_once()
866866

867867

868-
def test_retry_countdown_is_minutes_scale():
869-
"""Backoff is minutes-scale so retries land after a qdrant throttle episode."""
870-
samples = [_retry_countdown(0) for _ in range(50)]
871-
assert max(samples) > 60 # jittered up to 2 minutes, not seconds
872-
assert max(samples) <= 120
868+
def test_retry_countdown_is_minutes_scale(mocker):
869+
"""Backoff is jittered exponential at minutes scale, capped at 10m."""
870+
backoff = mocker.patch(
871+
"vector_search.tasks.get_exponential_backoff_interval", return_value=42
872+
)
873+
assert _retry_countdown(2) == 42
874+
backoff.assert_called_once_with(
875+
factor=120, retries=2, maximum=600, full_jitter=True
876+
)
873877

874878

875879
def test_generate_embeddings_records_on_exhaustion(mocker):

0 commit comments

Comments
 (0)