Track evaluated properties with pattern properties of schema = true

[P0lip]

What issue does this pull request resolve?

#1625

What changes did you make?

patternProperties that are always valid are tracked now to ensure unevaluatedProperties keyword 'works'.

Is there anything that requires more attention while reviewing?

checkForUnevaluatedProperties check?

[epoberezkin]

I will update it

[epoberezkin]

@P0lip the updated condition to bypass validation is:

if (
  patterns.length === 0 ||
  (alwaysValidPatterns.length === patterns.length &&
    (!it.opts.unevaluated || it.props === true))
) {
  return
}

to spell it, we want to skip validation in one of two cases:

• there are no patterns in the schema at all
• all patterns have always valid schemas AND we either do not track unevaluated properties (!it.opts.unevaluated) or all properties are already evaluated (it.props === true)

Does it make sense?

[P0lip]

Yup, make sense,
Thanks!