Skip to content

Commit b29eb9a

Browse files
ai: apply changes for #868 (1 review thread)
Addresses: - #3599845693 at src/databricks/sql/auth/retry.py:300 Signed-off-by: peco-engineer-bot[bot] <peco-engineer-bot[bot]@users.noreply.github.com>
1 parent 9f9ec88 commit b29eb9a

1 file changed

Lines changed: 10 additions & 0 deletions

File tree

tests/unit/test_retry.py

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,16 @@ def test_sleep__short_retry_after_is_not_inflated(self, t_mock, retry_policy):
5454

5555
t_mock.assert_called_with(2)
5656

57+
@patch("time.sleep")
58+
def test_sleep__large_retry_after_is_capped_at_delay_max(self, t_mock, retry_policy):
59+
# A large server Retry-After is clamped down to delay_max, which acts as
60+
# a ceiling on the wait. delay_max defaults to 30 in these fixtures.
61+
retry_policy._retry_start_time = time.time()
62+
retry_policy.history = []
63+
retry_policy.sleep(HTTPResponse(status=503, headers={"Retry-After": "120"}))
64+
65+
t_mock.assert_called_with(retry_policy.delay_max)
66+
5767
@patch("time.sleep")
5868
def test_sleep__no_retry_after_header__multiple_retries(self, t_mock, retry_policy):
5969
num_attempts = retry_policy.stop_after_attempts_count

0 commit comments

Comments
 (0)