Skip to content
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

Running when manifest URL does not match resource URI in document #77

Closed
jeswr opened this issue Jul 17, 2022 · 3 comments
Closed

Running when manifest URL does not match resource URI in document #77

jeswr opened this issue Jul 17, 2022 · 3 comments

Comments

@jeswr
Copy link
Contributor

jeswr commented Jul 17, 2022

In the case of the RDF* specifications, the manifest documents are located at URLs like:

https://w3c.github.io/rdf-star/tests/trig/syntax/manifest.ttl

But in the document they are referred to using a fragment identifier

https://w3c.github.io/rdf-star/tests/trig/syntax#manifest

Which results in the error

> rdf-test-suite spec/parser.js https://w3c.github.io/rdf-star/tests/trig/syntax/manifest.ttl -i '{ "format": "trig-star" }' -c .rdf-test-suite-cache/

Error: Could not find a resource https://w3c.github.io/rdf-star/tests/trig/syntax/manifest.ttl in the document at https://w3c.github.io/rdf-star/tests/trig/syntax/manifest.ttl
    at ManifestLoader.<anonymous> (/home/jeswr/Documents/rdfstar/N3.js/node_modules/rdf-test-suite/lib/ManifestLoader.js:51:27)
    at Generator.next (<anonymous>)
    at fulfilled (/home/jeswr/Documents/rdfstar/N3.js/node_modules/rdf-test-suite/lib/ManifestLoader.js:5:58)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Is there an option to override the resource URI (I realise that this is currently a problem with the way the spec is published but it would be good to have a way to get around it until then).

@rubensworks
Copy link
Owner

Many (most?) test suites unfortunately follow this convention.

There should be a workaround for that here. Perhaps it needs to be adapted slightly to cope with this case.

@jeswr
Copy link
Contributor Author

jeswr commented Jul 18, 2022

Linked issues but didn't follow up with comments.

The rdfstar test suite uses #manifest in the resource URI but /manifest.(jsonld|ttl) in the document URL.

There is a discussion around this (w3c/rdf-star#269) and solution in (#78).

@rubensworks
Copy link
Owner

This was fixed in #78.

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

No branches or pull requests

2 participants