]> source.dussan.org Git - jquery-ui.git/commitdiff
Tests: Increase robustness of a draggable `revert` test
authorMichał Gołębiowski-Owczarek <m.goleb@gmail.com>
Fri, 12 Apr 2024 13:23:05 +0000 (15:23 +0200)
committerGitHub <noreply@github.com>
Fri, 12 Apr 2024 13:23:05 +0000 (15:23 +0200)
One of the changes is increasing `revertDuration` from 200 to 300. For some
reason, when jQuery 3.1 is used in an iframe (which TestSwarm uses) the timings
are lower than expected and `setTimeout` too slow and the check for animation
happens too late.

Closes gh-2230

tests/unit/draggable/options.js

index 499454a650066c999d5781abf063ba8e1574682b..40a36ba67fdd63379c56f6863e185df1e3d08f49 100644 (file)
@@ -1018,7 +1018,7 @@ QUnit.test( "opacity, default, switching after initialization", function( assert
 } );
 
 QUnit.test( "revert and revertDuration", function( assert ) {
-       var ready = assert.async();
+       var ready = assert.async( 2 );
        assert.expect( 7 );
 
        var element = $( "#draggable2" ).draggable( {
@@ -1035,7 +1035,7 @@ QUnit.test( "revert and revertDuration", function( assert ) {
 
        $( "#draggable2" ).draggable( "option", {
                revert: true,
-               revertDuration: 200,
+               revertDuration: 300,
                stop: function() {
                        ready();
                }
@@ -1045,6 +1045,7 @@ QUnit.test( "revert and revertDuration", function( assert ) {
        testHelper.move( element, 50, 50 );
        setTimeout( function() {
                assert.ok( $( "#draggable2" ).is( ":animated" ), "revert: true with revertDuration should animate" );
+               ready();
        } );
 } );