Skip to content
Permalink
Browse files

Make test async to please Opera 11.1; supplements #11500

  • Loading branch information...
rwaldron authored and dmethvin committed Apr 16, 2012
1 parent 46d0a5c commit 8fadc5ba01bc517ebb07736d5af9a965ed133a39
Showing with 2 additions and 1 deletion.
  1. +2 −1 test/unit/event.js
@@ -2867,7 +2867,7 @@ test("fixHooks extensions", function() {

})();

test("trigger click on checkbox, fires change event", function() {
asyncTest("trigger click on checkbox, fires change event", function() {
expect(1);

var check = jQuery("#check2");
@@ -2876,5 +2876,6 @@ test("trigger click on checkbox, fires change event", function() {
// get it?
check.off("change");
ok( true, "Change event fired as a result of triggered click" );
start();
}).trigger("click");
});

2 comments on commit 8fadc5b

@Krinkle

This comment has been minimized.

Copy link
Member

replied Apr 20, 2012

Still problematic, QUnit somehow is failing to recover if this fails. Isn't the implicit stop() by asyncTest supposed to recover after a certain timeout? It's not doing that. I tried it just now with a plain test suite with just stop(), it never starts again...

Edit: Traced to 61137fd in QUnit

@Krinkle

This comment has been minimized.

Copy link
Member

replied Apr 20, 2012

PR #746

Please sign in to comment.
You can’t perform that action at this time.