Pass Configuration to Loader and Server.

lib/src/runner/configuration.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.configuration;
 
 import 'dart:io';
 import 'dart:math' as math;
 
 import 'package:args/args.dart';
 import 'package:path/path.dart' as p;
 
 import '../frontend/timeout.dart';
+import '../backend/metadata.dart';
 import '../backend/test_platform.dart';
 import '../util/io.dart';
 
 /// The default number of test suites to run at once.
 ///
 /// This defaults to half the available processors, since presumably some of
 /// them will be used for the OS and other processes.
 final _defaultConcurrency = math.max(1, Platform.numberOfProcessors ~/ 2);
 
 /// A class that encapsulates the command-line configuration of the test runner.
@@ skipping 99 matching lines @@
   /// Whether the load paths were passed explicitly or the default was used.
   final bool explicitPaths;
 
   /// The pattern to match against test names to decide which to run, or `null`
   /// if all tests should be run.
   final Pattern pattern;
 
   /// The set of platforms on which to run tests.
   final List<TestPlatform> platforms;
 
+  /// The global test metadata derived from this configuration.
+  Metadata get metadata =>
+      new Metadata(
+          timeout: pauseAfterLoad ? Timeout.none : null,
+          verboseTrace: verboseTrace);
+
   /// Parses the configuration from [args].
   ///
   /// Throws a [FormatException] if [args] are invalid.
   factory Configuration.parse(List<String> args) {
     var options = _parser.parse(args);
 
     var pattern;
     if (options['name'] != null) {
       if (options["plain-name"] != null) {
         throw new FormatException(
@@ skipping 12 matching lines @@
         verboseTrace: options['verbose-trace'],
         jsTrace: options['js-trace'],
         pauseAfterLoad: options['pause-after-load'],
         color: options['color'],
         packageRoot: options['package-root'],
         reporter: options['reporter'],
         pubServePort: _wrapFormatException(options, 'pub-serve', int.parse),
         concurrency: _wrapFormatException(options, 'concurrency', int.parse,
             orElse: () => _defaultConcurrency),
         pattern: pattern,
-        platforms: options['platforms'].map(TestPlatform.find).toList(),
+        platforms: options['platform'].map(TestPlatform.find),

[kevmoo, 2015/08/04 22:07:44]

was this mis-typed before?

[nweiz, 2015/08/04 22:20:56]

Yeah.

         paths: options.rest.isEmpty ? null : options.rest);
   }
 
   /// Runs [parse] on the value of the option [name], and wraps any
   /// [FormatException] it throws with additional information.
   static _wrapFormatException(ArgResults options, String name, parse(value),
       {orElse()}) {
     var value = options[name];
     if (value == null) return orElse == null ? null : orElse();
 
@@ skipping 15 matching lines @@
         packageRoot = packageRoot == null
             ? p.join(p.current, 'packages')
             : packageRoot,
         reporter = reporter == null ? 'compact' : reporter,
         pubServeUrl = pubServePort == null
             ? null
             : Uri.parse("http://localhost:$pubServePort"),
         concurrency = pauseAfterLoad
             ? 1
             : (concurrency == null ? _defaultConcurrency : concurrency),
-        platforms = platforms.toList(),
+        platforms = platforms == null ? [TestPlatform.vm] : platforms.toList(),
         paths = paths == null ? ["test"] : paths.toList(),
         explicitPaths = paths != null;
 }