RESOLVED FIXED205608
[GTK][WPE] Report number of total tests run and failed in API test runner like run-api-tests does
https://bugs.webkit.org/show_bug.cgi?id=205608
Summary [GTK][WPE] Report number of total tests run and failed in API test runner lik...
Carlos Alberto Lopez Perez
Reported 2019-12-27 10:44:47 PST
We can reuse and simplify the implementation of bug 202361 a lot by outputting at the end of the run-gtk-tests/run-wpe-tests step the information about the number of tests ran and failed like run-api-tests does: "Ran X tests of Y with Z successful" Where: X=total number of tests executed Y=total number of tests available Z=total number of tests that passed (or that failed as expected) If we do this, we can basically use the same logic on the EWS (and buildbot) for running the gtk and mac/ios API tests.
Attachments
Patch (6.89 KB, patch)
2019-12-27 10:53 PST, Carlos Alberto Lopez Perez
no flags
Carlos Alberto Lopez Perez
Comment 1 2019-12-27 10:53:30 PST
Created attachment 386450 [details] Patch Patch doesn't apply because it is implemented on top of the changes from bug 205579
Carlos Alberto Lopez Perez
Comment 2 2019-12-27 10:56:11 PST
(In reply to Carlos Alberto Lopez Perez from comment #1) > Created attachment 386450 [details] > Patch > > Patch doesn't apply because it is implemented on top of the changes from bug > 205579 mmmm, seems it applies anyway.. good :)
Carlos Alberto Lopez Perez
Comment 3 2019-12-30 09:17:06 PST
Comment on attachment 386450 [details] Patch Clearing flags on attachment: 386450 Committed r253947: <https://trac.webkit.org/changeset/253947>
Carlos Alberto Lopez Perez
Comment 4 2019-12-30 09:17:10 PST
All reviewed patches have been landed. Closing bug.
Aakash Jain
Comment 5 2019-12-31 06:09:10 PST
Does this script output results in json format as well? If not, we would need that as well. EWS use json output to parse the name of failing tests.
Carlos Alberto Lopez Perez
Comment 6 2019-12-31 06:12:57 PST
(In reply to Aakash Jain from comment #5) > Does this script output results in json format as well? If not, we would > need that as well. EWS use json output to parse the name of failing tests. Yes, it does now. See bug 205579
Note You need to log in before you can comment on or make changes to this bug.