Title: [265610] trunk/Tools
Revision
265610
Author
aakash_j...@apple.com
Date
2020-08-13 09:01:04 -0700 (Thu, 13 Aug 2020)

Log Message

[ews] test names in ews notification email should be in sorted order
https://bugs.webkit.org/show_bug.cgi?id=215446

Reviewed by Jonathan Bedard.

* BuildSlaveSupport/ews-build/steps.py:
(AnalyzeLayoutTestsResults.send_email_for_new_test_failures):

Modified Paths

Diff

Modified: trunk/Tools/BuildSlaveSupport/ews-build/steps.py (265609 => 265610)


--- trunk/Tools/BuildSlaveSupport/ews-build/steps.py	2020-08-13 15:59:34 UTC (rev 265609)
+++ trunk/Tools/BuildSlaveSupport/ews-build/steps.py	2020-08-13 16:01:04 UTC (rev 265610)
@@ -2150,7 +2150,7 @@
             patch_author = self.getProperty('patch_author', '')
             build_url = '{}#/builders/{}/builds/{}'.format(self.master.config.buildbotURL, self.build._builderid, self.build.number)
             test_names_string = ''
-            for test_name in test_names:
+            for test_name in sorted(test_names):
                 history_url = '{}?suite=layout-tests&test={}'.format(RESULTS_DB_URL, test_name)
                 test_names_string += '\n- {} (<a href="" history</a>)'.format(test_name, history_url)
 

Modified: trunk/Tools/ChangeLog (265609 => 265610)


--- trunk/Tools/ChangeLog	2020-08-13 15:59:34 UTC (rev 265609)
+++ trunk/Tools/ChangeLog	2020-08-13 16:01:04 UTC (rev 265610)
@@ -1,5 +1,15 @@
 2020-08-13  Aakash Jain  <aakash_j...@apple.com>
 
+        [ews] test names in ews notification email should be in sorted order
+        https://bugs.webkit.org/show_bug.cgi?id=215446
+
+        Reviewed by Jonathan Bedard.
+
+        * BuildSlaveSupport/ews-build/steps.py:
+        (AnalyzeLayoutTestsResults.send_email_for_new_test_failures):
+
+2020-08-13  Aakash Jain  <aakash_j...@apple.com>
+
         [ews] Improve wording for test failure email
         https://bugs.webkit.org/show_bug.cgi?id=215449
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to