The current implementation doesn't account for test binaries that have
not emitted any subtests. This is the case when a test binary doesn't
exist or it crashes.
Also, in the latter case, the stderr of the crashed processes was not
being outputted to the user, masking a crashing test binary as a
This patch fixes both issues: It adds several warning prints when
binaries can't be run or fail to add any subtest, emitting the stderr
generated by the failing or crashing binary.
Created attachment 418152 [details]
Created attachment 418421 [details]
Uploaded an updated version that marks these tests as CRASH. I expect the EWS to fail on the test suites that are currently crashing.
(In reply to Alicia Boya García from comment #3)
> Uploaded an updated version that marks these tests as CRASH. I expect the
> EWS to fail on the test suites that are currently crashing.
Well, EWS didn't fail because the crashes have been fixed in the meantime. Now the previously crashing test suites are reporting subtests again.
... and so on.
Created attachment 418538 [details]
Reuploaded removing the ChangeLog line about changing expectations since it's no longer necessary.
Remember to CC firstname.lastname@example.org if you're not using the GTK or WPE component, so it doesn't get missed!
Comment on attachment 418538 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=418538&action=review
> + # No subtests where emitted, either the test binary didn't exist, or we don't know how to run it, or it crashed.
Created attachment 418550 [details]
Committed r271955: <https://trac.webkit.org/changeset/271955>
All reviewed patches have been landed. Closing bug and clearing flags on attachment 418550 [details].