sgerrand / ex_deputy / 26519149127

100%

Build Type

push

github

Committed by sgerrand

Commit Message

refactor(error): collapse 429 path and dedupe body integer lookups

`from_response/1` had two separate 429 clauses, one for responses
with headers and one for responses without. `header_int(nil, _)`
already returns `nil` via the catch-all clause, so the
header-absent branch is redundant — merge into a single clause that
pulls headers with `Map.get/2`.

`get_retry_after/1`, `get_rate_limit/1`, and `get_rate_remaining/1`
were three near-identical helpers differing only in which two body
keys they checked. Collapse into one `body_int(body, keys)` helper
driven by three module-attribute key lists.

Coverage Stats

8 of 9 new or added lines in 1 file covered. (88.89%)

6 existing lines in 1 file now uncovered.

315 of 324 relevant lines covered (97.22%)

16.1 hits per line