Skip to content

Commit d83e169

Browse files
committed
Selectmenu: fixed unit tests due to initial refresh method changes
1 parent c5d9dbc commit d83e169

File tree

3 files changed

+22
-4
lines changed

3 files changed

+22
-4
lines changed

tests/unit/selectmenu/selectmenu_core.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ $.each([
5353

5454
equals( button.text(), selected.text(), "inital button text" );
5555

56+
link.simulate( "focus" );
5657
link.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
5758
equals( element.find("option:selected").val(), selected.next("option").val() , "after keydown original select state" );
5859
equals( button.text(), selected.next("option").text(), "after keydown button text" );

tests/unit/selectmenu/selectmenu_events.js

Lines changed: 20 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,11 @@ test("change", function () {
2222
var widget = this.element.selectmenu("widget"),
2323
menu = widget.filter(".ui-selectmenu-menu"),
2424
button = widget.filter(".ui-selectmenu-button"),
25+
link = button.find("a"),
2526
value = this.element.find("option").first().text();
26-
27-
button.find("a").simulate( "click" );
27+
28+
link.simulate( "focus" );
29+
link.simulate( "click" );
2830
menu.find("a").first().simulate( "mouseover" ).simulate( "click" );
2931
});
3032

@@ -40,6 +42,11 @@ test("close", function () {
4042
}
4143
});
4244

45+
var widget = this.element.selectmenu("widget"),
46+
button = widget.filter(".ui-selectmenu-button"),
47+
link = button.find("a");
48+
49+
link.simulate( "focus" );
4350
this.element.selectmenu("open").selectmenu("close");
4451
});
4552

@@ -63,9 +70,11 @@ test("focus", function () {
6370

6471
var widget = this.element.selectmenu("widget"),
6572
button = widget.filter(".ui-selectmenu-button"),
73+
link = button.find("a"),
6674
menu = widget.filter(".ui-selectmenu-menu");
6775

68-
button.find("a").simulate( "click" );
76+
link.simulate( "focus" );
77+
link.simulate( "click" );
6978
menu.find(".ui-menu-item").simulate("mouseover");
7079
});
7180

@@ -81,6 +90,11 @@ test("open", function () {
8190
}
8291
});
8392

93+
var widget = this.element.selectmenu("widget"),
94+
button = widget.filter(".ui-selectmenu-button"),
95+
link = button.find("a");
96+
97+
link.simulate( "focus" );
8498
this.element.selectmenu("open");
8599
});
86100

@@ -99,9 +113,11 @@ test("select", function () {
99113

100114
var widget = this.element.selectmenu("widget"),
101115
button = widget.filter(".ui-selectmenu-button"),
116+
link = button.find("a"),
102117
menu = widget.filter(".ui-selectmenu-menu");
103118

104-
button.find("a").simulate( "click" );
119+
link.simulate( "focus" );
120+
link.simulate( "click" );
105121
menu.find("a").first().simulate( "mouseover" ).simulate("click");
106122
});
107123

tests/unit/selectmenu/selectmenu_options.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ test("dropdown: CSS styles", function () {
2323
menu = widget.filter(".ui-selectmenu-menu"),
2424
ul = widget.find("ul");
2525

26+
link.simulate( "focus" );
2627
this.element.selectmenu("open");
2728
ok( link.hasClass("ui-corner-top") && !link.hasClass("ui-corner-all"), "button styles dropdown");
2829
ok( ul.hasClass("ui-corner-bottom") && !ul.hasClass("ui-corner-all"), "menu styles dropdown");

0 commit comments

Comments
 (0)