ruby-grape / grape / 25361575770 / 34

97%

master: 97%

push

github

Collapse `extract_handler` type-dispatch into a `case`/`when` (#2698)

`DSL::RequestResponse#extract_handler` dispatched on `with`'s type via three
`with.instance_of?` predicates, each on its own line. Lift the dispatch into a
single `case`/`when` block. Branches stay as plain expressions per the project
style; the early returns above the `case` short-circuit when there's no `with`
to inspect.

Side effect: `case`/`when` uses `===` (i.e. `is_a?`), not `instance_of?`. So a
hypothetical `Symbol`/`String`/`Proc` *subclass* now matches where it didn't
before — `is_a?` is the more conventional check and handles inheritance
correctly. None of the codebase or tests rely on the strict-instance behavior.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

283 of 1158 branches covered (24.44%)

Branch coverage included in aggregate %.

2376 of 3429 relevant lines covered (69.29%)

6.18 hits per line