Combine messages for minProperties/maxProperties

[jdesrosiers]

The messages produced by the maxProperties and minProperties handlers can be collapsed.

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

This produces multiple messages that could be combined

• Expected an object with no more than 2 properties
• Expected an object with no more than 3 properties

This can be collapsed to

• Expected an object with no more than 2 properties

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

[ShivaGupta-14]

@jdesrosiers

My Understanding

• maxProperties = maximum properties allowed in object
• minProperties = minimum properties required in object

Multiple maxProperties/minProperties from allOf -> separate messages. Instead of multiple type errors, compute strictest constraint (min of maxes, max of mins) across failing locations, single message with tightest bound.

Proposed Solution
update maxProperties.js: Math.min(all failing maxProperties)
update minProperties.js: Math.max(all failing minProperties)

I will open a PR soon, Thanks :)