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

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

Issue 2855073002: Small-scale clean ups in test.dart, mainly around test_progress. (Closed)
Patch Set: Created 3 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/test_configurations.dart
diff --git a/tools/testing/dart/test_configurations.dart b/tools/testing/dart/test_configurations.dart
index 5f68c2f8e984febb94657d9bb767721149d8cedd..86169f4cbee4444094addc39aeaa1f98537ef643 100644
--- a/tools/testing/dart/test_configurations.dart
+++ b/tools/testing/dart/test_configurations.dart
@@ -84,8 +84,8 @@ Future testConfigurations(List<Map> configurations) async {
if (!firstConf['append_logs']) {
var files = [
- new File(TestUtils.flakyFileName()),
- new File(TestUtils.testOutcomeFileName())
+ new File(TestUtils.flakyFileName),
+ new File(TestUtils.testOutcomeFileName)
];
for (var file in files) {
if (file.existsSync()) {
@@ -95,7 +95,7 @@ Future testConfigurations(List<Map> configurations) async {
}
DebugLogger.init(
- firstConf['write_debug_log'] ? TestUtils.debugLogfile() : null,
+ firstConf['write_debug_log'] ? TestUtils.debugLogFilePath : null,
append: firstConf['append_logs']);
// Print the configurations being run by this execution of
@@ -151,7 +151,7 @@ Future testConfigurations(List<Map> configurations) async {
conf['_servers_'] = servers;
if (verbose) {
serverFutures.last.then((_) {
- var commandline = servers.httpServerCommandline();
+ var commandline = servers.httpServerCommandLine();
print('Started HttpServers: $commandline');
});
}
@@ -224,19 +224,19 @@ Future testConfigurations(List<Map> configurations) async {
TestUtils.deleteTempSnapshotDirectory(configurations[0]);
}
- var eventListener = [];
+ var eventListener = <EventListener>[];
// We don't print progress if we list tests.
if (progressIndicator != 'silent' && !listTests) {
var printFailures = true;
- var formatter = new Formatter();
+ var formatter = Formatter.normal;
if (progressIndicator == 'color') {
progressIndicator = 'compact';
- formatter = new ColorFormatter();
+ formatter = Formatter.color;
}
if (progressIndicator == 'diff') {
progressIndicator = 'compact';
- formatter = new ColorFormatter();
+ formatter = Formatter.color;
printFailures = false;
eventListener.add(new StatusFileUpdatePrinter());
}
@@ -249,7 +249,7 @@ Future testConfigurations(List<Map> configurations) async {
eventListener.add(new TestFailurePrinter(printFailureSummary, formatter));
}
eventListener.add(
- progressIndicatorFromName(progressIndicator, startTime, formatter));
+ ProgressIndicator.fromName(progressIndicator, startTime, formatter));
if (printTiming) {
eventListener.add(new TimingPrinter(startTime));
}

Powered by Google App Engine
This is Rietveld 408576698