Add ability to read bindings test results
Created attachment 303145 [details] Patch
Attachment 303145 [details] did not pass style-queue: ERROR: Tools/Scripts/webkitpy/common/net/bindingstestresults.py:39: [BindingsTestResults.results_from_string] Class 'BindingsTestResults' has no 'parse_json_string' member [pylint/E1101] [5] ERROR: Tools/Scripts/webkitpy/tool/bot/bindingstestresultsreader.py:26: No name 'abstracttestresultsreader' in module 'webkitpy.tool.bot' [pylint/E0611] [5] ERROR: Tools/Scripts/webkitpy/tool/bot/bindingstestresultsreader.py:33: [BindingsTestResultsReader.results] Instance of 'BindingsTestResultsReader' has no '_host' member [pylint/E1101] [5] ERROR: Tools/Scripts/webkitpy/tool/bot/bindingstestresultsreader.py:33: [BindingsTestResultsReader.results] Instance of 'BindingsTestResultsReader' has no '_results_directory' member [pylint/E1101] [5] ERROR: Tools/Scripts/webkitpy/tool/bot/bindingstestresultsreader.py:34: [BindingsTestResultsReader.results] Instance of 'BindingsTestResultsReader' has no '_read_file_contents' member [pylint/E1101] [5] Total errors found: 5 in 4 files If any of these errors are false positives, please file a bug against check-webkit-style.
Style-queue thinks some methods are missing because the patch on bug 169053 is not landed yet.
Comment on attachment 303145 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=303145&action=review LGTM; Please add the unit test(s) I mention below. > Tools/Scripts/webkitpy/common/net/bindingstestresults_unittest.py:38 > + self.assertTrue(no_failures_results.all_passed()) Please add a test for success cases as well.
Created attachment 303248 [details] Patch
Comment on attachment 303248 [details] Patch Clearing flags on attachment: 303248 Committed r213321: <http://trac.webkit.org/changeset/213321>
All reviewed patches have been landed. Closing bug.