update flakiness dashboard after cutover to new test expectations syntax
authordpranke@chromium.org <dpranke@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 20 Sep 2012 01:13:32 +0000 (01:13 +0000)
committerdpranke@chromium.org <dpranke@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 20 Sep 2012 01:13:32 +0000 (01:13 +0000)
https://bugs.webkit.org/show_bug.cgi?id=97152

Unreviewed, build fix.

Handle (??) the new Bug notations as well. Hopefully we don't
still need the old ones.

* TestResultServer/static-dashboards/flakiness_dashboard.js:
(filterBugs):
(htmlForBugs):
* TestResultServer/static-dashboards/flakiness_dashboard_unittests.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@129080 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/TestResultServer/static-dashboards/flakiness_dashboard.js
Tools/TestResultServer/static-dashboards/flakiness_dashboard_unittests.js

index e78c4cf..01b82b3 100644 (file)
@@ -3,6 +3,21 @@
         update flakiness dashboard after cutover to new test expectations syntax
         https://bugs.webkit.org/show_bug.cgi?id=97152
 
+        Unreviewed, build fix.
+
+        Handle (??) the new Bug notations as well. Hopefully we don't
+        still need the old ones.
+
+        * TestResultServer/static-dashboards/flakiness_dashboard.js:
+        (filterBugs):
+        (htmlForBugs):
+        * TestResultServer/static-dashboards/flakiness_dashboard_unittests.js:
+
+2012-09-19  Dirk Pranke  <dpranke@chromium.org>
+
+        update flakiness dashboard after cutover to new test expectations syntax
+        https://bugs.webkit.org/show_bug.cgi?id=97152
+
         Reviewed by Ryosuke Niwa.
 
         This change clones the TestExpectation parsing state machine
index f09c57c..9428f9d 100644 (file)
@@ -505,7 +505,7 @@ function getExpectations(test, platform, buildType)
 
 function filterBugs(modifiers)
 {
-    var bugs = modifiers.match(/\bBUG\S*/g);
+    var bugs = modifiers.match(/\b(Bug|webkit.org|crbug.com|code.google.com)\S*/g);
     if (!bugs)
         return {bugs: '', modifiers: modifiers};
     for (var j = 0; j < bugs.length; j++)
@@ -610,7 +610,7 @@ function parsedExpectations()
             'Release': 'RELEASE',
             'Debug': 'DEBUG',
             'Mac': 'MAC',
-            'Win': 'Win',
+            'Win': 'WIN',
             'Linux': 'LINUX',
             'SnowLeopard': 'SNOWLEOPARD',
             'Lion': 'LION',
@@ -980,12 +980,10 @@ var WEBKIT_BUG_REPLACE_VALUE = BUG_URL_PREFIX + 'webkit.org/b' + WEBKIT_BUG_URL_
 
 function htmlForBugs(bugs)
 {
-    bugs = bugs.replace(/BUG(CR)?(\d{4})(\ |$)/g, EXTERNAL_BUG_REPLACE_VALUE);
-    bugs = bugs.replace(/BUG(CR)?(\d{5})(\ |$)/g, EXTERNAL_BUG_REPLACE_VALUE);
-    bugs = bugs.replace(/BUG(CR)?(1\d{5})(\ |$)/g, EXTERNAL_BUG_REPLACE_VALUE);
-    bugs = bugs.replace(/BUG(CR)?([2-9]\d{5})(\ |$)/g, INTERNAL_BUG_REPLACE_VALUE);
-    bugs = bugs.replace(/BUG(CR)?(\d{7})(\ |$)/g, INTERNAL_BUG_REPLACE_VALUE);
-    bugs = bugs.replace(/BUG(WK)(\d{5}\d*?)(\ |$)/g, WEBKIT_BUG_REPLACE_VALUE);
+    // FIXME: do we still need the INTERNAL_BUG replacements? They're
+    // broken temporarily.
+    bugs = bugs.replace(/crbug.com\/(\d+)(\ |$)/g, EXTERNAL_BUG_REPLACE_VALUE);
+    bugs = bugs.replace(/webkit.org\/b\/(\d{5}\d*?)(\ |$)/g, WEBKIT_BUG_REPLACE_VALUE);
     return bugs;
 }
 
index 8963063..966f4cd 100644 (file)
@@ -197,12 +197,12 @@ test('allTestsWithSamePlatformAndBuildType', 14, function() {
 });
 
 test('filterBugs',4, function() {
-    var filtered = filterBugs('SKIP BUG123 BUGCR123 BUGWK123 SLOW BUG_TONY DEBUG')
-    equal(filtered.modifiers, 'SKIP SLOW DEBUG');
-    equal(filtered.bugs, 'BUG123 BUGCR123 BUGWK123 BUG_TONY');
+    var filtered = filterBugs('Skip crbug.com/123 webkit.org/b/123 Slow Bug(Tony) Debug')
+    equal(filtered.modifiers, 'Skip Slow Debug');
+    equal(filtered.bugs, 'crbug.com/123 webkit.org/b/123 Bug(Tony)');
 
-    filtered = filterBugs('SKIP SLOW DEBUG')
-    equal(filtered.modifiers, 'SKIP SLOW DEBUG');
+    filtered = filterBugs('Skip Slow Debug')
+    equal(filtered.modifiers, 'Skip Slow Debug');
     equal(filtered.bugs, '');
 });