ruby-grape / grape / 25361575770

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>

1076 of 1160 branches covered (92.76%)

Branch coverage included in aggregate %.

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

3382 of 3438 relevant lines covered (98.37%)

32394.28 hits per line