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

Unified Diff: lib/test.dart

Issue 1187103004: Allow Suites to be added to an Engine over time. (Closed) Base URL: git@github.com:dart-lang/test@master
Patch Set: Created 5 years, 6 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: lib/test.dart
diff --git a/lib/test.dart b/lib/test.dart
index eeacd864c9eec391c6878091448fe4bb37bf46a6..7a5589a5646bf352900404c403c4023ff5b9e432 100644
--- a/lib/test.dart
+++ b/lib/test.dart
@@ -12,6 +12,7 @@ import 'src/backend/declarer.dart';
import 'src/backend/suite.dart';
import 'src/backend/test_platform.dart';
import 'src/frontend/timeout.dart';
+import 'src/runner/engine.dart';
import 'src/runner/reporter/expanded.dart';
import 'src/utils.dart';
@@ -55,10 +56,15 @@ Declarer get _declarer {
platform: "VM")
.forPlatform(TestPlatform.vm, os: currentOSGuess);
- var success = await new ExpandedReporter([suite], color: true).run();
+ var engine = new Engine();
+ engine.suiteSink.add(suite);
+ engine.suiteSink.close();
+ ExpandedReporter.watch(engine,
+ color: true, printPath: false, printPlatform: false);
+
+ var success = await engine.run();
// TODO(nweiz): Set the exit code on the VM when issue 6943 is fixed.
- // TODO(nweiz): Just "return;" when issue 23200 is fixed.
- if (success) return null;
+ if (success) return;
print('');
new Future.error("Dummy exception to set exit code.");
});

Powered by Google App Engine
This is Rietveld 408576698