ruby-grape / grape / 27480155161

97%

master: 97%

Build Type

Pull #2764

github

Committed by ericproulx

Commit Message

Route Grape::Json through the non-deprecated MultiJSON API

multi_json 1.21.0 deprecated the `MultiJson` constant (in favor of
`MultiJSON`) and `MultiJSON.dump` (in favor of `generate`), removed in
2.0. Grape detected the backend via `::MultiJson` and called `dump`, so
users on multi_json >= 1.21 saw deprecation warnings (#2763).

Detect `::MultiJSON` and give `Grape::Json` a small facade whose `dump`
delegates to `MultiJSON.generate` (byte-identical output), re-exposing
`parse` since the facade no longer inherits MultiJSON's methods. Behavior
is unchanged on every path; the stdlib and legacy multi_json fallbacks
are untouched.

Following #2762 (Grape standardizes on `parse`, never `load`, for JSON),
the two remaining `Grape::Json.load` spec helpers move to `parse`, so
`Grape::Json.load` no longer appears anywhere and the facade needs no
`load`.

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

Pull Request Pull Request #2764: Route Grape::Json through the non-deprecated MultiJSON API

Coverage Stats

1122 of 1221 branches covered (91.89%)

Branch coverage included in aggregate %.

11 of 12 new or added lines in 1 file covered. (91.67%)

1 existing line in 1 file now uncovered.

3568 of 3635 relevant lines covered (98.16%)

23078.98 hits per line