1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
|
from unittest import mock
import pytest
from globus_sdk.transport import (
RequestCallerInfo,
RequestsTransport,
RetryCheckResult,
RetryCheckRunner,
RetryConfig,
RetryContext,
)
from globus_sdk.transport.default_retry_checks import (
DEFAULT_RETRY_CHECKS,
check_retry_after_header,
check_transient_error,
)
@pytest.mark.parametrize("http_status", (429, 503))
def test_retry_policy_respects_retry_after(mocksleep, http_status):
retry_config = RetryConfig()
retry_config.checks.register_many_checks(DEFAULT_RETRY_CHECKS)
transport = RequestsTransport()
checker = RetryCheckRunner(retry_config.checks)
dummy_response = mock.Mock()
dummy_response.headers = {"Retry-After": "5"}
dummy_response.status_code = http_status
caller_info = RequestCallerInfo(retry_config=retry_config)
ctx = RetryContext(1, caller_info=caller_info, response=dummy_response)
assert checker.should_retry(ctx) is True
mocksleep.assert_not_called()
transport._retry_sleep(retry_config, ctx)
mocksleep.assert_called_once_with(5)
@pytest.mark.parametrize("http_status", (429, 503))
def test_retry_policy_ignores_retry_after_too_high(mocksleep, http_status):
# set explicit max sleep to confirm that the value is capped here
retry_config = RetryConfig(max_sleep=5)
retry_config.checks.register_many_checks(DEFAULT_RETRY_CHECKS)
transport = RequestsTransport()
checker = RetryCheckRunner(retry_config.checks)
dummy_response = mock.Mock()
dummy_response.headers = {"Retry-After": "20"}
dummy_response.status_code = http_status
caller_info = RequestCallerInfo(retry_config=retry_config)
ctx = RetryContext(1, caller_info=caller_info, response=dummy_response)
assert checker.should_retry(ctx) is True
mocksleep.assert_not_called()
transport._retry_sleep(retry_config, ctx)
mocksleep.assert_called_once_with(5)
@pytest.mark.parametrize("http_status", (429, 503))
def test_retry_policy_ignores_malformed_retry_after(mocksleep, http_status):
retry_config = RetryConfig()
retry_config.checks.register_many_checks(DEFAULT_RETRY_CHECKS)
transport = RequestsTransport()
checker = RetryCheckRunner(retry_config.checks)
dummy_response = mock.Mock()
dummy_response.headers = {"Retry-After": "not-an-integer"}
dummy_response.status_code = http_status
caller_info = RequestCallerInfo(retry_config=retry_config)
ctx = RetryContext(1, caller_info=caller_info, response=dummy_response)
assert checker.should_retry(ctx) is True
mocksleep.assert_not_called()
transport._retry_sleep(retry_config, ctx)
mocksleep.assert_called_once()
@pytest.mark.parametrize(
"check_method",
[check_retry_after_header, check_transient_error],
ids=lambda f: f.__name__,
)
def test_default_retry_check_noop_on_exception(check_method, mocksleep):
retry_config = RetryConfig()
retry_config.checks.register_many_checks(DEFAULT_RETRY_CHECKS)
caller_info = RequestCallerInfo(retry_config=retry_config)
ctx = RetryContext(1, caller_info=caller_info, exception=Exception("foo"))
assert check_method(ctx) is RetryCheckResult.no_decision
|
