ruby-grape / grape / 39058251026

100%

master: 100%

Pull #3038

github

Hold every method to RuboCop's default ABC size

Metrics/AbcSize was raised to 80; the 26 methods above the default of 17
are split instead, and the override is dropped.

Behaviour is unchanged. A few of the splits also simplify:

- Router#transaction keeps the static lookup and hands a bucket match and
  a union match to #bucket_transaction and #union_transaction. Each hands
  its answer to #settle (was #fall_back), which now also returns a
  response that does not cascade.
- Endpoint#to_routes builds one pattern per path and shares it across the
  route's methods, as a HEAD route already shares its GET route's pattern.
- Versioner::Base builds each vendor media type through one
  #append_media_types instead of two copies of the same loop.
- ParamsScope's two `:all` / `:none` forms share #declare_optional_fields.
- ValidationsSpec#parse_coerce reads `coerce_with` once.

Three specs pin what mutation testing showed unpinned among the moved
lines: the failure entity is the one for the status answered, a route a
bucket matched is not run again when it cascades, and a 405 under path
versioning is reached behind a prefix and under a mount path.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

1503 of 1505 branches covered (99.87%)

Branch coverage included in aggregate %.

191 of 191 new or added lines in 14 files covered. (100.0%)

4525 of 4526 relevant lines covered (99.98%)

29613.55 hits per line