Changeset 244769 in webkit

Timestamp:

Apr 30, 2019, 7:51:13 AM (7 years ago)

Author:

aakash_jain@apple.com

Message:

[ews-build] Parse and display webkitpy failures
​https://bugs.webkit.org/show_bug.cgi?id=197395

Reviewed by Lucas Forschler.

• BuildSlaveSupport/ews-build/steps.py:

(RunWebKitPyTests.start): Initialize log_observer for json output.
(RunWebKitPyTests.getResultSummary): Update step and build summary based on webkitpy results.
(RunWebKitPyTests._addToLog): Method to add message to log.

• BuildSlaveSupport/ews-build/steps_unittest.py: Updated unit-test accordingly.

Location:

trunk/Tools

Files:

• BuildSlaveSupport/ews-build/steps.py (modified) (1 diff)
• BuildSlaveSupport/ews-build/steps_unittest.py (modified) (1 diff)
• ChangeLog (modified) (1 diff)

trunk/Tools/BuildSlaveSupport/ews-build/steps.py

@@ -492 +492 @@
         super(RunWebKitPyTests, self).__init__(timeout=2 * 60, **kwargs)
 
+    def start(self):
+        self.log_observer = logobserver.BufferLogObserver()
+        self.addLogObserver('json', self.log_observer)
+        return shell.ShellCommand.start(self)
+
+    def getResultSummary(self):
+        if self.results == SUCCESS:
+            message = 'Passed webkitpy tests'
+            self.build.buildFinished([message], SUCCESS)
+            return {u'step': unicode(message)}
+
+        logLines = self.log_observer.getStdout()
+        json_text = ''.join([line for line in logLines.splitlines()])
+        try:
+            webkitpy_results = json.loads(json_text)
+        except Exception as ex:
+            self._addToLog('stderr', 'ERROR: unable to parse data, exception: {}'.format(ex))
+            return super(RunWebKitPyTests, self).getResultSummary()
+
+        failures = webkitpy_results.get('failures') + webkitpy_results.get('errors')
+        if not failures:
+            return super(RunWebKitPyTests, self).getResultSummary()
+        pluralSuffix = 's' if len(failures) > 1 else ''
+        failures_string = ', '.join([failure.get('name').replace('webkitpy.', '') for failure in failures])
+        message = 'Found {} WebKitPy test failure{}: {}'.format(len(failures), pluralSuffix, failures_string)
+        self.build.buildFinished([message], FAILURE)
+        return {u'step': unicode(message)}
+
+    @defer.inlineCallbacks
+    def _addToLog(self, logName, message):
+        try:
+            log = self.getLog(logName)
+        except KeyError:
+            log = yield self.addLog(logName)
+        log.addStdout(message)
+
 
 def appendCustomBuildFlags(step, platform, fullPlatform):

trunk/Tools/BuildSlaveSupport/ews-build/steps_unittest.py

@@ -373 +373 @@
             + 0,
         )
-        self.expectOutcome(result=SUCCESS, state_string='webkitpy-tests')
+        self.expectOutcome(result=SUCCESS, state_string='Passed webkitpy tests')
         return self.runStep()
 

trunk/Tools/ChangeLog

@@ +1 @@
+2019-04-30  Aakash Jain  <aakash_jain@apple.com>
+
+        [ews-build] Parse and display webkitpy failures
+        https://bugs.webkit.org/show_bug.cgi?id=197395
+
+        Reviewed by Lucas Forschler.
+
+        * BuildSlaveSupport/ews-build/steps.py:
+        (RunWebKitPyTests.start): Initialize log_observer for json output.
+        (RunWebKitPyTests.getResultSummary): Update step and build summary based on webkitpy results.
+        (RunWebKitPyTests._addToLog): Method to add message to log.
+        * BuildSlaveSupport/ews-build/steps_unittest.py: Updated unit-test accordingly.
+
 2019-04-29  Alex Christensen  <achristensen@webkit.org>