Fix test race condition with DOMContentLoaded

Test tap could somtimes fire before the event handlers were added
googlearchive · Sep 11, 2013 · c91f99c · c91f99c
1 parent 8a859ac
commit c91f99c
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/tests/tap.js b/tests/tap.js
@@ -1,7 +1,15 @@
 suite('Tap', function() {
   test('Up and Down', function(done) {
     prepare(target, 'tap', done);
-    target.dispatchEvent(new PointerEvent('pointerdown', {isPrimary: true, buttons: 1, pointerId: 1, bubbles: true}));
-    target.dispatchEvent(new PointerEvent('pointerup', {isPrimary: true, buttons: 1, pointerId: 1, bubbles: true}));
+    var fireTap = function() {
+      target.dispatchEvent(new PointerEvent('pointerdown', {isPrimary: true, buttons: 1, pointerId: 1, bubbles: true}));
+      target.dispatchEvent(new PointerEvent('pointerup', {isPrimary: true, buttons: 1, pointerId: 1, bubbles: true}));
+    }
+    // handle case where DOMContentLoaded has already fired
+    if (document.readyState === 'complete') {
+      fireTap();
+    } else {
+      document.addEventListener('DOMContentLoaded', fireTap);
+    }
   });
 });