ruby-grape / grape / 35913297405

100%

push

github

Merge a middleware's headers with a guard clause instead of a case (#2995)

`Middleware::Base#merge_headers` dispatched on the response with a
`case` whose implicit else -- a response that is neither a
Rack::Response nor an Array -- left the headers unmerged, and no spec
reached it: `call!` hands it what `after` or the app returned, which
the Rack SPEC makes a triplet unless it is a Rack::Response.

The Rack::Response branch is now a guard clause and the triplet the main
flow, so that unreachable third branch is gone.

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

1488 of 1502 branches covered (99.07%)

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%)

29169.85 hits per line