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

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

Issue 8833017: Add suite name to display name. This makes it easier to find failing tests. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 9 years 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 | « no previous file | 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
diff --git a/tools/testing/dart/test_suite.dart b/tools/testing/dart/test_suite.dart
index 38c4852c594d3570434a75315f00d0e8de269509..c8d08ebacb82342219d509b42973bd750a49ec79 100644
--- a/tools/testing/dart/test_suite.dart
+++ b/tools/testing/dart/test_suite.dart
@@ -88,7 +88,7 @@ class CCTestSuite implements TestSuite {
var args = [testName];
args.addAll(TestUtils.standardOptions(configuration));
- doTest(new TestCase(testName,
+ doTest(new TestCase('$suiteName/$testName',
runnerPath,
args,
configuration,
@@ -217,7 +217,7 @@ class StandardTestSuite implements TestSuite {
optionsFromFile,
enableFatalTypeErrors);
for (var args in argumentLists) {
- doTest(new TestCase(testName,
+ doTest(new TestCase('$suiteName/$testName',
shellPath(),
args,
configuration,
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698