Bug 138235 - Add --json flag to run-bindings-tests
Summary: Add --json flag to run-bindings-tests
Status: RESOLVED DUPLICATE of bug 168626
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Srinivasan Vijayaraghavan
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-30 17:48 PDT by Jake Nielsen
Modified: 2017-02-20 22:35 PST (History)
5 users (show)

See Also:


Attachments
First Attempt (7.39 KB, patch)
2014-10-30 18:22 PDT, Jake Nielsen
jake.nielsen.webkit: commit-queue-
Details | Formatted Diff | Diff
results.html displays the JSON correctly. (7.31 KB, patch)
2014-10-31 11:37 PDT, Jake Nielsen
dbates: review-
dbates: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jake Nielsen 2014-10-30 17:48:03 PDT
This will also have to change bindings test to print on stderr instead of stdout so that the JSON doesn't get interleaved with human readable output.
Comment 1 Jake Nielsen 2014-10-30 18:22:02 PDT
Created attachment 240717 [details]
First Attempt

I want to fiddle a little more before this gets CQ+ed, but I'm about to leave for the day. Any feedback would be much appreciated so I can roll it in when I make changes tomorrow morning.
Comment 2 WebKit Commit Bot 2014-10-30 18:23:20 PDT
Attachment 240717 [details] did not pass style-queue:


ERROR: Tools/Scripts/webkitpy/bindings/main.py:44:  whitespace before ':'  [pep8/E203] [5]
Total errors found: 1 in 3 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Jake Nielsen 2014-10-31 11:37:08 PDT
Created attachment 240748 [details]
results.html displays the JSON correctly.
Comment 4 Daniel Bates 2016-04-23 12:36:49 PDT
Comment on attachment 240748 [details]
results.html displays the JSON correctly.

This does not seem like the correct approach. It is weird that passing --json will write both human readable output to standard error and write JSON to standard output because both outputs will be visible to a person that runs run-bindings-tests in their terminal (assuming a default configured terminal). Instead we should have the --json flag take a path to an output file and write the JSON-formatted test results to this file. We should not change any other behavior of the script run-bindings-tests.
Comment 5 Alexey Proskuryakov 2017-02-20 22:35:33 PST

*** This bug has been marked as a duplicate of bug 168626 ***