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

Issue with "strange path" tests #124

Open
martinmaillard opened this issue Apr 6, 2020 · 2 comments
Open

Issue with "strange path" tests #124

martinmaillard opened this issue Apr 6, 2020 · 2 comments

Comments

@martinmaillard
Copy link

Hi,

I'm working on implementing the SHACL Test Suite for rdf-validate-shacl and am having issues with the two "strange path" tests.

The tests define the following expected resultPath: sh:resultPath ( ex:p ex:q ) ;

As far as I know, the spec doesn't specify how to handle these weird paths, so I don't understand why the extraneous inversePath is stripped out.

My ValidationReport graph contains sh:resultPath [ rdf:first ex:p ; rdf:rest [ rdf:first ex:q ; rdf:rest rdf:nil ] ; sh:inversePath ex:p ] ; (copied directly from the shape's sh:path). Shouldn't this be valid?

@HolgerKnublauch
Copy link
Contributor

Hmm, I am puzzled right now. This was a test that was added rather late in the process, at a time where the WG was basically out of time. From my current understanding the paths are ill-formed because they are both lists and inverse paths yet all these path types must be disjoint. If my interpretation is correct then the test result should be undefined, and the tests not be in the test suite.

There was some discussion about this in https://www.w3.org/2017/05/03-shapes-minutes.html that lead to some editorial clarification. Does anyone else reading this ticket have comments?

Meanwhile I suggest ignoring this test.

@martinmaillard
Copy link
Author

Ok, thanks for the feedback :)

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