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

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

Issue 2451623006: Reland "Merge more Kernel infrastructure from kernel_sdk SDK fork." (Closed)
Patch Set: Fix Created 4 years, 2 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/test_configurations.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 255564ac5c1688a84428a97b19f7fa2333f0428a..bee1a2c5fc814f797a04c723ab0abff380334119 100644
--- a/tools/testing/dart/test_options.dart
+++ b/tools/testing/dart/test_options.dart
@@ -26,6 +26,7 @@ const List<String> defaultTestSelectors = const [
'pkg',
'analyze_library',
'service',
+ 'kernel',
'observatory_ui'
];
@@ -75,9 +76,15 @@ class TestOptionsParser {
dart2app:
dart2appjit: Compile the Dart code into an app snapshot before running test
- (only valid with dart_app runtime)''',
+ (only valid with dart_app runtime)
+
+ dartk: Compile the Dart source into Kernel before running test.
+
+ dartkp: Compiler the Dart source into Kernel and then Kernel into AOT
+ snapshot before running the test.''',
['-c', '--compiler'],
- ['none', 'precompiler', 'dart2js', 'dart2analyzer', 'dart2app', 'dart2appjit'],
+ ['none', 'precompiler', 'dart2js', 'dart2analyzer', 'dart2app',
+ 'dart2appjit', 'dartk', 'dartkp'],
'none'),
// TODO(antonm): fix the option drt.
new _TestOptionSpecification(
@@ -677,6 +684,12 @@ Note: currently only implemented for dart2js.''',
case 'precompiler':
validRuntimes = const ['dart_precompiled'];
break;
+ case 'dartk':
+ validRuntimes = const ['vm'];
+ break;
+ case 'dartkp':
+ validRuntimes = const ['dart_precompiled'];
+ break;
case 'none':
validRuntimes = const [
'vm',
@@ -838,9 +851,9 @@ Note: currently only implemented for dart2js.''',
if (configuration['package_root'] == null &&
configuration['packages'] == null) {
configuration['packages'] =
- TestUtils.dartDirUri.resolve('.packages').toFilePath();
+ TestUtils.dartDirUri.resolve('.packages').toFilePath();
}
-
+
// Expand the architectures.
if (configuration['arch'].contains(',')) {
return _expandHelper('arch', configuration);
« no previous file with comments | « tools/testing/dart/test_configurations.dart ('k') | tools/testing/dart/test_runner.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698