Bug 187330

Summary: [ews-build] Bindings-tests-EWS should output result in json
Product: WebKit Reporter: Aakash Jain <aakash_jain>
Component: Tools / TestsAssignee: Aakash Jain <aakash_jain>
Status: RESOLVED FIXED    
Severity: Normal CC: aakash_jain, ap, ddkilzer, lforschler, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=187014
https://bugs.webkit.org/show_bug.cgi?id=189417
Attachments:
Description Flags
Proposed patch ddkilzer: review+

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.