[ 
https://issues.apache.org/jira/browse/JENA-431?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13625826#comment-13625826
 ] 

Andy Seaborne commented on JENA-431:
------------------------------------

The tests look like they are very old in origin.

There's no way to test for being an N-triples parser except to test it on 
N-triples and since Turtle can parse valid N-triples, to identify an N-triples 
only parser also requires testing negatives as well.  We have always allowed 
changing the parser - the tests just happened to run before an app could do 
that but it's not true with RIOT initializing automatically.

It is probably historical that the NT parser accepts raw newlines -- I found 
another historical feature for parsing XMLLiterals.I guess it all dates back to 
different NT designs in the RDF working group.  The outcome was one triple on 
one line.

With RIOT, the role of the jena-core N-triples parser is back to the original 
reason for its existence - testing RDF/XML output.  As of Jena 2.10.0, users 
are using RIOT parsers if they have jena-arq on the classpath as we encourage.

I have suggested we merge jena-arq and jena-core then re-modularise the code 
base.  Now we have a maven artifact "apache-jena-libs"  we can chnage the 
module structure of the implementation.

When Jena 2.10.1 is released and we are comfortable the new writers are 
working, we can do that, including deleting the old core package "n3" (which 
isn't N3 - it's Turtle). If people want to do further modularization, that 
would be great but we can stil release 

I fixed the parser and changed the test data to "foo\u000Abar".

(I don't understand AbstractTestPackage because it seems to adds individual 
tests itself.)
                
> com.hp.hpl.jena.rdf.model.test.TestReaders fails when ARQ included in 
> classpath.
> --------------------------------------------------------------------------------
>
>                 Key: JENA-431
>                 URL: https://issues.apache.org/jira/browse/JENA-431
>             Project: Apache Jena
>          Issue Type: Bug
>          Components: ARQ, Jena
>    Affects Versions: Jena 2.10.1
>            Reporter: Claude Warren
>         Attachments: TestReaderDemo.tar.gz
>
>
> Similar to an earlier bug, when ARQ is included the readers that are returned 
> fail the TestReaders tests.  The same tests pass when ARQ is not included.
> The attached file includes a simple example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to