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

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

Issue 10627008: Added the ability to filter which tests are run by name, using a --filter (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 8 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 side-by-side diff with in-line comments
Download patch
« lib/unittest/unittest.dart ('K') | « tools/testing/dart/test_options.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/testing/dart/test_suite.dart
===================================================================
--- tools/testing/dart/test_suite.dart (revision 8995)
+++ tools/testing/dart/test_suite.dart (working copy)
@@ -446,6 +446,9 @@
String testPath =
new File(info.filename).fullPathSync().replaceAll('\\', '/');
String tempDir = createOutputDirectory(testPath, '');
+ if (configuration['filter'] != '') {
+ args.add('--filter="${configuration["filter"]}"');
+ }
args.add('--out=$tempDir/out.js');
List<Command> commands = <Command>[new Command(shellPath(), args)];
if (configuration['runtime'] == 'd8') {
@@ -454,6 +457,9 @@
}
return commands;
} else {
+ if (configuration['filter'] != '') {
+ args.add('--filter="${configuration["filter"]}"');
+ }
return <Command>[new Command(shellPath(), args)];
}
}
@@ -563,7 +569,8 @@
File file = new File(dartWrapperFilename);
RandomAccessFile dartWrapper = file.openSync(FileMode.WRITE);
dartWrapper.writeStringSync(
- DartTestWrapper(dartDir, dartLibraryFilename));
+ DartTestWrapper(dartDir, dartLibraryFilename,
+ configuration['filter']));
dartWrapper.closeSync();
} else {
dartWrapperFilename = testPath;
« lib/unittest/unittest.dart ('K') | « tools/testing/dart/test_options.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698