Skip to content

Commit 9baf380

Browse files
committed
Fixed the test_retry_exponential_backoff integration test
1 parent bb47c92 commit 9baf380

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

tests/e2e/common/retry_test_mixins.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@ def test_retry_max_count_not_exceeded(self):
174174
def test_retry_exponential_backoff(self):
175175
"""GIVEN the retry policy is configured for reasonable exponential backoff
176176
WHEN the server sends nothing but 429 responses with retry-afters
177-
THEN the connector will use those retry-afters as a floor
177+
THEN the connector will use those retry-afters values as delay
178178
"""
179179
retry_policy = self._retry_policy.copy()
180180
retry_policy["_retry_delay_min"] = 1
@@ -191,10 +191,10 @@ def test_retry_exponential_backoff(self):
191191
assert isinstance(cm.value.args[1], MaxRetryDurationError)
192192

193193
# With setting delay_min to 1, the expected retry delays should be:
194-
# 3, 3, 4
195-
# The first 2 retries are allowed, the 3rd retry puts the total duration over the limit
194+
# 3, 3, 3, 3
195+
# The first 3 retries are allowed, the 4th retry puts the total duration over the limit
196196
# of 10 seconds
197-
assert mock_obj.return_value.getresponse.call_count == 3
197+
assert mock_obj.return_value.getresponse.call_count == 4
198198
assert duration > 6
199199

200200
# Should be less than 7, but this is a safe margin for CI/CD slowness

0 commit comments

Comments
 (0)