Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(417)

Side by Side Diff: tools/testing/dart/co19_test.dart

Issue 2908833002: Revert "Revert "Refactor test option parsing code."" (Closed)
Patch Set: Ignore "—failure-summary" so it doesn't break the bots. Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 /** 5 /**
6 * Tool for running co19 tests. Used when updating co19. 6 * Tool for running co19 tests. Used when updating co19.
7 * 7 *
8 * Currently, this tool is merely a convenience around multiple 8 * Currently, this tool is merely a convenience around multiple
9 * invocations of test.dart. Long term, we hope to evolve this into a 9 * invocations of test.dart. Long term, we hope to evolve this into a
10 * script that can automate most of the tasks necessary when updating 10 * script that can automate most of the tasks necessary when updating
11 * co19. 11 * co19.
12 * 12 *
13 * Usage: 13 * Usage:
14 * [: dart tools/testing/dart/co19_test.dart :] 14 * [: dart tools/testing/dart/co19_test.dart :]
15 */ 15 */
16 16
17 library co19_test; 17 library co19_test;
18 18
19 import "dart:io"; 19 import "dart:io";
20 20
21 import "test_options.dart"; 21 import "options.dart";
22 import "test_suite.dart"; 22 import "test_suite.dart";
23 import "test_configurations.dart"; 23 import "test_configurations.dart";
24 24
25 const List<String> COMMON_ARGUMENTS = const <String>[ 25 const List<String> COMMON_ARGUMENTS = const <String>[
26 '--report', 26 '--report',
27 '--progress=diff', 27 '--progress=diff',
28 'co19' 28 'co19'
29 ]; 29 ];
30 30
31 const List<List<String>> COMMAND_LINES = const <List<String>>[ 31 const List<List<String>> COMMAND_LINES = const <List<String>>[
(...skipping 20 matching lines...) Expand all
52 '-rdartium', 52 '-rdartium',
53 '-cnone', 53 '-cnone',
54 '--use-sdk', 54 '--use-sdk',
55 '--fast-startup' 55 '--fast-startup'
56 ], 56 ],
57 const <String>['-mrelease', '-rdartium', '-cnone', '--use-sdk'], 57 const <String>['-mrelease', '-rdartium', '-cnone', '--use-sdk'],
58 ]; 58 ];
59 59
60 void main(List<String> args) { 60 void main(List<String> args) {
61 TestUtils.setDartDirUri(Platform.script.resolve('../../..')); 61 TestUtils.setDartDirUri(Platform.script.resolve('../../..'));
62 var optionsParser = new TestOptionsParser(); 62 var optionsParser = new OptionsParser();
63 var configurations = <Map>[]; 63 var configurations = <Map>[];
64 for (var commandLine in COMMAND_LINES) { 64 for (var commandLine in COMMAND_LINES) {
65 var arguments = <String>[]; 65 var arguments = <String>[];
66 arguments.addAll(COMMON_ARGUMENTS); 66 arguments.addAll(COMMON_ARGUMENTS);
67 arguments.addAll(args); 67 arguments.addAll(args);
68 arguments.addAll(commandLine); 68 arguments.addAll(commandLine);
69 configurations.addAll(optionsParser.parse(arguments)); 69 configurations.addAll(optionsParser.parse(arguments));
70 } 70 }
71 71
72 if (configurations != null || configurations.length > 0) { 72 if (configurations != null || configurations.length > 0) {
73 testConfigurations(configurations); 73 testConfigurations(configurations);
74 } 74 }
75 } 75 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698