Changeset 269959 in webkit

Timestamp:

Nov 18, 2020 8:13:21 AM (3 years ago)

Author:

aakash_jain@apple.com

Message:

[build.webkit.org] Update RunPerlTests step for new buildbot
​https://bugs.webkit.org/show_bug.cgi?id=219083

Reviewed by Jonathan Bedard.

• CISupport/build-webkit-org/steps.py:

(RunPerlTests):
(RunPerlTests.start): Initialized log observer.
(RunPerlTests.parseOutputLine): Parse each log line as they come.
(RunPerlTests.countFailures): Updated to return failedTestCount.

Location:

trunk/Tools

Files:

• CISupport/build-webkit-org/steps.py (modified) (1 diff)
• ChangeLog (modified) (1 diff)

trunk/Tools/CISupport/build-webkit-org/steps.py

@@ -821 +821 @@
     command = ["perl", "./Tools/Scripts/test-webkitperl"]
     failedTestsFormatString = "%d perl test%s failed"
+    test_summary_re = re.compile(r'^Failed \d+/\d+ test programs\. (?P<count>\d+)/\d+ subtests failed\.')  # e.g.: Failed 2/19 test programs. 5/363 subtests failed.
+
+    def start(self):
+        if USE_BUILDBOT_VERSION2:
+            self.log_observer = ParseByLineLogObserver(self.parseOutputLine)
+            self.addLogObserver('stdio', self.log_observer)
+            self.failedTestCount = 0
+        return shell.Test.start(self)
+
+    def parseOutputLine(self, line):
+        match = self.test_summary_re.match(line)
+        if match:
+            self.failedTestCount = int(match.group('count'))
 
     def countFailures(self, cmd):
+        if USE_BUILDBOT_VERSION2:
+            return self.failedTestCount
+
         logText = cmd.logs['stdio'].getText()
         # We're looking for the line that looks like this: Failed 2/19 test programs. 5/363 subtests failed.

trunk/Tools/ChangeLog

@@ +1 @@
+2020-11-18  Aakash Jain  <aakash_jain@apple.com>
+
+        [build.webkit.org] Update RunPerlTests step for new buildbot
+        https://bugs.webkit.org/show_bug.cgi?id=219083
+
+        Reviewed by Jonathan Bedard.
+
+        * CISupport/build-webkit-org/steps.py:
+        (RunPerlTests):
+        (RunPerlTests.start): Initialized log observer.
+        (RunPerlTests.parseOutputLine): Parse each log line as they come.
+        (RunPerlTests.countFailures): Updated to return failedTestCount.
+
 2020-11-18  Aakash Jain  <aakash_jain@apple.com>