From bc5a43aa94e89ffdc8d9a7f8baf31233b7c04df5 Mon Sep 17 00:00:00 2001 From: Matthew Flaschen Date: Fri, 11 Jul 2014 18:40:55 -0400 Subject: [PATCH] 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 --- tests/qunit/data/testrunner.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) 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(). -- 2.20.1