fixes bugs: #190, #196, #192 where various keywords were not fully supported
Motivation
see their respective issues for background and references
I am not quite sure what error messages are best: "Missing property" was already being used by the 'required' keyword, and it's not great to have two different errors sharing the same message, so I made the 'required' error "Missing required property", so 'dependencies' can have "Missing dependent property". However, changing an existing error message might break unit tests in downstream code.
jhthorsen
commented
4 years ago
Summary
fixes bugs: #190, #196, #192 where various keywords were not fully supported
Motivation
see their respective issues for background and references
I am not quite sure what error messages are best: "Missing property" was already being used by the 'required' keyword, and it's not great to have two different errors sharing the same message, so I made the 'required' error "Missing required property", so 'dependencies' can have "Missing dependent property". However, changing an existing error message might break unit tests in downstream code.