RESOLVED FIXED 187330
[ews-build] Bindings-tests-EWS should output result in json 
https://bugs.webkit.org/show_bug.cgi?id=187330
Summary [ews-build] Bindings-tests-EWS should output result in json
Aakash Jain
Reported 2018-07-04 13:56:45 PDT
Bindings-tests-EWS should output result in json. This would be useful in easily parsing the errors.
Attachments
Proposed patch (2.68 KB, patch)
2018-07-04 14:00 PDT, Aakash Jain 		ddkilzer: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Aakash Jain
Comment 1 2018-07-04 14:00:03 PDT
Created attachment 344299 [details] Proposed patch Sample run: http://ews-build.webkit-uat.org/#/builders/14/builds/22
David Kilzer (:ddkilzer)
Comment 2 2018-07-05 10:18:25 PDT
Comment on attachment 344299 [details] Proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=344299&action=review r=me > Tools/BuildSlaveSupport/ews-build/steps_unittest.py:244 > + command=['Tools/Scripts/run-bindings-tests', '--json-output=bindings_test_results.json'], > + logfiles={'json': 'bindings_test_results.json'}, Nit: Don't repeat yourself by extracting `jsonFileName` variable as is done in steps.py. > Tools/BuildSlaveSupport/ews-build/steps_unittest.py:256 > + command=['Tools/Scripts/run-bindings-tests', '--json-output=bindings_test_results.json'], > + logfiles={'json': 'bindings_test_results.json'}, Nit: Don't repeat yourself by extracting `jsonFileName` variable as is done in steps.py.
Aakash Jain
Comment 3 2018-07-05 11:36:58 PDT
Committed r233531: <https://trac.webkit.org/changeset/233531>.
Radar WebKit Bug Importer
Comment 4 2018-07-05 11:37:20 PDT
<rdar://problem/41856838>
Note You need to log in before you can comment on or make changes to this bug.