Remove special case for layout tests when builders fail to load
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 20 Nov 2012 23:56:48 +0000 (23:56 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 20 Nov 2012 23:56:48 +0000 (23:56 +0000)
https://bugs.webkit.org/show_bug.cgi?id=102842

Reviewed by Tony Chang.

Now that we load only builders that actually run the given test
type, we can reliably show errors to the users because it
represents a broken bot.
* TestResultServer/static-dashboards/loader.js:

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

Tools/ChangeLog
Tools/TestResultServer/static-dashboards/loader.js
Tools/TestResultServer/static-dashboards/loader_unittests.js

index cc8a1b0aa750e3029174e68ab34a8aca9c77e8b3..9e6880dc1d3564fc9952c47ea5f3000745f9df16 100644 (file)
@@ -1,3 +1,15 @@
+2012-11-20  Ojan Vafai  <ojan@chromium.org>
+
+        Remove special case for layout tests when builders fail to load
+        https://bugs.webkit.org/show_bug.cgi?id=102842
+
+        Reviewed by Tony Chang.
+
+        Now that we load only builders that actually run the given test
+        type, we can reliably show errors to the users because it
+        represents a broken bot.
+        * TestResultServer/static-dashboards/loader.js:
+
 2012-11-15  Ojan Vafai  <ojan@chromium.org>
 
         Generate a list of builders/test suites from the buildbot json
index 0d7f7efb4530796803c45acd69c60ca1daaa608e..c748901617b8e47857cb68f43a55f7050e99ee81 100644 (file)
@@ -160,16 +160,10 @@ loader.Loader.prototype = {
     },
     _handleResultsFileLoadError: function(builderName)
     {
-        var error = 'Failed to load results file for ' + builderName + '.';
-
-        if (isLayoutTestResults()) {
-            console.error(error);
-            g_buildersThatFailedToLoad.push(builderName);
-        } else {
-            // Avoid to show error/warning messages for non-layout tests. We may be
-            // checking the builders that are not running the tests.
-            console.info('info:' + error);
-        }
+        console.error('Failed to load results file for ' + builderName + '.');
+
+        // FIXME: loader shouldn't depend on state defined in dashboard_base.js.
+        g_buildersThatFailedToLoad.push(builderName);
 
         // Remove this builder from builders, so we don't try to use the
         // data that isn't there.
index f8b6e9e3d24676dd2b16c962de4939a857282e1e..881e0179ad0bff596ddd648138d8a1709db7ec5f 100644 (file)
@@ -107,3 +107,27 @@ test('expectations files loading', 1, function() {
         loader.request = requestFunction;
     }
 });
+
+test('results file failing to load', 2, function() {
+    // FIXME: loader shouldn't depend on state defined in dashboard_base.js.
+    g_buildersThatFailedToLoad = [];
+    g_builders = {};
+
+    var resourceLoader = new loader.Loader();
+    var resourceLoadCount = 0;
+    resourceLoader._handleResourceLoad = function() {
+        resourceLoadCount++;
+    }
+
+    var builder1 = 'builder1';
+    g_builders[builder1] = true;
+    resourceLoader._handleResultsFileLoadError(builder1);
+
+    var builder2 = 'builder2';
+    g_builders[builder2] = true;
+    resourceLoader._handleResultsFileLoadError(builder2);
+
+    deepEqual(g_buildersThatFailedToLoad, [builder1, builder2]);
+    equal(resourceLoadCount, 2);
+
+})