TheKevJames / coveralls-python / 29912346135

96%

push

github

feat(api): retry transient HTTP failures (#760)

Route API POSTs through a requests Session with a urllib3 Retry adapter so
transient failures can be retried with exponential backoff and jitter. Only
connection errors, read timeouts, 429s and 5xx responses are retried; a 422 is
a configuration error and is never retried. Retries are off by default and are
configurable via the --retries flag, the COVERALLS_RETRIES env var, or a
retries key in .coveralls.yml, on every command that talks to the API.

The API HTTP tests move to the responses library for higher fidelity, which
replaces the previous fake "resubmission" tests with real retry coverage.

Closes #572

changelog:
API requests can now retry transient HTTP failures (connection errors, read
timeouts, 429 and 5xx responses) with exponential backoff and jitter. Retries
are off by default; enable with ``--retries``, ``COVERALLS_RETRIES``, or a
``retries`` key in ``.coveralls.yml``.

161 of 168 branches covered (95.83%)

Branch coverage included in aggregate %.

45 of 47 new or added lines in 3 files covered. (95.74%)

583 of 604 relevant lines covered (96.52%)

4.83 hits per line