Add names and plain_names fields.

lib/src/runner/configuration/args.dart

 // Copyright (c) 2016, 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.
 
 import 'dart:io';
 
 import 'package:args/args.dart';
 import 'package:boolean_selector/boolean_selector.dart';
 
 import '../../backend/test_platform.dart';
@@ skipping 16 matching lines @@
   parser.addOption("package-root", hide: true);
 
   // Note that defaultsTo declarations here are only for documentation purposes.
   // We pass null values rather than defaults to [new Configuration] so that it
   // merges properly with the config file.
 
   parser.addSeparator("======== Selecting Tests");
   parser.addOption("name",
       abbr: 'n',
       help: 'A substring of the name of the test to run.\n'
-          'Regular expression syntax is supported.');
+          'Regular expression syntax is supported.\n'
+          'If passed multiple times, tests must match all substrings.',
+      allowMultiple: true,
+      splitCommas: false);
   parser.addOption("plain-name",
       abbr: 'N',
-      help: 'A plain-text substring of the name of the test to run.');
-  // TODO(nweiz): Support the full platform-selector syntax for choosing which
-  // tags to run. In the shorter term, disallow non-"identifier" tags.
+      help: 'A plain-text substring of the name of the test to run.\n'
+          'If passed multiple times, tests must match all substrings.',
+      allowMultiple: true,
+      splitCommas: false);
   parser.addOption("tags",
       abbr: 't',
       help: 'Run only tests with all of the specified tags.\n'
           'Supports boolean selector syntax.',
       allowMultiple: true);
   parser.addOption("tag", hide: true, allowMultiple: true);
   parser.addOption("exclude-tags",
       abbr: 'x',
       help: "Don't run tests with any of the specified tags.\n"
           "Supports boolean selector syntax.",
@@ skipping 51 matching lines @@
 
 /// The usage string for the command-line arguments.
 String get usage => _parser.usage;
 
 /// Parses the configuration from [args].
 ///
 /// Throws a [FormatException] if [args] are invalid.
 Configuration parse(List<String> args) {
   var options = _parser.parse(args);
 
-  var pattern;
-  if (options['name'] != null) {
-    if (options["plain-name"] != null) {
-      throw new FormatException(
-          "--name and --plain-name may not both be passed.");
-    }
-
-    pattern = _wrapFormatException(
-        options, 'name', (value) => new RegExp(value));
-  } else if (options['plain-name'] != null) {
-    pattern = options['plain-name'];
-  }
+  var patterns = options['name']
+      .map((value) => _wrapFormatException('name', () => new RegExp(value)))
+      .toList()
+      ..addAll(options['plain-name']);
 
   var includeTagSet = new Set.from(options['tags'] ?? [])
     ..addAll(options['tag'] ?? []);
 
   var includeTags = includeTagSet.fold(BooleanSelector.all, (selector, tag) {
     var tagSelector = new BooleanSelector.parse(tag);
     return selector.intersection(tagSelector);
   });
 
   var excludeTagSet = new Set.from(options['exclude-tags'] ?? [])
@@ skipping 11 matching lines @@
 
   return new Configuration(
       help: ifParsed('help'),
       version: ifParsed('version'),
       verboseTrace: ifParsed('verbose-trace'),
       jsTrace: ifParsed('js-trace'),
       pauseAfterLoad: ifParsed('pause-after-load'),
       color: ifParsed('color'),
       packageRoot: ifParsed('package-root'),
       reporter: ifParsed('reporter'),
-      pubServePort: _wrapFormatException(options, 'pub-serve', int.parse),
-      concurrency: _wrapFormatException(options, 'concurrency', int.parse),
-      timeout: _wrapFormatException(options, 'timeout',
+      pubServePort: _parseOption(options, 'pub-serve', int.parse),
+      concurrency: _parseOption(options, 'concurrency', int.parse),
+      timeout: _parseOption(options, 'timeout',
           (value) => new Timeout.parse(value)),
-      pattern: pattern,
+      patterns: patterns,
       platforms: ifParsed('platform')?.map(TestPlatform.find),
       chosenPresets: ifParsed('preset'),
       paths: options.rest.isEmpty ? null : options.rest,
       includeTags: includeTags,
       excludeTags: excludeTags);
 }
 
 /// Runs [parse] on the value of the option [name], and wraps any
 /// [FormatException] it throws with additional information.
-_wrapFormatException(ArgResults options, String name, parse(value)) {
+_parseOption(ArgResults options, String name, parse(value)) {
   if (!options.wasParsed(name)) return null;
 
   var value = options[name];
   if (value == null) return null;
 
+  return _wrapFormatException(name, () => parse(value));
+}
+
+/// Runs [parse], and wraps any [FormatException] it throws with additional
+/// information.
+_wrapFormatException(String name, parse()) {
   try {
-    return parse(value);
+    return parse();
   } on FormatException catch (error) {
     throw new FormatException('Couldn\'t parse --$name "${options[name]}": '
         '${error.message}');
   }
 }