ruby-grape / grape / 28746672698

97%

push

github

Land #2775–#2779 on master (stacked-merge recovery) (#2780)

* Pass http_methods to Grape::Endpoint as a keyword instead of via options

`method` was pulled out of the endpoint's `**options` Hash, normalized in
place (`@options[:method] = Array(...)`), and renamed to `http_methods` on
the `Options` value object purely to dodge the `Object#method` shadow on
the generated Data accessor.

Expose it directly as a required `http_methods:` keyword on
`Grape::Endpoint#initialize` and pass it straight into `Options`, whose
field is now genuinely named `http_methods` (no rename, no shadow). The
value stays in `config` so endpoint equality — `config == other.config`,
which keeps same-path/different-verb endpoints distinct — is unchanged.

The endpoint's public `options` Hash no longer carries the redundant
`:method` entry; nothing read it, and grape-swagger reads the verb from
`route.request_method`. Documented in UPGRADING.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* Pass path to Grape::Endpoint as a keyword instead of via options

`path` was pulled from the endpoint's `**options` Hash and normalized in
place (`@options[:path] = Array(...)`), duplicating the normalization
`Options` already does.

Expose it as a required `path:` keyword on `Grape::Endpoint#initialize`
and pass it straight into `Options`, which stays the single normalized
source of truth. `path` remains in `config` because it is load-bearing
for route building (`config.path`) and for endpoint equality
(`config == other.config`, which keeps same-path/different-verb endpoints
distinct).

The endpoint's public `options` Hash no longer carries `:path`. Its only
reader was an internal test; `route.path` (the compiled pattern) is the
public path API and is what grape-swagger already uses. Documented in
UPGRADING.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* Add mounted_app reader and app: keyword to Grape::Endpoint

`app` (the Rack app or Grape ... (continued)

1130 of 1226 branches covered (92.17%)

Branch coverage included in aggregate %.

33 of 33 new or added lines in 7 files covered. (100.0%)

3591 of 3659 relevant lines covered (98.14%)

23423.7 hits per line