Added support for logging the output of failed flaky tests.

tools/testing/dart/test_suite.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /**
  * Classes and methods for enumerating and preparing tests.
  *
  * This library includes:
  *
  * - Creating tests by listing all the Dart files in certain directories,
@@ skipping 1333 matching lines @@
     return d8;
   }
 
   static String vmFileName(Map configuration) {
     var suffix = executableSuffix('vm');
     var vm = '${buildDir(configuration)}/dart$suffix';
     ensureExists(vm, configuration);
     return vm;
   }
 
+  static String flakyFileName() {
+    // If a flaky test did fail, infos about it (i.e. test name, stdin, stdout)

[ricow1, 2012/11/08 12:28:50]

s/infos/info

+    // will be written to this file. This is useful for the debugging of

[ricow1, 2012/11/08 12:28:50]

This is useful for debugging flaky tests.

+    // flaky tests.
+    // When running on a built bot, the file can be made visible in the waterfall UI.

[ricow1, 2012/11/08 12:28:50]

Long line

[ricow1, 2012/11/08 12:28:50]

s/built bot/buildbot

+    return ".flaky.log";

[ahe, 2012/11/09 10:35:02]

I suggest that you remove the first dot.

+  }
+
   static void ensureExists(String filename, Map configuration) {
     if (!configuration['list'] && !(new File(filename).existsSync())) {
       throw "Executable '$filename' does not exist";
     }
   }
 
   static String compilerPath(Map configuration) {
     if (configuration['compiler'] == 'none') {
       return null;  // No separate compiler for dartium tests.
     }
@@ skipping 129 matching lines @@
  * $pass tests are expected to pass
  * $failOk tests are expected to fail that we won't fix
  * $fail tests are expected to fail that we should fix
  * $crash tests are expected to crash that we should fix
  * $timeout tests are allowed to timeout
  * $compileErrorSkip tests are skipped on browsers due to compile-time error
 """;
     print(report);
    }
 }