1 no link to concept scheme

[sroertgen]

I removed the shape in question and also added basic test functionality for the shape.

See https://github.com/skohub-io/shapes/tree/1-no-link-to-concept-scheme#tests for my idea of testing the shape.

[sroertgen]

I just came across this SHACL Test Suite report: https://w3c.github.io/data-shapes/data-shapes-test-suite/

If we would like to dig deeper into testing this shape, this might be an interesting option.

[acka47]

Aren't tests/valid/ConceptShape.ttl and tests/valid/OrphanConcepts.ttl identical? I don't see why we need both.

[sroertgen]

They are indeed identical. My idea (that I apparently did not write down) was that maybe a test file is added per shape to test that shape. So OrpahnConcepts.ttl should be the data to test the :OrphanConcepts shape and ConceptShape.ttl would be the data to test :ConceptShape.

[acka47]

It would probably reduce possible misunderstanding when the "Error" notes in by the test.sh script would include the filename where the error occured insteadof just "Error: This should be invalid, but is valid" referring to the previous line. (I at least was unsure to what this refers in the first try.)

[acka47]

Generally, this looks fine. You may think about adressing my last comment but it is not necessary. I will have to understand this whole better in the future by taking a more thorough look at it.

[sroertgen]

That's a good point. I added the filename to the error message.