Fix autoload types and validators

[ericproulx]

Hi,

I've found a solution for the #2218 AND #2221. This PR adds a lazy_find_validator function that loads Grape's validators when needed. That way, we don't have to require all of them since the autoload doesn't trigger the inherited function.

Also, ArrayCoercer and SetCoercer were registering a DryTypeCoercer collection through inheritance. I had to find another way to register the collection. Therefore, I applied the same technique.

Finally, that way, we can remove safely the eager_load! #2216 in the spec_helper. I ran several specs in our stack and I haven't found any issues.

Happy New Year 🍾

[dm1try]

Do I understand right that the original PRs #2207 #2209 solve some "consistency problem" where some part of the code is autoloaded and some part is not? without some numbers, it is unclear whether this is a problem at all.

but I like the part with autoloading validators based on :short_name, it echoes to the first part of #2204(though it's not enough for proper integration with Rails for example)

[dblock]

Do I understand right that the original PRs #2207 #2209 solve some "consistency problem" where some part of the code is autoloaded and some part is not? without some numbers, it is unclear whether this is a problem at all.

i don’t think it’s a problem either way; if my memory serves me correctly there were issues trying to move validators to auto loading and so this was left alone - sounds like we don’t have any issues with going one way or another, I vote to move to auto loading for consistency

[dm1try]

I vote to move to auto loading for consistency to auto loading for consistency

that works for me

[dm1try]

So, take a look to the comments and rebase the branch(we should not have an intermediate "merge master" commits here). Thanks!

[dblock]

Capitalize Autoload

[dm1try]

The changes looks good to me.
though, the commits history looks sloppy :) I can help to fix it(later tonight).

[dblock]

The changes looks good to me. though, the commits history looks sloppy :) I can help to fix it(later tonight).

I just squash-merged, thanks @ericproulx for the code, and @dm1try for a thorough code review.