nicklaw5 / helix / 24290704284

93%

main: 93%

Build Type

Pull #259

github

Committed by web-flow

Commit Message

Return non-nil error when Twitch API responds with error body

Add ErrTwitchResponseError sentinel and pullErrorFromResponse helper to
helix.go. All endpoint methods now return a wrapped ErrTwitchResponseError
when the response body contains a non-zero error status field, making it
easier for callers to detect Twitch API errors via the standard error
interface or errors.Is.

Also reset response error fields at the start of each doRequest retry loop
so that a successful retry after a token refresh does not inherit the stale
error state from the preceding 401 response.

Update all test files to conditionally check err only for 2xx responses
or handle the new ErrTwitchResponseError for expected non-2xx responses.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: nicklaw5 <7908386+nicklaw5@users.noreply.github.com>

Pull Request Pull Request #259: Return non-nil error when Twitch API responds with an error body

Coverage Stats

123 of 126 new or added lines in 40 files covered. (97.62%)

32 existing lines in 1 file now uncovered.

1862 of 2005 relevant lines covered (92.87%)

6297.39 hits per line