Title: [251167] trunk/Tools
- Revision
- 251167
- Author
- aakash_j...@apple.com
- Date
- 2019-10-15 16:20:55 -0700 (Tue, 15 Oct 2019)
Log Message
Improve summary for WebKitPerl Tests build step
https://bugs.webkit.org/show_bug.cgi?id=203006
Reviewed by Jonathan Bedard.
* BuildSlaveSupport/ews-build/steps.py:
(RunWebKitPerlTests.getResultSummary): Override method to customize summary.
* BuildSlaveSupport/ews-build/steps_unittest.py: Updated unit-tests.
Modified Paths
Diff
Modified: trunk/Tools/BuildSlaveSupport/ews-build/steps.py (251166 => 251167)
--- trunk/Tools/BuildSlaveSupport/ews-build/steps.py 2019-10-15 23:17:18 UTC (rev 251166)
+++ trunk/Tools/BuildSlaveSupport/ews-build/steps.py 2019-10-15 23:20:55 UTC (rev 251167)
@@ -603,7 +603,14 @@
def __init__(self, **kwargs):
super(RunWebKitPerlTests, self).__init__(timeout=2 * 60, logEnviron=False, **kwargs)
+ def getResultSummary(self):
+ if self.results == SUCCESS:
+ message = 'Passed webkitperl tests'
+ self.build.buildFinished([message], SUCCESS)
+ return {u'step': unicode(message)}
+ return {u'step': u'Failed webkitperl tests'}
+
class RunBuildWebKitOrgUnitTests(shell.ShellCommand):
name = 'build-webkit-org-unit-tests'
description = ['build-webkit-unit-tests running']
Modified: trunk/Tools/BuildSlaveSupport/ews-build/steps_unittest.py (251166 => 251167)
--- trunk/Tools/BuildSlaveSupport/ews-build/steps_unittest.py 2019-10-15 23:17:18 UTC (rev 251166)
+++ trunk/Tools/BuildSlaveSupport/ews-build/steps_unittest.py 2019-10-15 23:20:55 UTC (rev 251167)
@@ -390,7 +390,7 @@
)
+ 0,
)
- self.expectOutcome(result=SUCCESS, state_string='webkitperl-tests')
+ self.expectOutcome(result=SUCCESS, state_string='Passed webkitperl tests')
return self.runStep()
def test_failure(self):
@@ -407,7 +407,7 @@
Failed 1/40 test programs. 10/630 subtests failed.''')
+ 2,
)
- self.expectOutcome(result=FAILURE, state_string='webkitperl-tests (failure)')
+ self.expectOutcome(result=FAILURE, state_string='Failed webkitperl tests')
return self.runStep()
Modified: trunk/Tools/ChangeLog (251166 => 251167)
--- trunk/Tools/ChangeLog 2019-10-15 23:17:18 UTC (rev 251166)
+++ trunk/Tools/ChangeLog 2019-10-15 23:20:55 UTC (rev 251167)
@@ -1,3 +1,14 @@
+2019-10-15 Aakash Jain <aakash_j...@apple.com>
+
+ Improve summary for WebKitPerl Tests build step
+ https://bugs.webkit.org/show_bug.cgi?id=203006
+
+ Reviewed by Jonathan Bedard.
+
+ * BuildSlaveSupport/ews-build/steps.py:
+ (RunWebKitPerlTests.getResultSummary): Override method to customize summary.
+ * BuildSlaveSupport/ews-build/steps_unittest.py: Updated unit-tests.
+
2019-10-15 Zhifei Fang <zhifei_f...@apple.com>
Tool to mark jsc test skip/enable
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes