-
Notifications
You must be signed in to change notification settings - Fork 325
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix the additionalProperties in oneOf failed test cases #516
Comments
Good one. Thanks. I did not test it on 1.0.44, but nowIi have and results differ. It is not flawless either, but where there is ambivalence this is acceptable of course. On the other hand: I think the messages are all correct:
the errormessages seem to be better (master: failFast=false): master:$.activities[0].weight: is missing but it is required 1.0.44$.activities[0].weight: is missing but it is required |
@huubfleuren Thanks a lot for your detailed comparison. I think we need a code analysis between the two versions to see what has been changed for the part. |
Completed in #720 |
These test cases were submitted by @huubfleuren and I just market them as disabled for now. Our developers can pick them up and get them fixed.
@huubfleuren Have you tested under version 1.0.44? Are they fail as well in 1.0.44? Thanks.
The text was updated successfully, but these errors were encountered: