ruby-grape / grape / 35912331064

100%

master: 100%

Build Type

push

github

Committed by ericproulx

Commit Message

Merge a middleware's headers with a guard clause instead of a case

`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>

Coverage Stats

1470 of 1502 branches covered (97.87%)

Branch coverage included in aggregate %.

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

42 existing lines in 7 files now uncovered.

4282 of 4324 relevant lines covered (99.03%)

12159.41 hits per line