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

Test failure: testXIncludeTestSuite #15

Open
io7m opened this issue Feb 13, 2017 · 5 comments
Open

Test failure: testXIncludeTestSuite #15

io7m opened this issue Feb 13, 2017 · 5 comments
Assignees
Labels

Comments

@io7m
Copy link

io7m commented Feb 13, 2017

In nu.xom.tests.XIncludeTest:

White spaces are required between publicId and systemId.

nu.xom.ParsingException: White spaces are required between publicId and systemId. at line 1, column 50 in http://dev.w3.org/cvsweb/~checkout~/2001/XInclude-Test-Suite/testdescr.xml?content-type=text/plain&only_with_tag=HEAD
at nu.xom.Builder.build(Unknown Source)
at nu.xom.Builder.build(Unknown Source)
at nu.xom.tests.XIncludeTest.testXIncludeTestSuite(Unknown Source)
Caused by: org.xml.sax.SAXParseException; systemId: http://dev.w3.org/cvsweb/~checkout~/2001/XInclude-Test-Suite/testdescr.xml?content-type=text/plain&only_with_tag=HEAD; lineNumber: 1; columnNumber: 50; White spaces are required between publicId and systemId.
at org.apache.xerces.parsers.AbstrractSAXParser.parse(Unknown Source)

Suspect this one is down to bit rot as well: Perhaps something changed on w3.org?

@elharo
Copy link
Owner

elharo commented Feb 13, 2017

Yes, probably some sort of redirect

@elharo
Copy link
Owner

elharo commented Feb 13, 2017

Are you manually typing these in or something? AbstrractSAXParser (two r's) and some of the other reports have misspelled class names

@io7m
Copy link
Author

io7m commented Feb 13, 2017

Are you manually typing these in or something?

Nope, copied and pasted from the reports.

http://ataxia.io7m.com/2017/02/13/testresults/

@elharo
Copy link
Owner

elharo commented Mar 3, 2018

I can reproduce. It's weird though. I can't see why this document should be reported as malformed, or with that error message.

@elharo elharo self-assigned this Mar 3, 2018
@elharo
Copy link
Owner

elharo commented Mar 3, 2018

I need to improve this but I think the problem is two-fold. Part of it is that the W3C CVS server is objecting to non-browser user agents or some such thing.

    // This test requires files that I have not received permission
    // to distribute so for the moment you won't be able to run it.
    // For my own use it checks to see if the files are present
    // and runs if it does find them. You can't just install the
    // XInclude-Test-Suite data as distributed by the W3C here.
    // Some of those tests rely on optional features XOM does not
    // support such as the xpointer() scheme and notations.
    // Plus some of those tests have mistakes. You need my patched 
    // version of the tests.
    public void testXIncludeTestSuite()  
      throws ParsingException, IOException, XIncludeException {
     
        File testDescription = new File("data");
        testDescription = new File(testDescription, "XInclude-Test-Suite");
        testDescription = new File(testDescription, "testdescr.xml");
        URL baseURL = testDescription.toURI().toURL();
        if (!testDescription.exists()) {
            baseURL = new URL(
              "http://dev.w3.org/cvsweb/~checkout~/2001/" +
              "XInclude-Test-Suite/testdescr.xml?content-type=text/" +
              "plain&only_with_tag=HEAD"
            );
        }

@elharo elharo added the testing label Jul 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants