Reduce the number of processes/tests run in parallel by test.dart

dart/tools/test.dart

 #!/usr/bin/env dart
 // Copyright (c) 2013, 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.
 
 /**
  * This file is the entrypoint of the dart test suite.  This suite is used
  * to test:
  *
  *     1. the dart vm
@@ skipping 138 matching lines @@
       if (verbose) {
         serverFutures.last.then((_) {
           var commandline = servers.httpServerCommandline();
           print('Started HttpServers: $commandline');
         });
       }
     }
 
     // There should not be more than one InternetExplorerDriver instance
     // running at a time. For details, see
     // http://code.google.com/p/selenium/wiki/InternetExplorerDriver.

[ahe, 2013/08/30 09:31:14]

Remove this comment.

[kustermann, 2013/08/30 09:34:39]

Done.

-    if (conf['runtime'].startsWith('ie') && !conf["use_browser_controller"]) {
+    if (conf['runtime'].startsWith('ie')) {

[ahe, 2013/08/30 09:31:14]

Could you include a comment inside this "then" block which explains the
motivation for these limits.

[kustermann, 2013/08/30 09:34:39]

Done.

+      maxProcesses = math.max(1, maxProcesses-2);
       maxBrowserProcesses = 1;
     } else if (conf['runtime'].startsWith('safari') &&
                conf['use_browser_controller']) {
       // Safari does not allow us to run from a fresh profile, so we can only
       // use one browser.
       maxBrowserProcesses = 1;
     }
 
     for (String key in selectors.keys) {
       if (key == 'co19') {
@@ skipping 128 matching lines @@
 void main() {
   deleteTemporaryDartDirectories().then((_) {
     var optionsParser = new TestOptionsParser();
     var configurations = optionsParser.parse(new Options().arguments);
     if (configurations != null && configurations.length > 0) {
       testConfigurations(configurations);
     }
   });
 }