Closed jgraham closed 4 years ago
@emilio I think you were interested in this. If you have feedback it would be great, particularly if you have evidence that sharing load-type tests (e.g. gecko crashtests) between engines is effective at finding issues.
Crashes and other issues caught by these tests may not be sufficiently common between browsers to make sharing these tests a good use of resources.
This was my concern while reading the proposal, so I'm glad you included it! Personally, I'm pessimistic that tests like these do provide much value between implementations. I'm no implementer, though, so I'm very curious about what @emilio has to say on the subject.
I have actually some data on this, need to write it down, but this are the results thus far (I haven't filed all bugs yet) from running Gecko's crashtest suite in WebKit and Chrome, and Chrome's crashtests in Gecko:
And as I said I'm still not done running tests and filing bugs. Pending stuff:
So yeah, I do think they're useful :)
And this was a very simple test runner, there's tons of test-cases that I haven't ran or what not.
I'd rather call them crash tests, given that's what they're called everywhere else.
The reason for not calingthem crashtests was that crashing is only one possible failure mode; they could also fail with an assert or sanitizer failure or similar in vendor infrastructure. But I'm sympathetic to the idea that "load" can be confusing and there is precedent here.
I think that's an overly pedantic viewpoint to take when there's prior art in all vendors here already, even if the name is slightly confusing. (Just make all assertions and saniziter failures crashes, pff!)
https://github.com/web-platform-tests/wpt/pull/20017 is a draft PR for this RFC. It's not done yet (hence draft), but what's there seems to work.
My preference is for -crash
.
OK changed to -crash
and crashtests
. So I don't think there are more remaining issues and will merge.
Doing some archeology here, I wonder when we changed the wait class from wait
to test-wait
? @jgraham
Add a test type that just ensures that a page can be loaded without error.