ruby-grape / grape / 25909757486

97%

master: 97%

Pull #2721

github

Use an internal Grape::Validations::SharedOptions value object

`Validators::Base#initialize` consumed the shared `opts` via
`opts.values_at(:fail_fast, :allow_blank)` and copied the two values
into per-validator ivars.

Introduce `Grape::Validations::SharedOptions`
(`Data.define(:allow_blank, :fail_fast)` with defaults mirroring the
prior `values_at` behaviour) as an *internal* representation: `Base`
builds it from the `opts` Hash —
`SharedOptions.new(**opts.slice(:allow_blank, :fail_fast))` — and
exposes the two fields via `Forwardable.def_delegators :@opts,
:allow_blank, :fail_fast`. `#fail_fast?` calls the delegated
`fail_fast`; `ValuesValidator` (the one subclass reading `@allow_blank`
directly) uses the `allow_blank` reader.

The public 5th-argument contract of `Validators::Base#initialize`
stays a plain Hash, so custom validators and any code that
hand-constructs a validator with a Hash are unaffected. `slice` keeps
the historical tolerance of ignoring unknown keys. `ValidationsSpec`
keeps emitting the `{ allow_blank:, fail_fast: }.freeze` Hash; nothing
about its public surface changes — no UPGRADING entry needed.

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

1084 of 1167 branches covered (92.89%)

Branch coverage included in aggregate %.

10 of 10 new or added lines in 3 files covered. (100.0%)

3502 of 3559 relevant lines covered (98.4%)

32115.0 hits per line