Skip to content

Conversation

jdesrosiers
Copy link
Member

I copied the tests from 2019-09 and made changes for the new and removed keywords. I haven't added or changed anything that wasn't necessary to get it working in 2020-12. That means it's not a complete suite for 2020-12, however I think the only thing that is lacking is $dynamicAnchor/$dynamicRef. Only the functionality that overlap with $recursiveAnchor/$recursiveRef is covered. That can be added in a separate PR.

I've verified these tests with my Hyperjump JSV implementation.

I put the copy of 2019-09 tests in a separate commit to make it easier to review. If you view only the commit(s) that come after that one, it should be easier to see what changed rather than a wall of green.

Copy link
Member

@gregsdennis gregsdennis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One correction. Everything else looks right. I have a couple things to update on my implementation.

Includes changes for `$recursiveAnchor` (removed), `$recursiveRef` (removed),
`$dynamicAnchor`, `$dynamicRef`, `$prefixItems`, `items`, and `addtionalItems`
(removed).
@karenetheridge karenetheridge merged commit fcae732 into json-schema-org:master Mar 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants