nil Value for type: Integer Does Not Return Error

[jackdesert]

Using 0.7.1, If I set

params do
  requires :id, type: Integer
end

And then I pass in

{ id: nil }

Grape does not detect an error.

[dblock]

You POST this via JSON? Is it different when it's in the URL query string? Would appreciate a failing test in Grape if you have a moment.

[jackdesert]

Just pushed a pull request with a failing test:
#624

On Tue, Apr 8, 2014 at 4:11 AM, Daniel Doubrovkine (dB.) @dblockdotorg <
notifications@github.com> wrote:

You POST this via JSON? Is it different when it's in the URL query string?
Would appreciate a failing test in Grape if you have a moment.

Reply to this email directly or view it on GitHubhttps://github.com//issues/622#issuecomment-39835932
.

[mynameisrufus]

I think nil is still a valid value, I think a validation would make more sense allow_nil: true.

requires :foo, allow_nil: true

What if grape did not have optional and required and only had param plus a mixture of validators?

param :foo, allow_nil: true, presence: true

[dm1try]

btw, you can use allow_blank to allow nil

requires :id, type: Integer, allow_blank: true #default

[dblock]

@dm1try is right on, modified specs with allow_blank in dblock@048c158 to demonstrate.