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

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

Issue 11348088: Changed: TestCase.output -> TestCase.commandOutputs[] (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 1 month 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_runner.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
diff --git a/tools/testing/dart/test_suite.dart b/tools/testing/dart/test_suite.dart
index 3d7c2da8afc46a9e2eedb500b1dced39f68543d7..84fd882b2df7fd3a2a5c5304f2ff6009ee9d9b3e 100644
--- a/tools/testing/dart/test_suite.dart
+++ b/tools/testing/dart/test_suite.dart
@@ -946,7 +946,7 @@ class StandardTestSuite extends TestSuite {
args.add('--out-expectation=${expectedOutput.toNativePath()}');
}
}
- List<String> commandSet = new List<String>.from(commands);
+ List<Command> commandSet = new List<Command>.from(commands);
if (subtestIndex != 0) {
commandSet = [];
if(TestUtils.usesWebDriver(runtime)) args.add('--force-refresh');
@@ -1505,7 +1505,8 @@ class TestUtils {
// If a flaky test did fail, infos about it (i.e. test name, stdin, stdout)
// will be written to this file. This is useful for the debugging of
// flaky tests.
- // When running on a built bot, the file can be made visible in the waterfall UI.
+ // When running on a built bot, the file can be made visible in the
+ // waterfall UI.
return ".flaky.log";
}
« no previous file with comments | « tools/testing/dart/test_runner.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698