From 77412e03056b2299309638212ae708acc190664c Mon Sep 17 00:00:00 2001 From: Dave Methvin Date: Thu, 9 Aug 2012 21:30:51 -0400 Subject: [PATCH] Wait for both .stop() tests to finish, don't count on timing. --- test/unit/effects.js | 20 +++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/test/unit/effects.js b/test/unit/effects.js index 4353e1566..0eb2a3156 100644 --- a/test/unit/effects.js +++ b/test/unit/effects.js @@ -552,27 +552,30 @@ test("animate non-element", function() { }); test("stop()", function() { - expect(4); + expect( 4 ); stop(); - var $foo = jQuery("#foo"); - var w = 0; + var $foo = jQuery("#foo"), + tests = 2, + w = 0; $foo.hide().css( "width", 200 ) - .animate( { "width": "show" }, 1000 ); + .animate( { "width": "show" }, 1500 ); setTimeout(function() { var nw = $foo.css("width"); - notEqual( parseFloat( nw ), w, "An animation occurred " + nw + " " + w + "px"); + notEqual( parseFloat( nw ), w, "An animation occurred " + nw + " " + w + "px" ); $foo.stop(); nw = $foo.css("width"); - notEqual( parseFloat( nw ), w, "Stop didn't reset the animation " + nw + " " + w + "px"); + notEqual( parseFloat( nw ), w, "Stop didn't reset the animation " + nw + " " + w + "px" ); setTimeout(function() { $foo.removeData(); $foo.removeData(undefined, true); equal( nw, $foo.css("width"), "The animation didn't continue" ); - start(); + if ( --tests === 0 ) { + start(); + } }, 100); }, 100); @@ -586,6 +589,9 @@ test("stop()", function() { equal( $two.css("opacity"), "0", "Stop does not interfere with animations on other elements (#6641)" ); // Reset styles $one.add( $two ).css("opacity", ""); + if ( --tests === 0 ) { + start(); + } }); }, 50); });