Migration of testing scripts in tools/ to libv2

tools/test-runtime.dart

 #!/usr/bin/env dart
 // Copyright (c) 2012, 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.
 
 // TODO(ager): Get rid of this version of test.dart when we don't have
 // to worry about the special runtime checkout anymore.
 // This file is identical to test.dart with test suites in the
 // directories samples, client, compiler, and utils removed.
 
@@ skipping 47 matching lines @@
     }
   }
   // Print the configurations being run by this execution of
   // test.dart. However, don't do it if the silent progress indicator
   // is used. This is only needed because of the junit tests.
   if (progressIndicator != 'silent') {
     List output_words = configurations.length > 1 ?
         ['Test configurations:'] : ['Test configuration:'];
     for (Map conf in configurations) {
       List settings =
-          ['compiler', 'runtime', 'mode', 'arch'].map((name) => conf[name]);
+          ['compiler', 'runtime', 'mode', 'arch'].mappedBy((name) => conf[name]);

[Bill Hesse, 2013/01/09 17:06:33]

You need toList here, unless you change settings to an iterator, don't you?

[kustermann, 2013/01/09 18:02:25]

Done.

       if (conf['checked']) settings.add('checked');
       output_words.add(Strings.join(settings, '_'));
     }
     print(Strings.join(output_words, ' '));
   }
 
-  var configurationIterator = configurations.iterator();
+  var configurationIterator = configurations.iterator;
   void enqueueConfiguration(ProcessQueue queue) {
-    if (!configurationIterator.hasNext) return;
+    if (!configurationIterator.moveNext()) return;
 
-    var conf = configurationIterator.next();
+    var conf = configurationIterator.current;
     if (selectors.containsKey('co19')) {
       queue.addTestSuite(new Co19TestSuite(conf));
     }
     if (conf['runtime'] == 'vm' && selectors.containsKey('vm')) {
       // vm tests contain both cc tests (added here) and dart tests (added in
       // [TEST_SUITE_DIRECTORIES]).
       queue.addTestSuite(new VMTestSuite(conf));
     }
 
     for (final testSuiteDir in TEST_SUITE_DIRECTORIES) {
       final name = testSuiteDir.filename;
       if (selectors.containsKey(name)) {
         queue.addTestSuite(
             new StandardTestSuite.forDirectory(conf, testSuiteDir));
       }
     }
   }
 
   // Start global http server that serves the entire dart repo.
   // The http server is available on localhost:9876 for any
   // test that needs to load resources from the repo over http.
   if (!listTests) {
     // Only start the server if we are running browser tests.
-    var runningBrowserTests = configurations.some((config) {
+    var runningBrowserTests = configurations.any((config) {
       return TestUtils.isBrowserRuntime(config['runtime']);
     });
     if (runningBrowserTests) startHttpServer('127.0.0.1', 9876);
   }
 
   // Start process queue.
   new ProcessQueue(
       maxProcesses,
       progressIndicator,
       startTime,
       printTiming,
       enqueueConfiguration,
       () => terminateHttpServer(),
       verbose,
       listTests);
 }