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

Unified Diff: tests/html/documentfragment_test.dart

Issue 10412015: aChange all asyncTest/callbackDone style tests to use the new expectAsync/ (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 8 years, 7 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: tests/html/documentfragment_test.dart
===================================================================
--- tests/html/documentfragment_test.dart (revision 7823)
+++ tests/html/documentfragment_test.dart (working copy)
@@ -290,16 +290,15 @@
fragment.webkitRequestFullScreen(2);
});
- asyncTest('default values', 1, () {
+ test('default values', () {
var fragment = new DocumentFragment();
- fragment.rect.then((ElementRect rect) {
+ fragment.rect.then(expectAsync1((ElementRect rect) {
expectEmptyRect(rect.client);
expectEmptyRect(rect.offset);
expectEmptyRect(rect.scroll);
expectEmptyRect(rect.bounding);
Expect.isTrue(rect.clientRects.isEmpty());
- callbackDone();
- });
+ }));
Expect.equals("false", fragment.contentEditable);
Expect.equals(-1, fragment.tabIndex);
Expect.equals("", fragment.id);
@@ -323,14 +322,13 @@
Expect.isFalse(fragment.matchesSelector("*"));
});
- asyncTest('style', 1, () {
+ test('style', () {
var fragment = new DocumentFragment();
var style = fragment.style;
expectEmptyStyleDeclaration(style);
- fragment.computedStyle.then((computedStyle) {
+ fragment.computedStyle.then(expectAsync1((computedStyle) {
expectEmptyStyleDeclaration(computedStyle);
- callbackDone();
- });
+ }));
});
// TODO(nweiz): re-enable when const is better supported in dartc and/or frog

Powered by Google App Engine
This is Rietveld 408576698