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

Unified Diff: lib/src/simple_configuration.dart

Issue 934413002: Replace the existing unittest APIs with the new runner infrastructure. (Closed) Base URL: git@github.com:dart-lang/unittest@master
Patch Set: Code review changes 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
« no previous file with comments | « lib/src/prints_matcher.dart ('k') | lib/src/test_case.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/src/simple_configuration.dart
diff --git a/lib/src/simple_configuration.dart b/lib/src/simple_configuration.dart
index c95550ce22f7d8aea7b4b72d8b74573ff8ef71d7..470b10f97ed78754c3099b0a76b891a1991d7044 100644
--- a/lib/src/simple_configuration.dart
+++ b/lib/src/simple_configuration.dart
@@ -4,119 +4,16 @@
library unittest.simple_configuration;
-import 'dart:isolate';
-
-import '../unittest.dart';
+import 'test_case.dart';
import 'configuration.dart';
-import 'utils.dart';
-/// A configuration that provides hooks to configure the unittest library for
-/// different platforms.
+/// This is a stub class used to preserve compatibility with unittest 0.11.*.
///
-/// This class implements the [Configuration] API in a platform-independent way.
-/// Tests that want to take advantage of the platform can create a subclass and
-/// override methods from this class.
+/// It will be removed before the next version is released.
+@deprecated
class SimpleConfiguration extends Configuration {
- /// A port that keeps the VM alive while we wait for asynchronous tests to
- /// finish.
- ///
- /// The VM won't shut down as long as there's an open receive port.
- ReceivePort _receivePort;
-
- /// If true (the default), throw an exception at the end if any tests failed.
bool throwOnTestFailures = true;
-
- /// The constructor sets up a failure handler for [expect] that redirects
- /// [expect] failures to [onExpectFailure].
SimpleConfiguration() : super.blank();
- void onInit() {
- // For Dart internal tests, we don't want stack frame filtering.
- // We turn it off here in the default config, but by default turn
- // it back on in the vm and html configs.
- filterStacks = false;
- _receivePort = new ReceivePort();
- _postMessage('unittest-suite-wait-for-done');
- }
-
- /// Called when each test starts. Useful to show intermediate progress on
- /// a test suite. Derived classes should call this first before their own
- /// override code.
- void onTestStart(TestCase testCase) {}
-
- /// Handles the logging of messages by a test case.
- ///
- /// The default in this base configuration is to call [print].
- void onLogMessage(TestCase testCase, String message) {
- print(message);
- }
-
- /// Format a test result.
- String formatResult(TestCase testCase) {
- var result = new StringBuffer();
- result.write(testCase.result.toUpperCase());
- result.write(": ");
- result.write(testCase.description);
- result.write("\n");
-
- if (testCase.message != '') {
- result.write(indent(testCase.message));
- result.write("\n");
- }
-
- if (testCase.stackTrace != null) {
- result.write(indent(testCase.stackTrace.toString()));
- result.write("\n");
- }
- return result.toString();
- }
-
- /// Called with the result of all test cases.
- ///
- /// The default implementation prints the result summary using [print],
- /// formatted with [formatResult]. Browser tests commonly override this to
- /// reformat the output.
- ///
- /// When [uncaughtError] is not null, it contains an error that occured
- /// outside of tests (e.g. setting up the test).
- void onSummary(int passed, int failed, int errors, List<TestCase> results,
- String uncaughtError) {
- // Print each test's result.
- for (var test in results) {
- print(formatResult(test).trim());
- }
-
- // Show the summary.
- print('');
-
- if (passed == 0 && failed == 0 && errors == 0 && uncaughtError == null) {
- print('No tests found.');
- // This is considered a failure too.
- } else if (failed == 0 && errors == 0 && uncaughtError == null) {
- print('All $passed tests passed.');
- } else {
- if (uncaughtError != null) {
- print('Top-level uncaught error: $uncaughtError');
- }
- print('$passed PASSED, $failed FAILED, $errors ERRORS');
- }
- }
-
- void onDone(bool success) {
- if (success) {
- _postMessage('unittest-suite-success');
- _receivePort.close();
- } else {
- _receivePort.close();
- if (throwOnTestFailures) {
- throw new Exception('Some tests failed.');
- }
- }
- }
-
- void _postMessage(String message) {
- // In dart2js browser tests, the JavaScript-based test controller
- // intercepts calls to print and listens for "secret" messages.
- print(message);
- }
+ String formatResult(TestCase testCase) => "";
}
« no previous file with comments | « lib/src/prints_matcher.dart ('k') | lib/src/test_case.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698