Combine messages for minLength/maxLength

[jdesrosiers]

The messages produced by the maxLength and minLength handlers can be collapsed.

{
  "allOf": [
    { "maxLength": 2 },
    { "maxLength": 3 }
  ]
}

This produces multiple messages that could be combined

• Expected a string with no more than 2 characters
• Expected a string with no more than 3 characters

This can be collapsed to

• Expected a string with no more than 2 characters

No handlers need to be combined for this one, but the maxLength and minLength handlers will need to be updated to track the lowest/highest number of characters allowed/required so it produces only one message.

[ShivaGupta-14]

@jdesrosiers

My Understanding

• maxLength = maximum characters allowed in string
• minLength = minimum characters required in string

Same solution as maxProperties/minProperties #137:

• Multiple allOf -> separate messages -> collapse to strictest constraint
• maxLength.js: Math.min(all failing maxLength)
• minLength.js: Math.max(all failing minLength)
• Track only the most constraining location (not all locations)

I will open a PR soon, thanks!