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

Unified Diff: lib/src/engine.dart

Issue 933083002: Add a test runner executable. (Closed) Base URL: git@github.com:dart-lang/unittest@master
Patch Set: fix loader_test? Created 5 years, 10 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/src/engine.dart
diff --git a/lib/src/engine.dart b/lib/src/engine.dart
index 7bd6958899129492a64c8717905e5ee0d12cde73..5e2b6508c3c450051fa12eb1a5e3fc27f20cb871 100644
--- a/lib/src/engine.dart
+++ b/lib/src/engine.dart
@@ -64,4 +64,9 @@ class Engine {
}).then((_) =>
liveTests.every((liveTest) => liveTest.state.result == Result.success));
}
+
+ /// Signals that the caller is done paying attention to test results and the
+ /// engine should release any resources it has allocated.
+ Future close() =>
+ Future.wait(liveTests.map((liveTest) => liveTest.close()));
kevmoo 2015/02/19 01:54:08 Could this be huge? Do we want to "pool" this?
nweiz 2015/02/19 02:10:18 It's important for the API that the caller have ac
}

Powered by Google App Engine
This is Rietveld 408576698