stores-com / http-error
100%
main: 100%

LAST BUILD BRANCH: test-consumed-body-error-loss
DEFAULT BRANCH: main
Repo Added
Files 1
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH test-consumed-body-error-loss
branch: SELECT
CHANGE BRANCH
x

coverage: 100.0%. Remained the same
26477762191

push

github

farhatraiyan 
Add TODO test asserting error messages are lost when response body is pre-consumed

HttpError.from() falls back to "200 OK" instead of extracting errors[] when
the body has already been read. This documents the gap as a failing TODO test.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

13 of 13 branches covered (100.0%)

Branch coverage included in aggregate %.

48 of 48 relevant lines covered (100.0%)

5.19 hits per line

Relevant lines Covered
48 RELEVANT LINES 48 COVERED LINES
5.19 HITS PER LINE
Source Files on main

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
26477762191 test-consumed-body-error-loss Add TODO test asserting error messages are lost when response body is pre-consumed HttpError.from() falls back to "200 OK" instead of extracting errors[] when the body has already been read. This documents the gap as a failing TODO test. Co-Auth... push farhatraiyan github
100.0
25708126656 main aggregate body errors[] into err.message in HttpError.from() For each entry in a parsed-body `errors[]` array, the first present of `message` or `detail` is joined by `; ` and set as `err.message` (replacing the default `"${status} ${statusText}"... push web-flow github
100.0
25708010537 errors-aggregation drop redundant parenthetical from GraphQL bullet Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Pull #2 freshlogic github
100.0
25707602529 errors-aggregation make from() resilient to consumed bodies, restore example with throw pattern response.clone() throws synchronously when the body has been disturbed, so wrap the body read in try/catch. After a caller does response.json() and then needs to throw o... Pull #2 freshlogic github
100.0
25707603686 errors-aggregation Merge 967f9b825 into a1f9c8a25 Pull #2 web-flow github
100.0
25707489527 errors-aggregation drop code block from errors-in-body section; describe behavior in prose Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Pull #2 freshlogic github
100.0
25707473839 errors-aggregation drop prescriptive wording about when callers should throw Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Pull #2 freshlogic github
100.0
25707399365 errors-aggregation drop title from aggregation fallback JSON:API title is a category/summary that shouldn't change between occurrences, so it's a weak per-instance message. Sticking with message and detail gives clean 2-for-2 coverage of GraphQL and JSON:API. Co-A... Pull #2 freshlogic github
100.0
25707400851 errors-aggregation Merge 737c0c367 into a1f9c8a25 Pull #2 web-flow github
100.0
25707349656 errors-aggregation fix README: GraphQL errors[] is top-level, not data.errors[] Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Pull #2 freshlogic github
100.0
See All Builds (35)