Skip to content

Commit 2c9f5a8

Browse files
committed
Selectmenu Tests: use trigger instead of simulate for click events in order to make tests work in IE/ and IE8
1 parent b735152 commit 2c9f5a8

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

tests/unit/selectmenu/selectmenu_core.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ $.each([
100100
setTimeout(function() {
101101
links = menu.find("li.ui-menu-item a");
102102

103-
button.simulate( "click" );
103+
button.trigger( "click" );
104104
menu.find( "a" ).last().simulate( "mouseover" ).trigger( "click" );
105105
equal(
106106
menu.attr( "aria-activedescendant" ),
@@ -137,10 +137,10 @@ $.each([
137137
setTimeout(function() {
138138
links = menu.find( "li.ui-menu-item a" );
139139
// open menu and click first item
140-
button.simulate( "click" );
140+
button.trigger( "click" );
141141
links.first().simulate( "mouseover" ).trigger( "click" );
142142
// open menu again and hover item
143-
button.simulate( "click" );
143+
button.trigger( "click" );
144144
links.eq( 3 ).simulate( "mouseover" );
145145
// close and use keyboard control on button
146146
button.simulate( "keydown", { keyCode: $.ui.keyCode.ESCAPE } );

tests/unit/selectmenu/selectmenu_events.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,8 @@ asyncTest( "change", function () {
2929
button.simulate( "focus" );
3030

3131
setTimeout(function() {
32-
button.simulate( "click" );
33-
menu.find( "a" ).eq( optionIndex ).simulate( "mouseover" ).simulate( "click" );
32+
button.trigger( "click" );
33+
menu.find( "a" ).eq( optionIndex ).simulate( "mouseover" ).trigger( "click" );
3434
start();
3535
}, 1 );
3636
});
@@ -49,7 +49,7 @@ test( "close", function () {
4949
this.element.selectmenu( "open" ).selectmenu( "close" );
5050

5151
this.element.selectmenu( "open" );
52-
$( "body" ).simulate( "click" );
52+
$( "body" ).trigger( "click" );
5353
});
5454

5555

@@ -78,7 +78,7 @@ asyncTest( "focus", function () {
7878
setTimeout(function() {
7979
button.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
8080

81-
button.simulate( "click" );
81+
button.trigger( "click" );
8282
links = menu.find( "li.ui-menu-item a" );
8383
optionIndex = 0;
8484
links.eq( optionIndex ).simulate( "mouseover" );
@@ -126,8 +126,8 @@ asyncTest( "select", function () {
126126
button.simulate( "focus" );
127127

128128
setTimeout(function() {
129-
button.simulate( "click" );
130-
menu.find( "a" ).eq( optionIndex ).simulate( "mouseover" ).simulate( "click" );
129+
button.trigger( "click" );
130+
menu.find( "a" ).eq( optionIndex ).simulate( "mouseover" ).trigger( "click" );
131131
start();
132132
}, 1 );
133133
});

0 commit comments

Comments
 (0)