Add a flag to control the use of color in the test runner.

lib/src/runner/console_reporter.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 unittest.runner.console_reporter;
 
 import 'dart:async';
 import 'dart:io';
 
 import '../backend/live_test.dart';
 import '../backend/state.dart';
 import '../backend/suite.dart';
-import '../util/io.dart';
 import '../utils.dart';
 import 'engine.dart';
 
-/// The terminal escape for green text, or the empty string if this is Windows
-/// or not outputting to a terminal.
-final _green = getSpecial('\u001b[32m');
-
-/// The terminal escape for red text, or the empty string if this is Windows or
-/// not outputting to a terminal.
-final _red = getSpecial('\u001b[31m');
-
-/// The terminal escape for removing test coloring, or the empty string if this
-/// is Windows or not outputting to a terminal.
-final _noColor = getSpecial('\u001b[0m');
-
 /// The maximum console line length.
 ///
 /// Lines longer than this will be cropped.
 const _lineLength = 100;
 
 /// A reporter that prints test results to the console in a single
 /// continuously-updating line.
 class ConsoleReporter {
+  /// The terminal escape for green text, or the empty string if this is Windows
+  /// or not outputting to a terminal.
+  final String _green;

[Bob Nystrom, 2015/03/03 00:28:57]

Not that it's harmful, but it feels weird to me to store these as fields. Maybe
just store _color and make these getters?

[nweiz, 2015/03/03 00:46:43]

It is a little weird, but it's also less code and less indirection, which I
like.

+
+  /// 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 removing test coloring, or the empty string if this

[Bob Nystrom, 2015/03/03 00:28:57]

Long lines.

[nweiz, 2015/03/03 00:46:43]

Done.

[nweiz, 2015/03/03 00:46:43]

Done.

+  /// is Windows or not outputting to a terminal.
+  final String _noColor;
+
   /// The engine used to run the tests.
   final Engine _engine;
 
   /// Whether multiple test suites are being run.
   final bool _multipleSuites;
 
   /// A stopwatch that tracks the duration of the full run.
   final _stopwatch = new Stopwatch();
 
   /// 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 failing or error.
   final _failed = new Set<LiveTest>();
 
   /// The size of [_passed] last time a progress notification was printed.
   int _lastProgressPassed;
 
   /// 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 [ConsoleReporter] that will run all tests in [suites].
-  ConsoleReporter(Iterable<Suite> suites)
+  ///
+  /// If [color] is `true`, this will use terminal colors; if it's `false`, it
+  /// won't.
+  ConsoleReporter(Iterable<Suite> suites, {bool color: true})
       : _multipleSuites = suites.length > 1,
-        _engine = new Engine(suites) {
-
+        _engine = new Engine(suites),
+        _green = color ? '\u001b[32m' : '',
+        _red = color ? '\u001b[31m' : '',
+        _noColor = color ? '\u001b[0m' : '' {
     _engine.onTestStarted.listen((liveTest) {
       _progressLine(_description(liveTest));
       liveTest.onStateChange.listen((state) {
         if (state.status != Status.complete) return;
         if (state.result == Result.success) {
           _passed.add(liveTest);
         } else {
           _passed.remove(liveTest);
           _failed.add(liveTest);
         }
@@ skipping 146 matching lines @@
 
   /// Returns a description of [liveTest].
   ///
   /// This differs from the test's own description in that it may also include
   /// the suite's name.
   String _description(LiveTest liveTest) {
     if (_multipleSuites) return "${liveTest.suite.name}: ${liveTest.test.name}";
     return liveTest.test.name;
   }
 }