| Summary: | [ews-build] Add build step to run WK1 layout-test | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Aakash Jain <aakash_jain> | ||||||
| Component: | Tools / Tests | Assignee: | Aakash Jain <aakash_jain> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | aakash_jain, ap, commit-queue, ews-watchlist, lforschler, webkit-bug-importer | ||||||
| Priority: | P2 | Keywords: | InRadar | ||||||
| Version: | Other | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=187674 | ||||||||
| Attachments: |
|
||||||||
|
Description
Aakash Jain
2018-08-12 22:50:44 PDT
Created attachment 346995 [details] Proposed patch Sample run: http://ews-build.webkit-uat.org/#/builders/26/builds/13/steps/3/logs/stdio Attachment 346995 [details] did not pass style-queue:
ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:777: [TestRunWebKit1Tests.test_success] Passing unexpected keyword argument 'state_string' in function call [pylint/E1123] [5]
ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:777: [TestRunWebKit1Tests.test_success] No value passed for parameter 'status_text' in function call [pylint/E1120] [5]
ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:791: [TestRunWebKit1Tests.test_failure] Passing unexpected keyword argument 'state_string' in function call [pylint/E1123] [5]
ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:791: [TestRunWebKit1Tests.test_failure] No value passed for parameter 'status_text' in function call [pylint/E1120] [5]
Total errors found: 4 in 3 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 346995 [details] Proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=346995&action=review I see we have style errors here... this seems to be common for buildbot code. I would rather us override the style check than continue to ignore them. > Tools/BuildSlaveSupport/ews-build/steps_unittest.py:769 > + self.setProperty('fullPlatform', 'ios-simulator') Do we only run WK1 on simulator? > Tools/BuildSlaveSupport/ews-build/steps_unittest.py:774 > + ) it looks like you add the "--dump-render-tree" option here, as well as in the start function of the RunWebKit1Tests class. is it needed both places? > Tools/BuildSlaveSupport/ews-build/steps_unittest.py:787 > + ) ditto here. I see this is for a test_success|failure function, but it wasn't clear to me if this was inherited. Comment on attachment 346995 [details] Proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=346995&action=review >> Tools/BuildSlaveSupport/ews-build/steps_unittest.py:769 >> + self.setProperty('fullPlatform', 'ios-simulator') > > Do we only run WK1 on simulator? Not really. I will change it. >> Tools/BuildSlaveSupport/ews-build/steps_unittest.py:774 >> + ) > > it looks like you add the "--dump-render-tree" option here, as well as in the start function of the RunWebKit1Tests class. is it needed both places? Yeah, one is in the code (steps.py), another one is in corresponding unit-test to match that. (In reply to Lucas Forschler from comment #3) > I see we have style errors here... this seems to be common for buildbot > code. I would rather us override the style check than continue to ignore them. Tracking the style checker issue in https://bugs.webkit.org/show_bug.cgi?id=188572 Created attachment 347103 [details]
Patch for landing
Comment on attachment 347103 [details] Patch for landing Clearing flags on attachment: 347103 Committed r234861: <https://trac.webkit.org/changeset/234861> All reviewed patches have been landed. Closing bug. |