ruby-grape / grape / 35912611389

100%

master: 100%

Pull #2996

github

Cover the header versioner's last two branches

`Versioner::Header#version_not_found!` guards `versions&.none?` for a
middleware built without versions, which only direct use of it does --
through the DSL a header-versioned API always declares them -- and no
spec did. A spec now passes such a middleware a request naming a
version.

`MediaTypeForAcceptCache` skipped an Accept header whose media type came
back nil. Its table is only read as `table[accept] || best_quality(...)`,
so a nil entry would fall through to the same answer, and no spec could
tell the skip from its absence. The table is now built with `to_h` and
`compact`, which keeps it free of nil entries without a branch.

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

1488 of 1501 branches covered (99.13%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 1 file covered. (100.0%)

4324 of 4324 relevant lines covered (100.0%)

29166.67 hits per line