This repository has been archived by the owner on Jan 23, 2024. It is now read-only.

testPostJsonLd Should NOT Include @graph urn:x-arq:DefaultGraphNode #197

Closed

catch-point opened this issue Oct 6, 2014 · 2 comments · Fixed by #200

Contributor

catch-point commented Oct 6, 2014

The graph urn:x-arq:DefaultGraphNode is not widely recognized (outside of jena) and should not sent to the LDP server when testing json-ld support. The test should be changed to omit any @graph directives or at least use the document URI as the graph URI.

Contributor

spadgett commented Oct 6, 2014

Related Jena issue:

https://issues.apache.org/jira/browse/JENA-794

Jena has few options for writing out JSON-LD. We might need to manually write the JSON-LD for this test or at least alter Jena's output to git rid of @graph.

Contributor

sspeiche commented Oct 6, 2014

Agree, since Jena's json-ld support isn't too flexible yet...we may just hand generate a simple representation.

spadgett pushed a commit to spadgett/ldp-testsuite that referenced this issue


          simplify JSON-LD serialization

a306e5d

- Don't use @graph if it's not needed
- Make sure Jena's serialization doesn't use
  graph URI urn:x-arq:DefaultGraphNode

Fixes w3c#197

spadgett pushed a commit to spadgett/ldp-testsuite that referenced this issue


          simplify JSON-LD serialization

662cc08

- Don't use @graph if it's not needed
- Make sure Jena's serialization doesn't use
  graph URI urn:x-arq:DefaultGraphNode

Fixes w3c#197

spadgett pushed a commit to spadgett/ldp-testsuite that referenced this issue


          simplify JSON-LD serialization

8e1b3a6

- Don't use @graph if it's not needed
- Make sure Jena's serialization doesn't use
  graph URI urn:x-arq:DefaultGraphNode

Fixes w3c#197

spadgett pushed a commit to spadgett/ldp-testsuite that referenced this issue


          simplify JSON-LD serialization

7070d99

- Don't use @graph if it's not needed
- Make sure Jena's serialization doesn't use
  graph URI urn:x-arq:DefaultGraphNode

Fixes w3c#197

spadgett mentioned this issue

simplify JSON-LD serialization #200

Merged

spadgett pushed a commit to spadgett/ldp-testsuite that referenced this issue


          simplify JSON-LD serialization

26723f0

- Don't use @graph if it's not needed
- Make sure Jena's serialization doesn't use
  graph URI urn:x-arq:DefaultGraphNode

Fixes w3c#197

spadgett pushed a commit to spadgett/ldp-testsuite that referenced this issue


          simplify JSON-LD serialization

cabbf20

- Don't use @graph if it's not needed
- Make sure Jena's serialization doesn't use
  graph URI urn:x-arq:DefaultGraphNode

Fixes w3c#197

sspeiche closed this as completed in #200

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.