Changeset 247646 in webkit

Timestamp:

Jul 19, 2019 8:54:29 AM (5 years ago)

Author:

aakash_jain@apple.com

Message:

[ews-build] Display pre-existing Layout test failure names in the build summary
​https://bugs.webkit.org/show_bug.cgi?id=199941

Reviewed by Jonathan Bedard.

• BuildSlaveSupport/ews-build/steps.py:

(AnalyzeLayoutTestsResults.report_pre_existing_failures):

Location:

trunk/Tools

Files:

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

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

@@ -1028 +1028 @@
         self.descriptionDone = 'Passed layout tests'
         pluralSuffix = 's' if len(clean_tree_failures) > 1 else ''
-        message = 'Found {} pre-existing test failure{}'.format(len(clean_tree_failures), pluralSuffix)
+        clean_tree_failures_string = ', '.join([failure_name for failure_name in clean_tree_failures])
+        message = 'Found {} pre-existing test failure{}: {}'.format(len(clean_tree_failures), pluralSuffix, clean_tree_failures_string)
         self.build.buildFinished([message], SUCCESS)
         return defer.succeed(None)

trunk/Tools/ChangeLog

@@ +1 @@
+2019-07-19  Aakash Jain  <aakash_jain@apple.com>
+
+        [ews-build] Display pre-existing Layout test failure names in the build summary
+        https://bugs.webkit.org/show_bug.cgi?id=199941
+
+        Reviewed by Jonathan Bedard.
+
+        * BuildSlaveSupport/ews-build/steps.py:
+        (AnalyzeLayoutTestsResults.report_pre_existing_failures):
+
 2019-07-19  Carlos Alberto Lopez Perez  <clopez@igalia.com>