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

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

Issue 11817012: Migration of testing scripts in tools/ to libv2 (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 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
Index: tools/testing/dart/multitest.dart
diff --git a/tools/testing/dart/multitest.dart b/tools/testing/dart/multitest.dart
index ca621750e97b19f3f65414e1ca37132f1c129e8b..e2b6aeec217560b1250de3ef22c9a742ee19247f 100644
--- a/tools/testing/dart/multitest.dart
+++ b/tools/testing/dart/multitest.dart
@@ -4,6 +4,7 @@
library multitest;
+import "dart:async";
import "dart:io";
import "test_suite.dart";
@@ -142,11 +143,12 @@ class _Annotation {
return null;
}
var annotation = new _Annotation();
- var parts = line.split('///')[1].split(':').map((s) => s.trim());
+ var parts = line.split('///')[1].split(':')
+ .mappedBy((s) => s.trim()).toList();
annotation.key = parts[0];
annotation.rest = parts[1];
annotation.outcomesList = annotation.rest.split(',')
- .map((s) => s.trim());
+ .mappedBy((s) => s.trim()).toList();
return annotation;
}
}
@@ -214,7 +216,7 @@ Future doMultitest(Path filePath, String outputDir, Path suiteDir,
}
// Wait until all imports are copied before scheduling test cases.
- return Futures.wait(futureCopies).transform((_) {
+ return Futures.wait(futureCopies).then((_) {
String baseFilename = filePath.filenameWithoutExtension;
for (String key in tests.keys) {
final Path multitestFilename =

Powered by Google App Engine
This is Rietveld 408576698