Request: Warn when functions are passed unexpected arguments

[ltegman]

Context

• node version: 4.X
• joi version: 9.0.4
• environment (node, browser): node
• used with (hapi, standalone, ...): hapi
• any other relevant information:

What are you trying to achieve or the steps to reproduce ?

We had a developer, who after working with Joi.object for a couple validations assumed that Joi.string would also have a default behavior when passed an argument (this default behavior being to enforce a max). I realize this assumed behavior is not documented anywhere, but I think it would be nice if Joi.string and other similar validators that don't accept an argument would warn you of that if you pass them something.

const schema = {
  messageText: Joi.string(1024).required()
};

Which result you had ?

Joi.string silently accepted the argument, doing nothing with it.

What did you expect ?

A Joi error indicating that Joi.string does not accept any arguments

[Marsup]

This seems like a pretty random assumption, why max ? Will take a PR including all the types.

[ar4mirez]

guys I got some free time if none of you is working on this PR I'm available to work on this one, lmk.

[AdriVanHoudt]

@ar4mirez feel free to submit a PR!

[ar4mirez]

Guys quick question I have a solution done for the first level types any(), array(), binary(), boolean(), date(), number(), string(), so my question is do you guys want to apply the same enhancement to second level types such as required(), forbidden() etc... ?

Thanks.

[Marsup]

Not for now, most inputs are validated or mistakes can't be avoided.

[ar4mirez]

Alright make total sense to me, I'll PR then what I have and receive your feedback.

[lock]

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.