Move test.dart into testing/dart.

tools/testing/dart/test_progress.dart

 // Copyright (c) 2013, 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.
 
 library test_progress;
 
-import "dart:async";
 import "dart:io";
 import "dart:io" as io;
 import "dart:convert" show JSON;
 import "path.dart";
 import "status_file_parser.dart";
 import "summary_report.dart";
 import "test_runner.dart";
 import "test_suite.dart";
 import "utils.dart";
 
@@ skipping 477 matching lines @@
   }
 
   void allDone() {
     if (_skippedCompilations > 0) {
       print('\n$_skippedCompilations compilations were skipped because '
           'the previous output was already up to date\n');
     }
   }
 }
 
+/// Discovers directories in the system temp directory that appear to have been
+/// created by "dart:io"'s `Directory.createTemp()` method.
 class LeftOverTempDirPrinter extends EventListener {
-  final MIN_NUMBER_OF_TEMP_DIRS = 50;
+  /// The minimum number of left over temp directories that is considered
+  /// excessive and causes a warning to be shown.
+  static const _excessiveCount = 50;
 
-  static RegExp _getTemporaryDirectoryRegexp() {
+  static RegExp get _tempDirPattern {
     // These are the patterns of temporary directory names created by
     // 'Directory.systemTemp.createTemp()' on linux/macos and windows.
     if (['macos', 'linux'].contains(Platform.operatingSystem)) {
       return new RegExp(r'^temp_dir1_......$');
     } else {
       return new RegExp(r'tempdir-........-....-....-....-............$');
     }
   }
 
-  static Stream<FileSystemEntity> getLeftOverTemporaryDirectories() {
-    var regExp = _getTemporaryDirectoryRegexp();
-    return Directory.systemTemp.list().where((FileSystemEntity fse) {
-      if (fse is Directory) {
-        if (regExp.hasMatch(new Path(fse.path).filename)) {
-          return true;
-        }
+  /// Returns a stream of all of the apparently Dart-created temp directories.
+  static Iterable<Directory> list() {
+    return Directory.systemTemp
+        .listSync()
+        .where((FileSystemEntity entity) =>
+            entity is Directory &&
+            _tempDirPattern.hasMatch(new Path(entity.path).filename))
+        .map((entity) => entity as Directory);
+  }
+
+  /// Attempts to delete all Dart-created temp directories.
+  ///
+  /// If any deletion fails, logs it and continues to delete others.
+  static void deleteAll() {
+    for (var directory in LeftOverTempDirPrinter.list()) {
+      try {
+        directory.deleteSync(recursive: true);
+      } catch (error) {
+        DebugLogger.error(error);
       }
-      return false;
-    });
+    }
   }
 
   void allDone() {

[Bill Hesse, 2017/05/02 14:33:09]

Could you try to locally test this code, by adding a test that creates 100 temp
directories?  And is it really true, about tempdir- and temp_dir1_ ?

[Bob Nystrom, 2017/05/02 23:48:15]

Uh, now that you mention it, no, it does not seem to be true. On my Mac, if I
call:

  print(Directory.systemTemp.createTempSync().path);

I get something like:

  /var/folders/vw/b46gfymx10ddqfg1m_tr72sw003cg5/T/hLqmvN

No "tempdir1" prefix at all. Likewise if I pass a prefix to createTempSync(), I
get:

  /var/folders/vw/b46gfymx10ddqfg1m_tr72sw003cg5/T/prefixcZM6RZ

Looks like the hardcoded "temp_dir" prefixes have been gone since 2013
(https://github.com/dart-lang/sdk/commit/b221c702a3db269e3db91ad88be205e0bcb48...
I'll remove this code completely.

-    getLeftOverTemporaryDirectories().length.then((int count) {
-      if (count > MIN_NUMBER_OF_TEMP_DIRS) {
-        DebugLogger.warning("There are ${count} directories "
-            "in the system tempdir "
-            "('${Directory.systemTemp.path}')! "
-            "Maybe left over directories?\n");
+    try {
+      var count = list().length;
+      if (count > _excessiveCount) {
+        DebugLogger
+            .warning("There are $count directories in the system tempdir "
+                "('${Directory.systemTemp.path}'). "
+                "Maybe left over directories?\n");
       }
-    }).catchError((error) {
+    } catch (error) {
       DebugLogger.warning("Could not list temp directories, got: $error");
-    });
+    }
   }
 }
 
 class LineProgressIndicator extends EventListener {
   void done(TestCase test) {
     var status = 'pass';
     if (test.unexpectedOutput) {
       status = 'fail';
     }
     print('Done ${test.configurationString} ${test.displayName}: $status');
@@ skipping 166 matching lines @@
       return new VerboseProgressIndicator(startTime);
     case 'status':
       return new ProgressIndicator(startTime);
     case 'buildbot':
       return new BuildbotProgressIndicator(startTime);
     default:
       assert(false);
       return null;
   }
 }