ruby-grape / grape / 25236304270

97%

master: 97%

Pull #2698

github

Collapse `extract_handler` type-dispatch into a `case`/`when`

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

1078 of 1162 branches covered (92.77%)

Branch coverage included in aggregate %.

4 of 5 new or added lines in 1 file covered. (80.0%)

3353 of 3408 relevant lines covered (98.39%)

32185.66 hits per line