| Summary: | [EWS] Add unit tests for factories | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Aakash Jain <aakash_jain> | ||||||
| Component: | Tools / Tests | Assignee: | Aakash Jain <aakash_jain> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | aakash_jain, ap, jbedard, webkit-bug-importer | ||||||
| Priority: | P2 | Keywords: | InRadar | ||||||
| Version: | Other | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| Attachments: |
|
||||||||
|
Description
Aakash Jain
2020-01-12 17:26:25 PST
Created attachment 387498 [details]
Patch
Comment on attachment 387498 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=387498&action=review > Tools/ChangeLog:3 > + [EWS] Add unit tests for factories Will this run as part of an existing test suite on bots? (In reply to Alexey Proskuryakov from comment #2) > Will this run as part of an existing test suite on bots? Yes, in fact these unit-tests were automatically run by the Services EWS for this patch as well (since runUnittests.py script automatically runs _unittest.py files in that directory). The corresponding build for this patch run 180 tests (in https://ews-build.webkit.org/#/builders/20/builds/11417/steps/7/logs/stdio), while previously 172 tests were being run (e.g.: https://ews-build.webkit.org/#/builders/20/builds/11092/steps/7/logs/stdio). Comment on attachment 387498 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=387498&action=review >> Tools/ChangeLog:3 >> + [EWS] Add unit tests for factories > > Will this run as part of an existing test suite on bots? Yes, Tools/BuildSlaveSupport/ews-build/runUnittests.py will pick it up. > Tools/BuildSlaveSupport/ews-build/factories_unittest.py:40 > + actual_steps = map(step_to_dict, actual_steps) I would expect the style checker to want an empty line here to separate the function Created attachment 387561 [details]
Patch
(In reply to Jonathan Bedard from comment #4) > I would expect the style checker to want an empty line here to separate the function Added empty line. Committed r254448: <https://trac.webkit.org/changeset/254448> |