-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
IndexedDB test running issues #4622
Comments
FWIW I couldn't reproduce your problems, although there is some arguably confusing output. |
@pwnall - can you take a look at the issue with the exceptions test? |
@brettz9 Can you please tell me how you're running the tests? Sorry for the cluelessness. I'm using |
@pwnall : I also ran it on Chrome 55 without the experimental web platform flag and with
As far as (Also, while that page lists a Harness status of "Error", as I can understand for an uncaught error, the aggregate shows it, as mentioned, within "Timeouts" rather than under "Errors"--I'd expect one or the other, e.g., if a timeout occurs because the test never completed but without producing an uncaught error.) Regarding @jgraham 's observations in the other issue, I can understand a timeout being reported (though again, I'd expect "Error" instead if there were an uncaught error as reported on the individual page). I'm wondering whether the aggregate page gives up sooner than when simply loading the page independently to completion, and as such does not get the total test count of 13. As far as @jgraham's other statement on reporting the subtest status as NOTRUN, if the runner is considering the total number of tests as 0, then I would expect it to report something other than "0/1" in the subttest pass rate (maybe 0/0) (but if the aggregate page is really working properly, I'd expect the aggregate page to get the total count of 13 and report 0/13). I also find it confusing to be reporting "NOTRUN" instead of "Timeout"--if the latter is the known cause of not running (and as it shows as "Timeout" in the totals and in the JSON)--and again, I think "Error" would be even better than "Timeout" if an uncaught error is the known specific reason for the timeout as occurs when visiting the page on its own. |
I've reported an issue regarding some behaviors of the test runner at w3c/wpt-tools/issues/152 in regard to some IndexedDB tests (running on Chrome), but since it pertains to IndexedDB specifically, I'm not sure whether to mention it here (or on the IndexedDB repo for that matter).
@inexorabletash
The text was updated successfully, but these errors were encountered: