-
Notifications
You must be signed in to change notification settings - Fork 128
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
Question about EARL reports #155
Comments
At least some of the failures seem to fit the pattern below:
|
I haven't looked into this, but it may in part be a case of the the test suites changing and pyld hasn't caught up. But looks like you got a bunch of failures and errors so maybe it's something else too. The reports really should be clear that results are for a test suite and implementation as of a certain date. As far as tracking down and fixing the issues, help would be appreciated. |
Happy to pitch in -- just wanted to be certain I understood the starting point. |
Ok - part of the issue is that URL's of the form
If I do
So - do we update the manifest and any embedded links? How to proceed? |
Tests should now all be at https://w3c.github.io/json-ld-api/tests/ and /json-ld-framing/tests/. Those in json-ld.org are obsolete. |
Will update the manifests accordingly. |
Ah - I see what is going on. The instructions for running the tests are a bit ambiguous.... |
Was using https://github.com/digitalbazaar/pyld/blob/master/CONTRIBUTING.rst , which wasn't clear on where the tests came from. Will file a separate issue |
When I run
python tests/runtests.py ../json-ld-api/tests/ ../json-ld-framing/tests/ -e pyld-earl.jsonld
, I getFAILED (failures=18, errors=168, skipped=7)
This doesn't seem to line up with the summary printed in
https://w3c.github.io/json-ld-api/reports/
Digging into the report itself, the first failure shows at:
Which, if I understand the reports correctly, appeared to have passed previously. Help!
The text was updated successfully, but these errors were encountered: