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

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

Issue 1126363003: Remove dart2dart support from testing scripts and status files (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: address comments Created 5 years, 7 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 | Annotate | Revision Log
« no previous file with comments | « tools/testing/dart/status_reporter.dart ('k') | tools/testing/dart/test_runner.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, 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 library test_options_parser; 5 library test_options_parser;
6 6
7 import "dart:io"; 7 import "dart:io";
8 import "drt_updater.dart"; 8 import "drt_updater.dart";
9 import "test_suite.dart"; 9 import "test_suite.dart";
10 import "path.dart"; 10 import "path.dart";
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
53 ['-m', '--mode'], 53 ['-m', '--mode'],
54 ['all', 'debug', 'release'], 54 ['all', 'debug', 'release'],
55 'debug'), 55 'debug'),
56 new _TestOptionSpecification( 56 new _TestOptionSpecification(
57 'compiler', 57 'compiler',
58 '''Specify any compilation step (if needed). 58 '''Specify any compilation step (if needed).
59 59
60 none: Do not compile the Dart code (run native Dart code on the VM). 60 none: Do not compile the Dart code (run native Dart code on the VM).
61 (only valid with the following runtimes: vm, drt) 61 (only valid with the following runtimes: vm, drt)
62 62
63 dart2dart: Compile Dart code to Dart code
64 (only valid with the following runtimes: vm, drt)
65
66 dart2js: Compile dart code to JavaScript by running dart2js. 63 dart2js: Compile dart code to JavaScript by running dart2js.
67 (only valid with the following runtimes: d8, drt, chrome, 64 (only valid with the following runtimes: d8, drt, chrome,
68 safari, ie9, ie10, ie11, firefox, opera, chromeOnAndroid, 65 safari, ie9, ie10, ie11, firefox, opera, chromeOnAndroid,
69 none (compile only)), 66 none (compile only)),
70 67
71 dartanalyzer: Perform static analysis on Dart code by running the analyzer on Java. 68 dartanalyzer: Perform static analysis on Dart code by running the analyzer on Java.
72 dart2analyzer: Perform static analysis on Dart code by running the analyzer o n Dart. 69 dart2analyzer: Perform static analysis on Dart code by running the analyzer o n Dart.
73 (only valid with the following runtimes: none)''', 70 (only valid with the following runtimes: none)''',
74 ['-c', '--compiler'], 71 ['-c', '--compiler'],
75 ['none', 'dart2dart', 'dart2js', 'dartanalyzer', 'dart2analyzer'], 72 ['none', 'dart2js', 'dartanalyzer', 'dart2analyzer'],
76 'none'), 73 'none'),
77 // TODO(antonm): fix the option drt. 74 // TODO(antonm): fix the option drt.
78 new _TestOptionSpecification( 75 new _TestOptionSpecification(
79 'runtime', 76 'runtime',
80 '''Where the tests should be run. 77 '''Where the tests should be run.
81 vm: Run Dart code on the standalone dart vm. 78 vm: Run Dart code on the standalone dart vm.
82 79
83 d8: Run JavaScript from the command line using v8. 80 d8: Run JavaScript from the command line using v8.
84 81
85 jsshell: Run JavaScript from the command line using firefox js-shell. 82 jsshell: Run JavaScript from the command line using firefox js-shell.
(...skipping 558 matching lines...) Expand 10 before | Expand all | Expand 10 after
644 // with dart2js, we should remove it from here. 641 // with dart2js, we should remove it from here.
645 validRuntimes = const ['d8', 'jsshell', 'drt', 'none', 'dartium', 642 validRuntimes = const ['d8', 'jsshell', 'drt', 'none', 'dartium',
646 'ff', 'chrome', 'safari', 'ie9', 'ie10', 'ie11', 643 'ff', 'chrome', 'safari', 'ie9', 'ie10', 'ie11',
647 'opera', 'chromeOnAndroid', 'safarimobilesim']; 644 'opera', 'chromeOnAndroid', 'safarimobilesim'];
648 break; 645 break;
649 case 'dartanalyzer': 646 case 'dartanalyzer':
650 case 'dart2analyzer': 647 case 'dart2analyzer':
651 validRuntimes = const ['none']; 648 validRuntimes = const ['none'];
652 break; 649 break;
653 case 'none': 650 case 'none':
654 case 'dart2dart':
655 validRuntimes = const ['vm', 'drt', 'dartium', 651 validRuntimes = const ['vm', 'drt', 'dartium',
656 'ContentShellOnAndroid', 'DartiumOnAndroid']; 652 'ContentShellOnAndroid', 'DartiumOnAndroid'];
657 break; 653 break;
658 } 654 }
659 if (!validRuntimes.contains(config['runtime'])) { 655 if (!validRuntimes.contains(config['runtime'])) {
660 isValid = false; 656 isValid = false;
661 print("Warning: combination of compiler '${config['compiler']}' and " 657 print("Warning: combination of compiler '${config['compiler']}' and "
662 "runtime '${config['runtime']}' is invalid. " 658 "runtime '${config['runtime']}' is invalid. "
663 "Skipping this combination."); 659 "Skipping this combination.");
664 } 660 }
(...skipping 232 matching lines...) Expand 10 before | Expand all | Expand 10 after
897 return option; 893 return option;
898 } 894 }
899 } 895 }
900 print('Unknown test option $name'); 896 print('Unknown test option $name');
901 exit(1); 897 exit(1);
902 } 898 }
903 899
904 900
905 List<_TestOptionSpecification> _options; 901 List<_TestOptionSpecification> _options;
906 } 902 }
OLDNEW
« no previous file with comments | « tools/testing/dart/status_reporter.dart ('k') | tools/testing/dart/test_runner.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698