Add a test runner executable.

bin/unittest.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.unittest;
+
+import 'dart:async';
+import 'dart:io';
+import 'dart:isolate';
+
+import 'package:args/args.dart';
+import 'package:stack_trace/stack_trace.dart';
+import 'package:unittest/src/console_reporter.dart';
+import 'package:unittest/src/exit_codes.dart' as exit_codes;
+import 'package:unittest/src/io.dart';

[kevmoo, 2015/02/19 01:54:08]

unused import

[nweiz, 2015/02/19 02:10:18]

Done.

+import 'package:unittest/src/load_exception.dart';
+import 'package:unittest/src/loader.dart';
+import 'package:unittest/src/utils.dart';
+
+/// The argument parser used to parse the executable arguments.
+final _parser = new ArgParser();
+
+void main(List<String> args) {
+  _parser.addFlag("help", abbr: "h", negatable: false,

[kevmoo, 2015/02/19 01:54:08]

Just set these up in the field declaration. Use cascades. They don't sting.

[nweiz, 2015/02/19 02:10:18]

Setting them up here has several advantages. It's more flexible, so if we want
to e.g. show/hide a flag based on external considerations that's possible. It's
also more predictable, so we know exactly where the options are being added in
case there's an error in one of them.

+      help: "Shows this usage information.");
+  _parser.addOption("package-root", hide: true);
+
+  var options;
+  try {
+    options = _parser.parse(args);
+  } on FormatException catch (error) {
+    _printUsage(error.message);
+    exitCode = exit_codes.usage;
+    return;
+  }
+
+  if (options["help"]) {
+    _printUsage();
+    return;
+  }
+
+  var loader = new Loader(packageRoot: options["package-root"]);
+  new Future.sync(() {
+    var paths = options.rest;

[kevmoo, 2015/02/19 01:54:08]

Could all of this be outside of the Future.sync call?

Then your first async call can be Future.wait

[nweiz, 2015/02/19 02:10:18]

We need the [LoadException] to be handled by the [catchError] clause below.

+    if (paths.isEmpty) {
+      if (!new Directory("test").existsSync()) {
+        throw new LoadException("test",
+            "No test files were passed and the default directory doesn't "
+                "exist.");
+      }
+      paths = ["test"];
+    }
+
+    return Future.wait(paths.map((path) {

[kevmoo, 2015/02/19 01:54:08]

Could this be huge? Do we want to "pool" this to some reasonable number?

[nweiz, 2015/02/19 02:10:18]

We don't run them all at once, and loading them in isolates isn't very
expensive. I think it's fine.

+      if (new Directory(path).existsSync()) return loader.loadDir(path);
+      if (new File(path).existsSync()) return loader.loadFile(path);
+      throw new LoadException(path, 'Does not exist.');
+    }));
+  }).then((suites) {
+    var reporter = new ConsoleReporter(flatten(suites));
+    return reporter.run().then((success) {
+      exitCode = success ? 0 : 1;
+    }).whenComplete(() => reporter.close());
+  }).catchError((error, stackTrace) {
+    if (error is LoadException) {
+      // TODO(nweiz): color this message?
+      stderr.writeln(getErrorMessage(error));
+
+      // Only print stack traces for load errors that come from the user's 
+      if (error.innerError is! IOException &&
+          error.innerError is! IsolateSpawnException &&
+          error.innerError is! String) {
+        // TODO(nweiz): filter out runInIsolate and package:unittest stuff.
+        stderr.write(terseChain(stackTrace));
+      }
+
+      exitCode = error.innerError is IOException
+          ? exit_codes.io
+          : exit_codes.data;
+    } else {
+      stderr.writeln(getErrorMessage(error));
+      stderr.writeln(new Trace.from(stackTrace).terse);
+      stderr.writeln(
+          "This is an unexpected error. Please file an issue at "
+              "http://github.com/dart-lang/unittest\n"
+          "with the stack trace and instructions for reproducing the error.");
+      exitCode = exit_codes.software;
+    }
+  }).whenComplete(() => loader.close());
+}
+
+/// Print usage information for this command.
+///
+/// If [error] is passed, it's used in place of the usage message and the whole
+/// thing is printed to stderr instead of stdout.
+void _printUsage([String error]) {
+  var output = stdout;
+
+  var message = "Runs tests in this package.";
+  if (error != null) {
+    message = error;
+    output = stderr;
+  }
+
+  output.write("""$message
+
+Usage: pub run unittest:unittest [files or directories...]
+
+${_parser.usage}
+""");
+}