Skip to content

Conversation

davidlehn
Copy link
Contributor

@davidlehn davidlehn commented Dec 6, 2023

Adds a test for a protected @graph @container property that nullifies it's @context. This is similar to a pattern found in the VC Data Model 2.0 context. There are other test variations of context nullification, graph properties, and protection rules, etc, but there doesn't seem to be one with this combination. jsonld.js was failing this test but other implementations pass.

Are other variations of this test needed? It looks like most base features are covered, but perhaps tricky combinations of features might expose edge case issues. Help needed.

See:
w3c/vc-data-model#1373

@davidlehn davidlehn force-pushed the add-graph-container-tests branch from 626350a to 385d101 Compare March 8, 2024 04:44
@davidlehn
Copy link
Contributor Author

This had to be rebased due to conflicting "pr42" tests. Ready to merge with others are ready.

@pchampin and other implementors, can you please review this.

@gkellogg The generate process output expand-manifest.html but not toRdf-manifest.html. Is that a script error or an issue with the PR?

@gkellogg gkellogg merged commit 8baa4b8 into main Mar 29, 2024
@gkellogg gkellogg deleted the add-graph-container-tests branch March 29, 2024 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants