Changeset 181116 in webkit

Timestamp:

Mar 5, 2015, 4:40:15 PM (10 years ago)

Author:

ddkilzer@apple.com

Message:

Dashboard has trouble parsing iOS test output
<​http://webkit.org/b/142364>

Reviewed by Alex Christensen.

• BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotTestResults.js:

(BuildbotTestResults.prototype._parseResults.resultSummarizer):
Add fallback path to build a regex using 'matchString' so that
it matches anywhere in the line. In practice, 'matchString' is
a prefix for the label following the number.

Location:

trunk/Tools

Files:

• BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotTestResults.js (modified) (1 diff)
• ChangeLog (modified) (1 diff)

trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotTestResults.js

@@ -74 +74 @@
         {
             var match = /^(\d+)\s/.exec(outputLine);
+            if (!match) {
+                var regex = new RegExp("(\\d+)\\s" + matchString);
+                match = regex.exec(outputLine);
+            }
             if (!match)
                 return sum;

trunk/Tools/ChangeLog

@@ +1 @@
+2015-03-05  David Kilzer  <ddkilzer@apple.com>
+
+        Dashboard has trouble parsing iOS test output
+        <http://webkit.org/b/142364>
+
+        Reviewed by Alex Christensen.
+
+        * BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotTestResults.js:
+        (BuildbotTestResults.prototype._parseResults.resultSummarizer):
+        Add fallback path to build a regex using 'matchString' so that
+        it matches anywhere in the line.  In practice, 'matchString' is
+        a prefix for the label following the number.
+
 2015-03-05  Alex Christensen  <achristensen@webkit.org>