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

Unified Diff: tools/testing/dart/test_options.dart

Issue 1659163007: Rename fletch -> dartino (Closed) Base URL: https://github.com/dartino/sdk.git@master
Patch Set: address comments Created 4 years, 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/testing/dart/runtime_configuration.dart ('k') | tools/testing/dart/test_runner.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/testing/dart/test_options.dart
diff --git a/tools/testing/dart/test_options.dart b/tools/testing/dart/test_options.dart
index ef7a8a6146f70e6ccb3165eac717e46c5f574885..4796cd0e06f545a0192341940d39d8d3fc937df3 100644
--- a/tools/testing/dart/test_options.dart
+++ b/tools/testing/dart/test_options.dart
@@ -22,7 +22,7 @@ const List<String> defaultTestSelectors = const [
'ffi',
'samples',
'warnings',
- 'fletch_tests',
+ 'dartino_tests',
'cc_tests',
'lib',
'os',
@@ -85,7 +85,7 @@ class TestOptionsParser {
dart2analyzer: Perform static analysis on Dart code by running the analyzer on Dart.
(only valid with the following runtimes: none)''',
['-c', '--compiler'],
- ['none', 'fletchc'],
+ ['none', 'dartino_compiler'],
'none'),
// TODO(antonm): fix the option drt.
new _TestOptionSpecification(
@@ -113,10 +113,10 @@ class TestOptionsParser {
none: No runtime, compile only (for example, used for dartanalyzer static
analysis tests).''',
['-r', '--runtime'],
- ['none', 'fletchc', 'fletchvm', 'fletch_warnings',
- 'fletch_tests', 'fletch_cc_tests'],
- 'fletch_warnings,fletch_tests,fletchc,'
- 'fletch_cc_tests'),
+ ['none', 'dartino_compiler', 'dartinovm', 'dartino_warnings',
+ 'dartino_tests', 'dartino_cc_tests'],
+ 'dartino_warnings,dartino_tests,dartino_compiler,'
+ 'dartino_cc_tests'),
new _TestOptionSpecification(
'arch',
'The architecture to run tests for',
@@ -317,7 +317,7 @@ Note: currently only implemented for dart2js.''',
'Do not run tests in batch mode',
['-n', '--nobatch'],
[],
- true, // Batch-mode breaks fletch_warnings.
+ true, // Batch-mode breaks dartino_warnings.
type: 'bool'),
new _TestOptionSpecification(
'dart2js_batch',
@@ -496,10 +496,10 @@ Note: currently only implemented for dart2js.''',
type: 'int'),
new _TestOptionSpecification(
'settings_file_name',
- 'The fletch settings file to use for testing',
- ['--fletch-settings-file'],
+ 'The dartino settings file to use for testing',
+ ['--dartino-settings-file'],
[],
- '.fletch-settings'),
+ '.dartino-settings'),
new _TestOptionSpecification(
'no_java',
"Don't require running java tests",
@@ -686,12 +686,12 @@ Note: currently only implemented for dart2js.''',
bool isValid = true;
List<String> validRuntimes;
switch (config['compiler']) {
- case 'fletchc':
- validRuntimes = const ['none', 'fletchvm'];
+ case 'dartino_compiler':
+ validRuntimes = const ['none', 'dartinovm'];
break;
case 'none':
validRuntimes = const [
- 'fletchc', 'fletch_warnings', 'fletch_tests', 'fletch_cc_tests'];
+ 'dartino_compiler', 'dartino_warnings', 'dartino_tests', 'dartino_cc_tests'];
break;
}
if (!validRuntimes.contains(config['runtime'])) {
@@ -718,18 +718,18 @@ Note: currently only implemented for dart2js.''',
"--use-public-packages");
}
- if (config['runtime'] == 'fletchc' && !config['host_checked']) {
+ if (config['runtime'] == 'dartino_compiler' && !config['host_checked']) {
isValid = false;
// TODO(ahe): Find a way to make this optional.
- print("fletch requires --host-checked option.");
+ print("dartino requires --host-checked option.");
}
if (config['system'] == 'lk' &&
- (config['compiler'] != 'fletchc' ||
- config['runtime'] != 'fletchvm')) {
+ (config['compiler'] != 'dartino_compiler' ||
+ config['runtime'] != 'dartinovm')) {
isValid = false;
print("Running tests on LK works only in the "
- "--compiler=fletchc --runtime=fletchvm configuration.");
+ "--compiler=dartino_compiler --runtime=dartinovm configuration.");
}
return isValid;
« no previous file with comments | « tools/testing/dart/runtime_configuration.dart ('k') | tools/testing/dart/test_runner.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698