ruby-grape / grape / 25872788683

97%

master: 97%

Build Type

Pull #2717

github

Committed by ericproulx

Commit Message

Convert Grape::Exceptions::ErrorResponse to a Data value object

ErrorResponse was already structured as a value object — five readers,
kwarg-only constructor, hand-written `==`/`eql?`/`hash` — so swapping
the hand-rolled class for `Data.define` is a near-zero-risk
simplification. Twelve lines of boilerplate (the ivar-setting
constructor and the equality/hash trio) collapse into the Data
declaration itself, while the two class-level factories
(`from_exception` and `coerce`) and the custom `to_s` move into the
Data block unchanged.

Defaults stay on `#initialize` for the `Hash`-coerced branch in
`.coerce` (which slices a partial input) and for callers like
`Middleware::Error#default_rescue_handler` that build instances
without supplying `status`/`headers`.

No production behaviour change.

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

Pull Request Pull Request #2717: Convert Grape::Exceptions::ErrorResponse to a Data value object

Coverage Stats

1073 of 1155 branches covered (92.9%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 1 file covered. (100.0%)

1 existing line in 1 file now uncovered.

3453 of 3509 relevant lines covered (98.4%)

32149.56 hits per line