ruby-grape / grape
100%
master: 100%

LAST BUILD BRANCH: dblock/automate-gem-release
DEFAULT BRANCH: master
Repo Added
Files 179
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH fix/nested-required-values-nil
branch: fix/nested-required-values-nil
CHANGE BRANCH
x

coverage: 99.983% (-0.02%) from 100.0%
36408319341

Pull #3010

github

ericproulx 
Reject nil for a required param with values in a nested scope

A `requires` param with `values:` rejected `nil` only at the root scope.
In a Hash scope, an Array scope or a `given` block, `nil` (or a value
that coerces to it, such as `""` for `type: Integer`) passed validation
and reached the endpoint, so `{"lines":[{"qty":null}]}` got through
`requires :qty, type: Integer, values: 1..99` and broke code as ordinary
as `params[:lines].sum { |line| line[:qty] }`. The README says the
opposite: "`requires` enforces presence and `:values` rejects `nil`."

The root-only rule came from f60eaa62 (2014, #671). Back then a child
scope's validators ran even when its optional parent was absent, so
rejecting `nil` there failed every request that left the parent out.
Scopes now skip validation when an optional parent is absent, so the
workaround only let `nil` through where the parent *is* given.

The validator now rejects `nil` whenever the param is required. A
missing nested param also reports `does not have a valid value` after
`is missing`, as a root param already did.

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Pull Request #3010: Reject nil for a required param with values in a nested scope

1493 of 1493 branches covered (100.0%)

Branch coverage included in aggregate %.

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

1 existing line in 1 file now uncovered.

4314 of 4315 relevant lines covered (99.98%)

29460.12 hits per line

Relevant lines Covered
4315 RELEVANT LINES 4314 COVERED LINES
29460.12 HITS PER LINE
Source Files on fix/nested-required-values-nil

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
36408319341 fix/nested-required-values-nil Reject nil for a required param with values in a nested scope A `requires` param with `values:` rejected `nil` only at the root scope. In a Hash scope, an Array scope or a `given` block, `nil` (or a value that coerces to it, such as `""` for `typ... Pull #3010 ericproulx github
99.98
36249910608 fix/nested-required-values-nil Reject nil for a required param with values in a nested scope A `requires` param with `values:` rejected `nil` only at the root scope. In a Hash scope, an Array scope or a `given` block, `nil` (or a value that coerces to it, such as `""` for `typ... Pull #3010 ericproulx github
99.98
36249889184 fix/nested-required-values-nil Reject nil for a required param with values in a nested scope A `requires` param with `values:` rejected `nil` only at the root scope. In a Hash scope, an Array scope or a `given` block, `nil` (or a value that coerces to it, such as `""` for `typ... Pull #3010 ericproulx github
99.98
See All Builds (4537)