aChange all asyncTest/callbackDone style tests to use the new expectAsync/

tests/html/indexeddb_2_test.dart

Index: tests/html/indexeddb_2_test.dart
===================================================================
--- tests/html/indexeddb_2_test.dart	(revision 7823)
+++ tests/html/indexeddb_2_test.dart	(working copy)
@@ -18,8 +18,9 @@
   var db;
 
   fail(e) {
-    callbackDone();
-    Expect.fail('IndexedDB failure');
+    guardAsync(() {
+      Expect.fail('IndexedDB failure');
+    });
   }
 
   createObjectStore() {
@@ -27,21 +28,22 @@
     Expect.isNotNull(store);
   }
 
-  step2() {
+  step2(e) {
     var transaction = db.transaction(storeName, IDBTransaction.READ_ONLY);
     var request = transaction.objectStore(storeName).getObject(key);
-    request.on.success.add((e) {
+    request.on.success.add(
+      expectAsync1((e) {

[Siggi Cherem (dart-lang), 2012/05/21 22:20:47]

style-nit: I'd keep the expectAsync1((e) { in the previous line.

[gram, 2012/05/22 17:36:36]

Done.

         var object = e.target.result;
         check(value, object);
-        callbackDone();
-      });
+      })
+    );
     request.on.error.add(fail);
   }
 
   step1() {
     var transaction = db.transaction([storeName], IDBTransaction.READ_WRITE);
     var request = transaction.objectStore(storeName).put(value, key);
-    request.on.success.add((e) { step2(); });
+    request.on.success.add(expectAsync1(step2));
     request.on.error.add(fail);
   }
 
@@ -52,7 +54,12 @@
       // open call and listening to onversionchange.  Can we feature-detect the
       // difference and make it work?
       var request = db.setVersion(version);
-      request.on.success.add((e) { createObjectStore(); step1(); });
+      request.on.success.add(
+        expectAsync1((e) {
+          createObjectStore();
+          step1();
+        })
+      );
       request.on.error.add(fail);
     } else {
       step1();
@@ -61,7 +68,7 @@
 
   var request = window.webkitIndexedDB.open(dbName);
   Expect.isNotNull(request);
-  request.on.success.add(initDb);
+  request.on.success.add(expectAsync1(initDb));
   request.on.error.add(fail);
 };
 
@@ -78,10 +85,10 @@
   obj4['a'] = 100;
   obj4['b'] = 's';
 
-  asyncTest('test_simple', 1, testReadWrite(123, obj1, verifyGraph));
-  asyncTest('test_DAG', 1, testReadWrite(123, obj2, verifyGraph));
-  asyncTest('test_cycle', 1, testReadWrite(123, obj3, verifyGraph));
-  asyncTest('test_simple_splay', 1, testReadWrite(123, obj4, verifyGraph));
+  test('test_simple', testReadWrite(123, obj1, verifyGraph));
+  test('test_DAG', testReadWrite(123, obj2, verifyGraph));
+  test('test_cycle', testReadWrite(123, obj3, verifyGraph));
+  test('test_simple_splay', testReadWrite(123, obj4, verifyGraph));
 }
 
 main() {