Add a --reporter flag and expose the expanded reporter.

lib/src/runner/reporter/expanded.dart

 // Copyright (c) 2015, 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.runner.reporter.no_io_compact;
 
 import 'dart:async';
 
 import '../../backend/live_test.dart';
 import '../../backend/state.dart';
 import '../../backend/suite.dart';
 import '../../utils.dart';
 import '../engine.dart';
 
 /// The maximum console line length.
 ///
 /// Lines longer than this will be cropped.
 const _lineLength = 100;
 
-// TODO(nweiz): Get rid of this when issue 6943 is fixed.
-/// A reporter that doesn't import `dart:io`, even transitively.
+/// A reporter that prints each test on its own line.
 ///
-/// This is used in place of [CompactReporter] by `lib/test.dart`, which
-/// can't transitively import `dart:io` but still needs access to a runner so
-/// that test files can be run directly.
-class NoIoCompactReporter {
+/// This is currently used in place of [CompactReporter] by `lib/test.dart`,
+/// which can't transitively import `dart:io` but still needs access to a runner
+/// so that test files can be run directly. This means that until issue 6943 is
+/// fixed, this must not import `dart:io`.
+class ExpandedReporter {
   /// The terminal escape for green text, or the empty string if this is Windows
   /// or not outputting to a terminal.
   final String _green;
 
   /// The terminal escape for red text, or the empty string if this is Windows
   /// or not outputting to a terminal.
   final String _red;
 
   /// The terminal escape for yellow text, or the empty string if this is
   /// Windows or not outputting to a terminal.
@@ skipping 17 matching lines @@
 
   /// The set of tests that have completed and been marked as passing.
   final _passed = new Set<LiveTest>();
 
   /// The set of tests that have completed and been marked as skipped.
   final _skipped = new Set<LiveTest>();
 
   /// The set of tests that have completed and been marked as failing or error.
   final _failed = new Set<LiveTest>();
 
+  /// The set of tests that are still running.
+  final _active = new List<LiveTest>();
+
   /// The size of [_passed] last time a progress notification was printed.
   int _lastProgressPassed;
 
   /// The size of [_skipped] last time a progress notification was printed.
   int _lastProgressSkipped;
 
   /// The size of [_failed] last time a progress notification was printed.
   int _lastProgressFailed;
 
   /// The message printed for the last progress notification.
   String _lastProgressMessage;
 
   /// Creates a [NoIoCompactReporter] that will run all tests in [suites].
   ///
   /// If [color] is `true`, this will use terminal colors; if it's `false`, it
   /// won't.
-  NoIoCompactReporter(Iterable<Suite> suites, {bool color: true})
+  ExpandedReporter(Iterable<Suite> suites, {int concurrency, bool color: true})

[kevmoo, 2015/04/23 00:13:14]

TODO: document concurrency here

[nweiz, 2015/04/23 00:15:52]

Done.

       : _multiplePaths = suites.map((suite) => suite.path).toSet().length > 1,
         _multiplePlatforms =
             suites.map((suite) => suite.platform).toSet().length > 1,
-        _engine = new Engine(suites),
+        _engine = new Engine(suites, concurrency: concurrency),
         _green = color ? '\u001b[32m' : '',
         _red = color ? '\u001b[31m' : '',
         _yellow = color ? '\u001b[33m' : '',
         _noColor = color ? '\u001b[0m' : '' {
     _engine.onTestStarted.listen((liveTest) {
+      if (_active.isEmpty) _progressLine(_description(liveTest));
+      _active.add(liveTest);
+
       liveTest.onStateChange.listen((state) {
         if (state.status != Status.complete) return;
+        _active.remove(liveTest);
 
         if (state.result != Result.success) {
           _passed.remove(liveTest);
           _failed.add(liveTest);
         } else if (liveTest.test.metadata.skip) {
           _skipped.add(liveTest);
         } else {
           _passed.add(liveTest);
         }
 
-        _progressLine(_description(liveTest));
-
         if (liveTest.test.metadata.skip &&
             liveTest.test.metadata.skipReason != null) {
+          _progressLine(_description(liveTest));
           print(indent('${_yellow}Skip: ${liveTest.test.metadata.skipReason}'
               '$_noColor'));
+        } else if (_active.isNotEmpty) {
+          // If any tests are running, display the name of the oldest active
+          // test.
+          _progressLine(_description(_active.first));
         }
       });
 
       liveTest.onError.listen((error) {
         if (liveTest.state.status != Status.complete) return;
 
         _progressLine(_description(liveTest));
         print(indent(error.error.toString()));
         print(indent(terseChain(error.stackTrace).toString()));
       });
 
       liveTest.onPrint.listen((line) {
         _progressLine(_description(liveTest));
         print(line);
       });
     });
   }
 
   /// Runs all tests in all provided suites.
   ///
   /// This returns `true` if all tests succeed, and `false` otherwise. It will
   /// only return once all tests have finished running.
   Future<bool> run() {
     if (_stopwatch.isRunning) {
-      throw new StateError("CompactReporter.run() may not be called more than "
+      throw new StateError("ExpandedReporter.run() may not be called more than "
           "once.");
     }
 
     if (_engine.liveTests.isEmpty) {
       print("No tests ran.");
       return new Future.value(true);
     }
 
     _stopwatch.start();
     return _engine.run().then((success) {
@@ skipping 30 matching lines @@
     _lastProgressPassed = _passed.length;
     _lastProgressSkipped = _skipped.length;
     _lastProgressFailed = _failed.length;
     _lastProgressMessage = message;
 
     if (color == null) color = '';
     var duration = _stopwatch.elapsed;
     var buffer = new StringBuffer();
 
     // \r moves back to the beginning of the current line.
-    buffer.write('\r${_timeString(duration)} ');
+    buffer.write('${_timeString(duration)} ');
     buffer.write(_green);
     buffer.write('+');
     buffer.write(_passed.length);
     buffer.write(_noColor);
 
     if (_skipped.isNotEmpty) {
       buffer.write(_yellow);
       buffer.write(' ~');
       buffer.write(_skipped.length);
       buffer.write(_noColor);
@@ skipping 32 matching lines @@
   /// This differs from the test's own description in that it may also include
   /// the suite's name.
   String _description(LiveTest liveTest) {
     var name = liveTest.test.name;
 
     if (_multiplePaths && liveTest.suite.path != null) {
       name = "${liveTest.suite.path}: $name";
     }
 
     if (_multiplePlatforms && liveTest.suite.platform != null) {
-      name = "[$liveTest.suite.platform] $name";
+      name = "[${liveTest.suite.platform}] $name";
     }
 
     return name;
   }
 }