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/multitest.dart

Issue 10247004: test rename overhaul: step 6 - isolate tests (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 8 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 6c2f84281e291ca1872645739a1104cda11ba8d1..b287794c2d478ba7b3e8dc8a19cf124bd1ac03eb 100644
--- a/tools/testing/dart/multitest.dart
+++ b/tools/testing/dart/multitest.dart
@@ -263,8 +263,11 @@ String CreateMultitestDirectory(String outputDir, String testDir) {
generatedTestDir.createSync();
}
var split = testDir.split('/');
- var lastComponent = split.removeLast();
- Expect.isTrue(lastComponent == 'src');
+ if (split.last() == 'src') {
+ // TODO(sigmund): remove this once all tests are migrated to use
+ // TestSuite.forDirectory.
+ split.removeLast();
+ }
String path = '${generatedTestDir.path}/${split.last()}';
Directory dir = new Directory(path);
if (!dir.existsSync()) {

Powered by Google App Engine
This is Rietveld 408576698