Effects: Adjust animation duration in tests

With jQuery 3 using `requestAnimationFrame()`, the `setTimeout()` timing
for short animations wasn't working consistently. This resulted in infrequent
failures everywhere (but infrequent enough that it's hard to even notice), but
consistent failures in IE and Edge. Bumping up the duration and running the
assertions in the middle seems to give consistent results.

Eventually, we should refactor this to use `requestAnimationFrame()` in the
tests themselves to avoid problems like this.

Closes gh-1738

diff --git a/tests/unit/effects/core.js b/tests/unit/effects/core.js
index 0812e118404e30d3a9369f5dfdb79e4afba633a8..9863cd8ca81a38053e757b76d9b4316d7bf9a0cd 100644 (file)
--- a/tests/unit/effects/core.js
+++ b/tests/unit/effects/core.js
@@ -29,7 +29,7 @@ QUnit.assert.notPresent = function( value, array, message ) {
 };
 
 // MinDuration is used for "short" animate tests where we are only concerned about the final
-var minDuration = 15,
+var minDuration = 60,
 
        // Duration is used for "long" animates where we plan on testing properties during animation
        duration = 200;
@@ -332,7 +332,7 @@ $.each( $.effects.effect, function( effect ) {
 
                function duringTest( fn ) {
                        return function( next ) {
-                               setTimeout( fn );
+                               setTimeout( fn, minDuration / 2 );
                                next();
                        };
                }