w3c / json-ld-framing

JSON-LD 1.1 Framing Specification
https://w3c.github.io/json-ld-framing/
Other
24 stars 20 forks source link

Tests for #99: Framing matches on specified value #100

Closed michaelcpuckett closed 4 years ago

michaelcpuckett commented 4 years ago

I noticed a discrepancy between JSON-LD Playground and RDF Distiller regarding frame matching.

This PR adds 3 tests for frame matching on a specified value.

Fixes https://github.com/w3c/json-ld-framing/issues/99

Original issue here:

https://github.com/digitalbazaar/jsonld.js/issues/300

I created a fix for this in jsonld.js repo and the I'm hoping to commit the tests here.

michaelcpuckett commented 4 years ago

One check has failed. I joined the JSON-LD group a few days ago but it's reporting that I haven't, I think. https://www.w3.org/users/121373

iherman commented 4 years ago

Admin note: even in case of acceptance, this PR does not produce substantive (ie, IPR related) changes in the spec, so I set the checks accordingly.

iherman commented 4 years ago

Marked as non substantive for IPR from ash-nazg.

iherman commented 4 years ago

One check has failed. I joined the JSON-LD group a few days ago but it's reporting that I haven't, I think. https://www.w3.org/users/121373

Unfortunately, this is more complicated...

Anyway, your contribution is not affecting IPR (the main reason of all this check on the PR), and I so noted. So your PR is o.k.

gkellogg commented 4 years ago

Tests look good, thanks!

gkellogg commented 4 years ago

Fixes #99.