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

Unified Diff: lib/src/runner.dart

Issue 1461293005: Add a JSON reporter. (Closed) Base URL: git@github.com:dart-lang/test@master
Patch Set: Created 5 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
Index: lib/src/runner.dart
diff --git a/lib/src/runner.dart b/lib/src/runner.dart
index e5831e58a8d29d6d44921f6ec68b54ea8f9253df..9b634ef261c801303fef9f1e32cb21aa23103318 100644
--- a/lib/src/runner.dart
+++ b/lib/src/runner.dart
@@ -19,6 +19,7 @@ import 'runner/loader.dart';
import 'runner/reporter.dart';
import 'runner/reporter/compact.dart';
import 'runner/reporter/expanded.dart';
+import 'runner/reporter/json.dart';
import 'runner/runner_suite.dart';
import 'util/io.dart';
import 'utils.dart';
@@ -53,17 +54,28 @@ class Runner {
var loader = new Loader(config);
var engine = new Engine(concurrency: config.concurrency);
- var watch = config.reporter == "compact"
- ? CompactReporter.watch
- : ExpandedReporter.watch;
-
- var reporter = watch(
- engine,
- color: config.color,
- verboseTrace: config.verboseTrace,
- printPath: config.paths.length > 1 ||
- new Directory(config.paths.single).existsSync(),
- printPlatform: config.platforms.length > 1);
+ var reporter;
+ switch (config.reporter) {
+ case "compact":
+ case "expanded":
+ var watch = config.reporter == "compact"
+ ? CompactReporter.watch
+ : ExpandedReporter.watch;
+
+ reporter = watch(
+ engine,
+ color: config.color,
+ verboseTrace: config.verboseTrace,
+ printPath: config.paths.length > 1 ||
+ new Directory(config.paths.single).existsSync(),
+ printPlatform: config.platforms.length > 1);
+ break;
+
+ case "json":
+ reporter = JsonReporter.watch(engine,
+ verboseTrace: config.verboseTrace);
+ break;
+ }
return new Runner._(config, loader, engine, reporter);
}

Powered by Google App Engine
This is Rietveld 408576698