Skip to content

Exclude 101 from random interim statuses in the HTTP/2 1xx test - #538

Open
ericmj wants to merge 1 commit into
mainfrom
test-http2-interim-no-101
Open

ericmj wants to merge 1 commit into
mainfrom
test-http2-interim-no-101

Conversation

@ericmj

@ericmj ericmj commented Oct 16, 2026

Copy link
Copy Markdown
Member

The "multiple before a single HEADERS" test picked each interim status from 100..199 and expected Mint to accept it. Mint rejects 101 in HTTP/2 since RFC 9113 section 8.6 doesn't support the Switching Protocols status, so the test failed for seeds that drew 101 (for example --seed 338383). The statuses are now drawn from 100..199 without 101.

The "multiple before a single HEADERS" test picked each interim status
from 100..199 and expected Mint to accept it. Mint rejects 101 in HTTP/2
since RFC 9113 section 8.6 doesn't support the Switching Protocols
status, so the test failed for seeds that drew 101 (for example
--seed 338383). The statuses are now drawn from 100..199 without 101.
@ericmj
ericmj added this pull request to stack #544 October 16, 2026 13:10
@github-actions

Copy link
Copy Markdown

Coverage Report for CI Build 0

Coverage remained the same at 90.508%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 1 coverage regression across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

1 previously-covered line in 1 file lost coverage.

File Lines Losing Coverage Coverage
lib/mint/http1/response.ex 1 96.05%

Coverage Stats

Coverage Status
Relevant Lines: 1970
Covered Lines: 1783
Line Coverage: 90.51%
Coverage Strength: 703.36 hits per line

馃挍 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant