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

Side by Side 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library unittestTest; 5 library unittest.async_exception_test;
6 6
7 import 'dart:async'; 7 import 'dart:async';
8 import 'dart:isolate';
9 8
10 import 'package:unittest/unittest.dart'; 9 import 'package:unittest/unittest.dart';
11 10
12 part 'utils.dart'; 11 import 'package:metatest/metatest.dart';
13 12
14 var testName = 'async exception test'; 13 void main() => initTests(_test);
15 14
16 var testFunction = (_) { 15 void _test(message) {
17 test(testName, () { 16 initMetatest(message);
18 expectAsync(() {});
19 _defer(() { throw "error!"; });
20 });
21 };
22 17
23 final expected = buildStatusString(0, 1, 0, testName, message: 'Caught error!'); 18 expectTestResults('async errors cause tests to fail', () {
19 test('test', () {
20 expectAsync(() {});
21 new Future.sync(() {
22 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.
23 });
24 });
25 }, [{
26 'description': 'test',
27 'message': 'Caught an error!',
28 }]);
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.
29 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698