QUnit testrunner: Fix comment regarding animation and AJAX check
authorMatthew Flaschen <mflaschen@wikimedia.org>
Fri, 11 Jul 2014 22:40:55 +0000 (18:40 -0400)
committerKrinkle <krinklemail@gmail.com>
Fri, 11 Jul 2014 22:51:11 +0000 (22:51 +0000)
Follows-up b90c69cdf7.

This takes the approach of throwing an error to make the exact issue
obvious.

Unlike other versions (including the code in jQuery it is loosely
based on), this does not attempt to cleanup by stopping the effects
queue.

Change-Id: Ibb6ca0d083d4e8f1556e6f3493da17f7a5f44024

tests/qunit/data/testrunner.js

index f3f884a..2eda8f1 100644 (file)
                                        // still suppressed by the end of the test.
                                        restoreWarnings();
 
-                                       // Check for (and clean up, if possible) incomplete animations/requests/etc.
+                                       // Check for incomplete animations/requests/etc and throw
+                                       // error if there are any.
                                        if ( $.timers && $.timers.length !== 0 ) {
                                                // Test may need to use fake timers, wait for animations or
                                                // call $.fx.stop().