Conversation
| ## Negative cases | ||
|
|
||
| TODO: include info in the manifest on why and how should the producer fail | ||
| - `neg_001` – a set of quads with RDF 1.1 non-conformant terms for s, p, o, g. Stream options from `pos_001`. |
There was a problem hiding this comment.
I'm not sure how does this test case make sense in the producer context. Jelly-JVM will simply pass this through, without checks. I think we talked about this, it is the responsibility of the user to ensure this, so this test case makes no sense. Unless I'm missing something / forgot something....
There was a problem hiding this comment.
we talked about it today :) The specification requires the user to make sure the generalized statements option conforms with the actual data and rdf validate command catches this mismatch instantly, so to me the question is only where to place this test here or in the quads_1_1_generalized cases
There was a problem hiding this comment.
Generalized then. This test can't be executed at all by a non-generalized RDF processor, and the spec only demands implication:
isGenRdfUsed => generalized_statements = 1
test/rdf/README.md
Outdated
|
|
||
| [NOTE]: Generalized RDF test cases require a generalized N-Triples parser. If the library does not implement gen RDF, then the generalized RDF tests should be skipped | ||
| [NOTES]: | ||
| 1. Generalized RDF test cases require a generalized N-Triples parser. If the library does not implement gen RDF, then the generalized RDF tests should be skipped |
There was a problem hiding this comment.
Do you mean the negative cases? Or just tests in rdf_1_1_generalized, which don't exist yet?
There was a problem hiding this comment.
both -- negative cases that misuse generalied_statements option and future rdf_1_1_generalized
There was a problem hiding this comment.
But here we just settled that the misuse cases won't be here. So, please clarify that this is only about the latter.
There was a problem hiding this comment.
well, now it will be only for rdf_1_1_generalized where the current negative cases will migrate
|
@Ostrzyciel the sketchy negative test was removed and the readme was reworded a bit |
Relates to #39
Implements test cases for the QUADS 1.1 for the producer: