Summary: | Add support for getting test name list json from test results server | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Victor Wang <victorw> | ||||||
Component: | Tools / Tests | Assignee: | Victor Wang <victorw> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | ojan | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | PC | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Victor Wang
2010-08-17 10:22:03 PDT
Created attachment 64630 [details]
Proposed Patch
Comment on attachment 64630 [details]
Proposed Patch
WebKitTools/ChangeLog:5
+ Add support to test results server for downloading json that
Nit: Add support to the test results server...
WebKitTools/ChangeLog:9
+ results so it includes results not only for tests failed in
Nit: results so that it includes...for tests that failed in...
WebKitTools/TestResultServer/model/jsonresults.py:437
+ Json file with test name list only. The json format is the same
Nit: s/Json/JSON or s/Json/json
WebKitTools/TestResultServer/model/jsonresults.py:454
+ "tests": dict.fromkeys(tests, {"results": [], "times": []})}
This is a lot of bloat just for the tests list. Can "tests" just be tests.keys()? Does this need to be in the same format as the one stored in the datastore?
Created attachment 64640 [details]
Updated per comments
(In reply to comment #2) > (From update of attachment 64630 [details]) > WebKitTools/ChangeLog:5 > + Add support to test results server for downloading json that > Nit: Add support to the test results server... done > > WebKitTools/ChangeLog:9 > + results so it includes results not only for tests failed in > Nit: results so that it includes...for tests that failed in... done > > WebKitTools/TestResultServer/model/jsonresults.py:437 > + Json file with test name list only. The json format is the same > Nit: s/Json/JSON or s/Json/json done > > WebKitTools/TestResultServer/model/jsonresults.py:454 > + "tests": dict.fromkeys(tests, {"results": [], "times": []})} > This is a lot of bloat just for the tests list. Can "tests" just be tests.keys()? Does this need to be in the same format as the one stored in the datastore? not required by datastore format. I was doing this to avoid modifying the json results generator so we can share the existing logic for generating results.json. No need to have "results' and "times" if I refactor the json results generator little bit, so removed them from the update patch. Comment on attachment 64640 [details] Updated per comments Clearing flags on attachment: 64640 Committed r65567: <http://trac.webkit.org/changeset/65567> All reviewed patches have been landed. Closing bug. |