Skip to content

Commit 3ef4db2

Browse files
committed
Menu: Focus the menu before performing keyboard unit tests.
1 parent b968ac3 commit 3ef4db2

File tree

1 file changed

+15
-8
lines changed

1 file changed

+15
-8
lines changed

tests/unit/menu/menu_events.js

Lines changed: 15 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -144,13 +144,14 @@ test("handle keyboard navigation on menu without scroll and without submenus", f
144144
});
145145

146146
menu_log("keydown",true);
147+
element.focus();
147148
element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
148149
element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
149-
equals( $("#log").html(), "1,0,keydown,", "Keydown DOWN");
150+
equals( $("#log").html(), "2,1,0,keydown,", "Keydown DOWN");
150151

151152
menu_log("keydown",true);
152153
element.simulate( "keydown", { keyCode: $.ui.keyCode.UP } );
153-
equals( $("#log").html(), "0,keydown,", "Keydown UP");
154+
equals( $("#log").html(), "1,keydown,", "Keydown UP");
154155

155156
menu_log("keydown",true);
156157
element.simulate( "keydown", { keyCode: $.ui.keyCode.LEFT } );
@@ -205,13 +206,15 @@ asyncTest("handle keyboard navigation on menu without scroll and with submenus",
205206
});
206207

207208
menu_log("keydown",true);
209+
element.focus();
208210
element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
209211
element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
210-
equals( $("#log").html(), "1,0,keydown,", "Keydown DOWN");
212+
equals( $("#log").html(), "2,1,0,keydown,", "Keydown DOWN");
211213

212214
menu_log("keydown",true);
213215
element.simulate( "keydown", { keyCode: $.ui.keyCode.UP } );
214-
equals( $("#log").html(), "0,keydown,", "Keydown UP");
216+
element.simulate( "keydown", { keyCode: $.ui.keyCode.UP } );
217+
equals( $("#log").html(), "0,1,keydown,", "Keydown UP");
215218

216219
menu_log("keydown",true);
217220
element.simulate( "keydown", { keyCode: $.ui.keyCode.LEFT } );
@@ -294,13 +297,15 @@ test("handle keyboard navigation on menu with scroll and without submenus", func
294297
});
295298

296299
menu_log("keydown",true);
300+
element.focus();
297301
element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
298302
element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
299-
equals( $("#log").html(), "1,0,keydown,", "Keydown DOWN");
303+
equals( $("#log").html(), "2,1,0,keydown,", "Keydown DOWN");
300304

301305
menu_log("keydown",true);
302306
element.simulate( "keydown", { keyCode: $.ui.keyCode.UP } );
303-
equals( $("#log").html(), "0,keydown,", "Keydown UP");
307+
element.simulate( "keydown", { keyCode: $.ui.keyCode.UP } );
308+
equals( $("#log").html(), "0,1,keydown,", "Keydown UP");
304309

305310
menu_log("keydown",true);
306311
element.simulate( "keydown", { keyCode: $.ui.keyCode.LEFT } );
@@ -363,13 +368,15 @@ asyncTest("handle keyboard navigation on menu with scroll and with submenus", fu
363368
});
364369

365370
menu_log("keydown",true);
371+
element.focus();
366372
element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
367373
element.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
368-
equals( $("#log").html(), "1,0,keydown,", "Keydown DOWN");
374+
equals( $("#log").html(), "2,1,0,keydown,", "Keydown DOWN");
369375

370376
menu_log("keydown",true);
371377
element.simulate( "keydown", { keyCode: $.ui.keyCode.UP } );
372-
equals( $("#log").html(), "0,keydown,", "Keydown UP");
378+
element.simulate( "keydown", { keyCode: $.ui.keyCode.UP } );
379+
equals( $("#log").html(), "0,1,keydown,", "Keydown UP");
373380

374381
menu_log("keydown",true);
375382
element.simulate( "keydown", { keyCode: $.ui.keyCode.LEFT } );

0 commit comments

Comments
 (0)