Open
Bug 1263568
Opened 9 years ago
Updated 2 years ago
Make dbaron happier about wptrunner's reftest support
Categories
(Testing :: web-platform-tests, defect)
Testing
web-platform-tests
Tracking
(Not tracked)
NEW
People
(Reporter: Ms2ger, Unassigned)
References
Details
David: I'm not sure if there are bugs for the things you mentioned in the email you sent me, but please make them block this bug.
NI-ing myself to nag you about it, since you're not accepting requests :)
Flags: needinfo?(Ms2ger)
Reporter | ||
Updated•9 years ago
|
Flags: needinfo?(Ms2ger)
Version: Version 3 → Trunk
Reporter | ||
Updated•9 years ago
|
Flags: needinfo?(Ms2ger)
Updated•9 years ago
|
Flags: needinfo?(Ms2ger) → needinfo?(dbaron)
Updated•9 years ago
|
Comment 2•9 years ago
|
||
I'm not done. Still need to look into:
* assertion checking
* whether it still uses a slow method of image comparison
Comment 3•9 years ago
|
||
(And I'd also say more generally that a lot of work has gone into our existing reftest harness, and you're not going to magically produce something the same level of quality by starting over from scratch.)
Comment 4•9 years ago
|
||
OK, I think I've now filed the issues I'm aware of.
My preferred approach to fixing them, though, would be to use the existing reftest harness instead of trying to write a new one, rather than fixing all the bugs that I just filed.
As part of doing that, though, we should start switching to running reftests at the standard 600x600 (which is what the wpt reftest harness appears to do). The best way to do that would probably to make that be configurable in our reftest harness on at least a per-directory basis, so that we can convert existing reftests as well (which we can then go through a directory at a time rather than all at once). There's an existing bug on that that's been pretty quiet for a few years.
Flags: needinfo?(dbaron)
Comment 5•9 years ago
|
||
Our existing reftest harness isn't cross-browser compatible (we can't use it with Servo, for example), and doesn't afaik support continuing in the face of tests that e.g. crash the browser. Therefore it isn't suitable for running web-platform-tests (or CSSWG tests). As such I don't think that using the existing reftest harness is a viable solution.
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•