Chromium Code Reviews| Index: tools/test.dart |
| diff --git a/tools/test.dart b/tools/test.dart |
| index 8d7388c0a2e29a4b6d4e39133614263e7575d189..e76bbe0ca26f3a15726d9f206081fcd56ea5e636 100755 |
| --- a/tools/test.dart |
| +++ b/tools/test.dart |
| @@ -97,18 +97,18 @@ main() { |
| ['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
toList()
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; |
| for (String key in selectors.keys) { |
| if (key == 'co19') { |
| queue.addTestSuite(new Co19TestSuite(conf)); |
| @@ -136,7 +136,7 @@ main() { |
| // 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); |