From: Matthew Flaschen <mflaschen@wikimedia.org>
Date: Fri, 11 Jul 2014 22:40:55 +0000 (-0400)
Subject: QUnit testrunner: Fix comment regarding animation and AJAX check
X-Git-Tag: 1.31.0-rc.0~14982^2
X-Git-Url: http://git.cyclocoop.org/%22.%24h.%22?a=commitdiff_plain;h=bc5a43aa94e89ffdc8d9a7f8baf31233b7c04df5;p=lhc%2Fweb%2Fwiklou.git

QUnit testrunner: Fix comment regarding animation and AJAX check

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
---

diff --git a/tests/qunit/data/testrunner.js b/tests/qunit/data/testrunner.js
index f3f884a49a..2eda8f1504 100644
--- a/tests/qunit/data/testrunner.js
+++ b/tests/qunit/data/testrunner.js
@@ -241,7 +241,8 @@
 					// 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().