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

Unified Diff: pkg/unittest/test/async_exception_test.dart

Issue 524153002: Sharing metatest logic between unittest and scheduled_test (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: CR feedback Created 6 years, 3 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: pkg/unittest/test/async_exception_test.dart
diff --git a/pkg/unittest/test/async_exception_test.dart b/pkg/unittest/test/async_exception_test.dart
index 85744165d797b30e1a00f576a4dfaaa4d3624727..30b1af3bc81305307f9016d38877b81287437251 100644
--- a/pkg/unittest/test/async_exception_test.dart
+++ b/pkg/unittest/test/async_exception_test.dart
@@ -2,22 +2,28 @@
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.
-library unittestTest;
+library unittest.async_exception_test;
import 'dart:async';
-import 'dart:isolate';
import 'package:unittest/unittest.dart';
-part 'utils.dart';
+import 'package:metatest/metatest.dart';
-var testName = 'async exception test';
+void main() => initTests(_test);
-var testFunction = (_) {
- test(testName, () {
- expectAsync(() {});
- _defer(() { throw "error!"; });
- });
-};
+void _test(message) {
+ initMetatest(message);
-final expected = buildStatusString(0, 1, 0, testName, message: 'Caught error!');
+ expectTestResults('async errors cause tests to fail', () {
+ test('test', () {
+ expectAsync(() {});
+ new Future.sync(() {
+ throw "an error!";
nweiz 2014/09/10 21:04:42 This doesn't actually run asynchronously; Future.s
kevmoo 2014/09/17 21:16:29 Done.
+ });
+ });
+ }, [{
+ 'description': 'test',
+ 'message': 'Caught an error!',
+ }]);
nweiz 2014/09/10 21:04:42 Un-indent these lines two spaces. Same goes for ot
kevmoo 2014/09/17 21:16:29 Done.
+}

Powered by Google App Engine
This is Rietveld 408576698