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

Side by Side Diff: tests/lib/unittest/unittest_test.dart

Issue 10830128: Ability to guard callbacks that may never get called. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 8 years, 4 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
« lib/unittest/unittest.dart ('K') | « lib/unittest/unittest.dart ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, 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 // TODO(gram): 5 // TODO(gram):
6 // Unfortunately I can't seem to test anything that involves timeouts, e.g. 6 // Unfortunately I can't seem to test anything that involves timeouts, e.g.
7 // insufficient callbacks, because the timeout is controlled externally 7 // insufficient callbacks, because the timeout is controlled externally
8 // (test.dart?), and we would need to use a shorter timeout for the inner tests 8 // (test.dart?), and we would need to use a shorter timeout for the inner tests
9 // so the outer timeout doesn't fire. So I removed all such tests. 9 // so the outer timeout doesn't fire. So I removed all such tests.
10 // I'd like to revisit this at some point. 10 // I'd like to revisit this at some point.
(...skipping 157 matching lines...) Expand 10 before | Expand all | Expand 10 after
168 buildStatusString(0, 1, 0, tests[2], message: 'Caught Exception: Fail.'), 168 buildStatusString(0, 1, 0, tests[2], message: 'Caught Exception: Fail.'),
169 buildStatusString(2, 0, 0, 'a a::a b b'), 169 buildStatusString(2, 0, 0, 'a a::a b b'),
170 buildStatusString(1, 0, 0, 'a ${tests[4]}', 0, 'setup'), 170 buildStatusString(1, 0, 0, 'a ${tests[4]}', 0, 'setup'),
171 buildStatusString(1, 0, 0, 'a ${tests[5]}', 0, '', 'teardown'), 171 buildStatusString(1, 0, 0, 'a ${tests[5]}', 0, '', 'teardown'),
172 buildStatusString(1, 0, 0, 'a ${tests[6]}', 0, 172 buildStatusString(1, 0, 0, 'a ${tests[6]}', 0,
173 'setup', 'teardown'), 173 'setup', 'teardown'),
174 buildStatusString(1, 0, 0, tests[7], 1), 174 buildStatusString(1, 0, 0, tests[7], 1),
175 buildStatusString(0, 0, 1, tests[8], 1, 175 buildStatusString(0, 0, 1, tests[8], 1,
176 message: 'Callback called more times than expected (2 > 1).'), 176 message: 'Callback called more times than expected (2 > 1).'),
177 buildStatusString(1, 0, 0, tests[9], 10), 177 buildStatusString(1, 0, 0, tests[9], 10),
178 buildStatusString(0, 0, 1, tests[10], message: 'Caught error!'), 178 buildStatusString(0, 1, 0, tests[10], message: 'Caught error!'),
179 ]; 179 ];
180 180
181 actual = []; 181 actual = [];
182 182
183 nextTest(0); 183 nextTest(0);
184 } 184 }
185 185
OLDNEW
« lib/unittest/unittest.dart ('K') | « lib/unittest/unittest.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698