]> source.dussan.org Git - jquery-ui.git/commitdiff
Tests: Reduce timeouts to optimize testsuite runtime
authorJörn Zaefferer <joern.zaefferer@gmail.com>
Tue, 17 Feb 2015 15:11:58 +0000 (16:11 +0100)
committerJörn Zaefferer <joern.zaefferer@gmail.com>
Wed, 4 Mar 2015 22:04:07 +0000 (23:04 +0100)
tests/unit/autocomplete/autocomplete_core.js
tests/unit/autocomplete/autocomplete_events.js
tests/unit/autocomplete/autocomplete_options.js
tests/unit/dialog/dialog_events.js
tests/unit/dialog/dialog_methods.js
tests/unit/menu/menu_core.js
tests/unit/menu/menu_events.js
tests/unit/menu/menu_methods.js
tests/unit/tabs/tabs_core.js
tests/unit/tabs/tabs_methods.js
tests/unit/tabs/tabs_options.js

index 669cefb8623b0c4a5470b8e608e26efe5d664279..c79e04dd9d627a4ac0bf168200c26005b45bf28b 100644 (file)
@@ -187,7 +187,7 @@ asyncTest( "past end of menu in multiline autocomplete", function() {
                element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
                equal( element.text(), customVal );
                start();
-       }, 50 );
+       });
 });
 
 asyncTest( "ESCAPE in multiline autocomplete", function() {
@@ -213,7 +213,7 @@ asyncTest( "ESCAPE in multiline autocomplete", function() {
                element.simulate( "keydown", { keyCode: $.ui.keyCode.ESCAPE } );
                equal( element.text(), customVal );
                start();
-       }, 50 );
+       });
 });
 
 asyncTest( "handle race condition", function() {
@@ -226,8 +226,8 @@ asyncTest( "handle race condition", function() {
                                equal( count, 1, "request with 1 character is first" );
                                setTimeout(function() {
                                        response([ "one" ]);
-                                       setTimeout( checkResults, 1 );
-                               }, 1 );
+                                       setTimeout( checkResults );
+                               });
                                return;
                        }
                        equal( count, 2, "request with 2 characters is second" );
index fb1cf73a696bcffb6b3feae0eee7d6ad7b131cce..a7c366800f37da315b5f24160237ea28e0106843 100644 (file)
@@ -71,8 +71,8 @@ $.each([
                        // blur must be async for IE to handle it properly
                        setTimeout(function() {
                                element.simulate( "blur" );
-                       }, 1 );
-               }, 50 );
+                       });
+               });
        });
 });
 
@@ -117,8 +117,8 @@ asyncTest( "cancel search", function() {
                        ok( menu.is( ":visible" ), "menu is visible after second search" );
                        equal( menu.find( ".ui-menu-item" ).length, 2, "# of menu items" );
                        start();
-               }, 50 );
-       }, 50 );
+               });
+       });
 });
 
 asyncTest( "cancel focus", function() {
@@ -137,7 +137,7 @@ asyncTest( "cancel focus", function() {
                element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
                equal( element.val(), customVal );
                start();
-       }, 50 );
+       });
 });
 
 asyncTest( "cancel select", function() {
@@ -157,7 +157,7 @@ asyncTest( "cancel select", function() {
                element.simulate( "keydown", { keyCode: $.ui.keyCode.ENTER } );
                equal( element.val(), customVal );
                start();
-       }, 50 );
+       });
 });
 
 asyncTest( "blur during remote search", function() {
@@ -170,7 +170,7 @@ asyncTest( "blur during remote search", function() {
                        setTimeout(function() {
                                response([ "result" ]);
                                start();
-                       }, 100 );
+                       }, 25);
                },
                open: function() {
                        ok( false, "opened after a blur" );
index 8215dbe61465527c0ba4178bc2dc6d1a6d4309d9..4b4957222b499699075742f916b5ae8aa2c34878 100644 (file)
@@ -102,7 +102,7 @@ asyncTest( "delay", function() {
        expect( 2 );
        var element = $( "#autocomplete" ).autocomplete({
                        source: data,
-                       delay: 50
+                       delay: 25
                }),
                menu = element.autocomplete( "widget" );
        element.val( "ja" ).keydown();
@@ -112,7 +112,7 @@ asyncTest( "delay", function() {
        setTimeout(function() {
                ok( menu.is( ":visible" ), "menu is open after delay" );
                start();
-       }, 100 );
+       }, 50 );
 });
 
 asyncTest( "disabled", function() {
@@ -133,7 +133,7 @@ asyncTest( "disabled", function() {
        setTimeout(function() {
                ok( menu.is( ":hidden" ) );
                start();
-       }, 50 );
+       });
 });
 
 test( "minLength", function() {
@@ -158,7 +158,7 @@ asyncTest( "minLength, exceed then drop below", function() {
                                equal( req.term, "12", "correct search term" );
                                setTimeout(function() {
                                        res([ "item" ]);
-                               }, 1 );
+                               });
                        }
                }),
                menu = element.autocomplete( "widget" );
@@ -172,7 +172,7 @@ asyncTest( "minLength, exceed then drop below", function() {
        setTimeout(function() {
                ok( menu.is( ":hidden" ), "menu is hidden after searches" );
                start();
-       }, 50 );
+       });
 });
 
 test( "minLength, exceed then drop below then exceed", function() {
index 49b93bdd17caea8eb63115646f37e6e9d938b9b6..f4b353b8dd459eb728d05aede1172beb2e629d63 100644 (file)
@@ -337,7 +337,7 @@ asyncTest("ensure dialog's container doesn't scroll on resize and focus", functi
                equal($(window).scrollTop(), initialScroll, "scroll hasn't moved after focus moved to dialog");
                element.dialog("destroy");
                start();
-       }, 500);
+       });
 });
 
 test("#5184: isOpen in dialogclose event is true", function() {
index c449ad0d41365c32160c4487f289436eaea32a45..ea4a7002a229d6549367a415ec7a8ff2494bc258 100644 (file)
@@ -87,7 +87,7 @@ asyncTest("#9000: Dialog leaves broken event handler after close/destroy in cert
                $( "#favorite-animal" ).focus();
                ok( true, "close and destroy modal dialog before its really opened" );
                start();
-       }, 2 );
+       });
 });
 
 test("#4980: Destroy should place element back in original DOM position", function(){
index f02f97fca50a49cee5b8843a55007d1fb9f1e9b2..5be717a669162e4cb0a35848d0b39234b9cf5fb4 100644 (file)
@@ -54,9 +54,9 @@ asyncTest( "#9532: Need a way in Menu to keep ui-state-active class on selected
        element.menu( "focus", null, firstChild );
        wrapper.addClass( "ui-state-active" );
        setTimeout( function() {
-               ok( wrapper.is( ".ui-state-active" ), "ui-state-active improperly removed" );
+               ok( wrapper.is( ".ui-state-active" ) );
                start();
-       }, 500 );
+       });
 });
 
 })( jQuery );
index b1ade3b85b85733c95e020b610eb73b2cd604013..ad13de3b8959fe76bc9585d2182dd6ab30c3c062 100644 (file)
@@ -111,62 +111,66 @@ asyncTest( "handle focus of menu with active item", function() {
        });
 });
 
-asyncTest( "handle submenu auto collapse: mouseleave", function() {
+asyncTest( "handle submenu auto collapse: mouseleave, default markup", function() {
        expect( 4 );
+       $.ui.menu.prototype.delay = 1;
        var element = $( "#menu2" ).menu(),
                event = $.Event( "mouseenter" );
 
        function menumouseleave1() {
                equal( element.find( "ul[aria-expanded='true']" ).length, 1, "first submenu expanded" );
                element.menu( "focus", event, element.find( "li:nth-child(7) li:first" ) );
-               setTimeout( menumouseleave2, 350 );
+               setTimeout( menumouseleave2, 25 );
        }
        function menumouseleave2() {
                equal( element.find( "ul[aria-expanded='true']" ).length, 2, "second submenu expanded" );
                element.find( "ul[aria-expanded='true']:first" ).trigger( "mouseleave" );
-               setTimeout( menumouseleave3, 350 );
+               setTimeout( menumouseleave3, 25 );
        }
        function menumouseleave3() {
                equal( element.find( "ul[aria-expanded='true']" ).length, 1, "second submenu collapsed" );
                element.trigger( "mouseleave" );
-               setTimeout( menumouseleave4, 350 );
+               setTimeout( menumouseleave4, 25 );
        }
        function menumouseleave4() {
                equal( element.find( "ul[aria-expanded='true']" ).length, 0, "first submenu collapsed" );
+               $.ui.menu.prototype.delay = 300;
                start();
        }
 
        element.find( "li:nth-child(7)" ).trigger( "mouseenter" );
-       setTimeout( menumouseleave1, 350 );
+       setTimeout( menumouseleave1, 25 );
 });
 
-asyncTest( "handle submenu auto collapse: mouseleave", function() {
+asyncTest( "handle submenu auto collapse: mouseleave, custom markup", function() {
        expect( 4 );
+       $.ui.menu.prototype.delay = 1;
        var element = $( "#menu5" ).menu({ menus: ".menu" }),
                event = $.Event( "mouseenter" );
 
        function menumouseleave1() {
                equal( element.find( "div[aria-expanded='true']" ).length, 1, "first submenu expanded" );
                element.menu( "focus", event, element.find( ":nth-child(7)" ).find( ".menu" ).eq( 0 ).children().eq( 0 ) );
-               setTimeout( menumouseleave2, 350 );
+               setTimeout( menumouseleave2, 25 );
        }
        function menumouseleave2() {
                equal( element.find( "div[aria-expanded='true']" ).length, 2, "second submenu expanded" );
                element.find( "div[aria-expanded='true']:first" ).trigger( "mouseleave" );
-               setTimeout( menumouseleave3, 350 );
+               setTimeout( menumouseleave3, 25 );
        }
        function menumouseleave3() {
                equal( element.find( "div[aria-expanded='true']" ).length, 1, "second submenu collapsed" );
                element.trigger( "mouseleave" );
-               setTimeout( menumouseleave4, 350 );
+               setTimeout( menumouseleave4, 25 );
        }
        function menumouseleave4() {
                equal( element.find( "div[aria-expanded='true']" ).length, 0, "first submenu collapsed" );
+               $.ui.menu.prototype.delay = 300;
                start();
        }
 
        element.find( ":nth-child(7)" ).trigger( "mouseenter" );
-       setTimeout( menumouseleave1, 350 );
+       setTimeout( menumouseleave1, 25 );
 });
 
 asyncTest( "handle keyboard navigation on menu without scroll and without submenus", function() {
@@ -251,7 +255,7 @@ asyncTest( "handle keyboard navigation on menu without scroll and with submenus"
                element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
                element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
                equal( logOutput(), "keydown,1,2", "Keydown DOWN" );
-               setTimeout( menukeyboard1, 50 );
+               setTimeout( menukeyboard1 );
        });
        element.focus();
 
@@ -274,8 +278,8 @@ asyncTest( "handle keyboard navigation on menu without scroll and with submenus"
 
                setTimeout(function() {
                        equal( logOutput(), "keydown,1,2,3,4,0", "Keydown RIGHT (open submenu)" );
-                       setTimeout( menukeyboard2, 50 );
-               }, 50 );
+                       setTimeout( menukeyboard2 );
+               } );
        }
 
        function menukeyboard2() {
@@ -285,7 +289,7 @@ asyncTest( "handle keyboard navigation on menu without scroll and with submenus"
 
                // re-open submenu
                element.simulate( "keydown", { keyCode: $.ui.keyCode.RIGHT } );
-               setTimeout( menukeyboard3, 50 );
+               setTimeout( menukeyboard3 );
        }
 
        function menukeyboard3() {
@@ -319,7 +323,7 @@ asyncTest( "handle keyboard navigation on menu without scroll and with submenus"
 
                log( "keydown", true );
                element.simulate( "keydown", { keyCode: $.ui.keyCode.SPACE } );
-               setTimeout( menukeyboard4, 50 );
+               setTimeout( menukeyboard4 );
        }
 
        function menukeyboard4() {
@@ -344,9 +348,9 @@ asyncTest( "handle keyboard navigation on menu without scroll and with submenus"
 
                                log( "keydown", true );
                                element.simulate( "keydown", { keyCode: $.ui.keyCode.ENTER } );
-                               setTimeout( menukeyboard6, 50 );
-                       }, 50 );
-               }, 50 );
+                               setTimeout( menukeyboard6 );
+                       } );
+               } );
        }
 
        function menukeyboard6() {
@@ -446,7 +450,7 @@ asyncTest( "handle keyboard navigation on menu with scroll and with submenus", f
                element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
                element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
                equal( logOutput(), "keydown,1,2", "Keydown DOWN" );
-               setTimeout( menukeyboard1, 50 );
+               setTimeout( menukeyboard1 );
        });
        element.focus();
 
index ef8279018e0f013af8813f71db86b8106c902203..1cc3ec17952011606561aed1d20ff8fcb43fb98f 100644 (file)
@@ -11,19 +11,13 @@ module( "menu: methods", {
 });
 
 test( "destroy", function() {
-       expect( 4 );
-       domEqual( "#menu1", function() {
-               $( "#menu1" ).menu().menu( "destroy" );
-       });
+       expect( 2 );
        domEqual( "#menu2", function() {
                $( "#menu2" ).menu().menu( "destroy" );
        });
        domEqual( "#menu5", function() {
                $( "#menu5").menu().menu( "destroy" );
        });
-       domEqual( "#menu6", function() {
-               $( "#menu6" ).menu().menu( "destroy" );
-       });
 });
 
 test( "enable/disable", function() {
index 2d18a6f1ab7556efc5e3d30da1ef33acf3503850..ab3c8117559e23f1a9e294dd86a9a61fbcac7d74 100644 (file)
@@ -155,7 +155,7 @@ asyncTest( "keyboard support - LEFT, RIGHT, UP, DOWN, HOME, END, SPACE, ENTER",
                panels = element.find( ".ui-tabs-panel" ),
                keyCode = $.ui.keyCode;
 
-       element.tabs( "instance" ).delay = 50;
+       element.tabs( "instance" ).delay = 1;
 
        equal( tabs.filter( ".ui-state-focus" ).length, 0, "no tabs focused on init" );
        tabs.eq( 0 ).simulate( "focus" );
@@ -208,7 +208,7 @@ asyncTest( "keyboard support - LEFT, RIGHT, UP, DOWN, HOME, END, SPACE, ENTER",
                equal( tabs.eq( 0 ).attr( "aria-expanded" ), "true", "first tab has aria-expanded=true" );
                equal( panels.eq( 0 ).attr( "aria-hidden" ), "false", "first panel has aria-hidden=false" );
 
-               setTimeout( step2, 100 );
+               setTimeout( step2, 25 );
        }
 
        // left, home, space
@@ -246,7 +246,7 @@ asyncTest( "keyboard support - LEFT, RIGHT, UP, DOWN, HOME, END, SPACE, ENTER",
 
                // SPACE activates, cancels delay
                tabs.eq( 0 ).simulate( "keydown", { keyCode: keyCode.SPACE } );
-               setTimeout( step3, 1 );
+               setTimeout( step3 );
        }
 
        // end, enter
@@ -273,7 +273,7 @@ asyncTest( "keyboard support - LEFT, RIGHT, UP, DOWN, HOME, END, SPACE, ENTER",
 
                // ENTER activates, cancels delay
                tabs.eq( 0 ).simulate( "keydown", { keyCode: keyCode.ENTER } );
-               setTimeout( step4, 1 );
+               setTimeout( step4 );
        }
 
        // enter (collapse)
@@ -293,10 +293,10 @@ asyncTest( "keyboard support - LEFT, RIGHT, UP, DOWN, HOME, END, SPACE, ENTER",
                equal( tabs.eq( 2 ).attr( "aria-expanded" ), "false", "third tab has aria-expanded=false" );
                equal( panels.eq( 2 ).attr( "aria-hidden" ), "true", "third panel has aria-hidden=true" );
 
-               setTimeout( start, 1 );
+               setTimeout( start );
        }
 
-       setTimeout( step1, 1 );
+       setTimeout( step1 );
 });
 
 // Navigation with CTRL and COMMAND (both behave the same)
@@ -311,7 +311,7 @@ $.each({
                        panels = element.find( ".ui-tabs-panel" ),
                        keyCode = $.ui.keyCode;
 
-               element.tabs( "instance" ).delay = 50;
+               element.tabs( "instance" ).delay = 1;
 
                equal( tabs.filter( ".ui-state-focus" ).length, 0, "no tabs focused on init" );
                tabs.eq( 0 ).simulate( "focus" );
@@ -337,7 +337,7 @@ $.each({
                        equal( tabs.eq( 0 ).attr( "aria-expanded" ), "true", "first tab has aria-expanded=true" );
                        equal( panels.eq( 0 ).attr( "aria-hidden" ), "false", "first panel has aria-hidden=false" );
 
-                       setTimeout( step2, 100 );
+                       setTimeout( step2, 25 );
                }
 
                // right
@@ -364,7 +364,7 @@ $.each({
                        equal( tabs.eq( 0 ).attr( "aria-expanded" ), "true", "first tab has aria-expanded=true" );
                        equal( panels.eq( 0 ).attr( "aria-hidden" ), "false", "first panel has aria-hidden=false" );
 
-                       setTimeout( step3, 100 );
+                       setTimeout( step3, 25 );
                }
 
                // down (wrap)
@@ -387,7 +387,7 @@ $.each({
                        equal( tabs.eq( 0 ).attr( "aria-expanded" ), "true", "first tab has aria-expanded=true" );
                        equal( panels.eq( 0 ).attr( "aria-hidden" ), "false", "first panel has aria-hidden=false" );
 
-                       setTimeout( step4, 100 );
+                       setTimeout( step4, 25 );
                }
 
                // up (wrap)
@@ -411,7 +411,7 @@ $.each({
                        equal( tabs.eq( 0 ).attr( "aria-expanded" ), "true", "first tab has aria-expanded=true" );
                        equal( panels.eq( 0 ).attr( "aria-hidden" ), "false", "first panel has aria-hidden=false" );
 
-                       setTimeout( step5, 100 );
+                       setTimeout( step5, 25 );
                }
 
                // left
@@ -438,7 +438,7 @@ $.each({
                        equal( tabs.eq( 0 ).attr( "aria-expanded" ), "true", "first tab has aria-expanded=true" );
                        equal( panels.eq( 0 ).attr( "aria-hidden" ), "false", "first panel has aria-hidden=false" );
 
-                       setTimeout( step6, 100 );
+                       setTimeout( step6, 25 );
                }
 
                // home
@@ -465,7 +465,7 @@ $.each({
                        equal( tabs.eq( 0 ).attr( "aria-expanded" ), "true", "first tab has aria-expanded=true" );
                        equal( panels.eq( 0 ).attr( "aria-hidden" ), "false", "first panel has aria-hidden=false" );
 
-                       setTimeout( step7, 100 );
+                       setTimeout( step7, 25 );
                }
 
                // end
@@ -489,7 +489,7 @@ $.each({
                        equal( tabs.eq( 0 ).attr( "aria-expanded" ), "true", "first tab has aria-expanded=true" );
                        equal( panels.eq( 0 ).attr( "aria-hidden" ), "false", "first panel has aria-hidden=false" );
 
-                       setTimeout( step8, 100 );
+                       setTimeout( step8, 25 );
                }
 
                // space
@@ -512,10 +512,10 @@ $.each({
                        equal( tabs.eq( 0 ).attr( "aria-expanded" ), "false", "first tab has aria-expanded=false" );
                        equal( panels.eq( 0 ).attr( "aria-hidden" ), "true", "first panel has aria-hidden=true" );
 
-                       setTimeout( start, 1 );
+                       setTimeout( start );
                }
 
-               setTimeout( step1, 1 );
+               setTimeout( step1 );
        });
 });
 
@@ -567,7 +567,7 @@ asyncTest( "keyboard support - CTRL+UP, ALT+PAGE_DOWN, ALT+PAGE_UP", function()
                equal( panels.eq( 2 ).attr( "aria-hidden" ), "true", "third panel has aria-hidden=true" );
 
                panels.eq( 0 ).simulate( "focus" );
-               setTimeout( step2, 1 );
+               setTimeout( step2 );
        }
 
        function step2() {
@@ -596,7 +596,7 @@ asyncTest( "keyboard support - CTRL+UP, ALT+PAGE_DOWN, ALT+PAGE_UP", function()
                equal( panels.eq( 2 ).attr( "aria-hidden" ), "true", "third panel has aria-hidden=true" );
 
                panels.eq( 1 ).simulate( "focus" );
-               setTimeout( step3, 1 );
+               setTimeout( step3 );
        }
 
        function step3() {
@@ -605,10 +605,10 @@ asyncTest( "keyboard support - CTRL+UP, ALT+PAGE_DOWN, ALT+PAGE_UP", function()
                panels.eq( 1 ).simulate( "keydown", { keyCode: keyCode.UP, ctrlKey: true } );
                strictEqual( document.activeElement, tabs[ 1 ], "second tab is activeElement" );
 
-               setTimeout( start, 1 );
+               setTimeout( start );
        }
 
-       setTimeout( step1, 1 );
+       setTimeout( step1 );
 });
 
 test( "#3627 - Ajax tab with url containing a fragment identifier fails to load", function() {
index a3d663f272d53d1c9409aaf9a15defacc5744ff8..073430b77904fa63ba5ae6a4125aa9ca4e2b00df 100644 (file)
@@ -206,7 +206,7 @@ asyncTest( "load", function() {
                strictEqual( uiPanel[ 0 ], panel[ 0 ], "panel" );
                equal( uiPanel.find( "p" ).length, 1, "panel html" );
                state( element, 1, 0, 0, 0, 0 );
-               setTimeout( tabsload1, 100 );
+               setTimeout( tabsload1 );
        });
        element.tabs( "load", 3 );
        state( element, 1, 0, 0, 0, 0 );
@@ -218,7 +218,7 @@ asyncTest( "load", function() {
                });
                element.one( "tabsload", function() {
                        ok( true, "tabsload invoked" );
-                       setTimeout( tabsload2, 100 );
+                       setTimeout( tabsload2 );
                });
                element.tabs( "option", "active", 3 );
                state( element, 0, 0, 0, 1, 0 );
index 7f5352bac56bb3d4b88807454f8c5fd970ee9f64..56e32b7da31f8e606b8cfb7e08b9599a54bc862f 100644 (file)
@@ -347,7 +347,7 @@ asyncTest( "hide and show - animation", function() {
                equal( options, "drop", "correct options in _show()" );
                setTimeout(function() {
                        callback();
-               }, 1 );
+               });
        };
        widget._hide = function( element, options, callback ) {
                strictEqual( element[ 0 ], panels[ 0 ], "correct element in _hide()" );
@@ -355,7 +355,7 @@ asyncTest( "hide and show - animation", function() {
                setTimeout(function() {
                        callback();
                        start();
-               }, 1 );
+               });
        };
 
        ok( panels.eq( 0 ).is( ":visible" ), "first panel visible" );