results.html should have a link to historical results for a test/all failing tests
authortimothy_horton@apple.com <timothy_horton@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Nov 2013 17:42:14 +0000 (17:42 +0000)
committertimothy_horton@apple.com <timothy_horton@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Nov 2013 17:42:14 +0000 (17:42 +0000)
https://bugs.webkit.org/show_bug.cgi?id=124402

Reviewed by Simon Fraser.

* fast/harness/results.html:
Add a new rightmost column, 'history'.
Clicking on the column header will open the flakiness dashboard
for the entire set of failing tests; clicking on the link
in a particular test's row will open it just for that single test.

* fast/harness/resources/results-test.js:
Update the harness test to expect the new number of columns.

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

LayoutTests/ChangeLog
LayoutTests/fast/harness/resources/results-test.js
LayoutTests/fast/harness/results.html

index d7796ca..d0e3254 100644 (file)
@@ -1,3 +1,19 @@
+2013-11-15  Tim Horton  <timothy_horton@apple.com>
+
+        results.html should have a link to historical results for a test/all failing tests
+        https://bugs.webkit.org/show_bug.cgi?id=124402
+
+        Reviewed by Simon Fraser.
+
+        * fast/harness/results.html:
+        Add a new rightmost column, 'history'.
+        Clicking on the column header will open the flakiness dashboard
+        for the entire set of failing tests; clicking on the link
+        in a particular test's row will open it just for that single test.
+
+        * fast/harness/resources/results-test.js:
+        Update the harness test to expect the new number of columns.
+
 2013-11-15  Radu Stavila  <stavila@adobe.com>
 
         [CSS Regions] Implement visual overflow for first & last regions
index 5d0229b..fab319c 100644 (file)
@@ -344,10 +344,10 @@ function runTests()
     results.uses_expectations_file = false;
     runTest(results, function() {
         assertTrue(window.getComputedStyle(document.getElementById('results-table').parentNode, null)['display'] != 'none');
-        assertTrue(document.querySelectorAll('#results-table tbody td').length == 3);
+        assertTrue(document.querySelectorAll('#results-table tbody td').length == 4);
         assertTrue(!document.querySelector('tbody.expected'));
         assertTrue(!document.getElementById('passes-table'));
-        assertTrue(document.querySelectorAll('#flaky-tests-table tbody td').length == 4);
+        assertTrue(document.querySelectorAll('#flaky-tests-table tbody td').length == 5);
     });
 
     results = mockResults();
index 683525f..3a1b530 100644 (file)
@@ -607,6 +607,19 @@ function imageResultsCell(testObject, testPrefix, actual) {
     return row;
 }
 
+function flakinessDashboardURLForTests(testObjects)
+{
+    var testList = "";
+    for (var i = 0; i < testObjects.length; ++i) {
+        testList += testObjects[i].name;
+
+        if (i != testObjects.length - 1)
+            testList += ",";
+    }
+
+    return 'http://webkit-test-results.appspot.com/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=' + encodeURIComponent(testList);
+}
+
 function tableRow(testObject)
 {    
     var row = '<tbody'
@@ -652,6 +665,8 @@ function tableRow(testObject)
     if (globalState().results.uses_expectations_file)
         row += '<td>' + (actual.indexOf('MISSING') == -1 ? testObject.expected : '') + '</td>';
 
+    row += '<td><a href="' + flakinessDashboardURLForTests([testObject]) + '">history</a></td>';
+
     row += '</tr></tbody>';
     return row;
 }
@@ -1314,6 +1329,8 @@ function failingTestsTable(tests, title, id)
     if (globalState().results.uses_expectations_file)
         header += '<th>actual failure</th><th>expected failure</th>';
 
+    header += '<th><a href="' + flakinessDashboardURLForTests(tests) + '">history</a></th>';
+
     if (id == 'flaky-tests-table')
         header += '<th>failures</th>';