Fix (validation) use joi's concat #23

Merged

joeybaker merged 2 commits from fix-joi-concat into master

2016-09-08 20:26:50 +00:00

Author	SHA1	Message	Date
Joey Baker	8ee5661252	Test: only run src test files #oops	2016-09-08 13:24:08 -07:00
Joey Baker	c59943a717	Fix (validation) use joi's concat It turns out defaultsDeep doesn't ever correctly combine Joi objects. So, the only option is to use Joi's concat method to combine Joi schemas. This complicates `getConfigForMethod`, but simplifies actual route creation. I ran into this because I'm setting up [lout](https://github.com/hapijs/lout) on a server, and it requires properly formatted Joi schemas. This leads me to believe there was something already wrong and Lout just exposed the problem.	2016-09-08 13:20:50 -07:00

Author

SHA1

Message

Date

Joey Baker

8ee5661252

Test: only run src test files #oops

2016-09-08 13:24:08 -07:00

Joey Baker

c59943a717

Fix (validation) use joi's concat

It turns out defaultsDeep doesn't ever correctly combine Joi objects.
So, the only option is to use Joi's concat method to combine Joi
schemas. This complicates `getConfigForMethod`, but simplifies actual
route creation.

I ran into this because I'm setting up [lout](https://github.com/hapijs/lout)
on a server, and it requires properly formatted Joi schemas. This leads
me to believe there was something already wrong and Lout just exposed
the problem.

2016-09-08 13:20:50 -07:00

Fix (validation) use joi's concat #23

2 Commits