NaN printed in cosole as null

[rashtao]

This code:

var schema = Joi.object().keys({
        date: Joi.date().allow(null)
});

Joi.assert({date: NaN}, schema);

prints in the console:

Error: {
  "date" [1]: null
}

[1] "date" must be a number of milliseconds or valid date string

and could be really confusing to debug!

[Marsup]

Unfortunately joi relies on JSON.stringify which transforms a few types, NaN being converted to null.
If you find a clever way to deal with it I'll take it.

[mdibaiee]

@Marsup I searched for this issue, the only "workaround" seems to be transforming NaN and Infinity to their string equivalents and then transforming them back while doing JSON.parse. Although I think we don't need the parse case here, as we're only printing it out.

If you think we would use this, I'd be happy to make a PR.

[Marsup]

Yup, I should port what I did in lab. If someone else is happy to do it then I'm fine with it :)

[Marsup]

Fixed by #735.