Skip to content

Commit 51c0878

Browse files
committed
Menu tests: More complete role option tests.
1 parent 566c5f4 commit 51c0878

File tree

1 file changed

+30
-3
lines changed

1 file changed

+30
-3
lines changed

tests/unit/menu/menu_options.js

Lines changed: 30 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,13 @@ test( "{ disabled: false }", function() {
4040
equal( logOutput(), "click,1,afterclick", "Click order not valid." );
4141
});
4242

43-
test( "{ role: 'menu' } ", function () {
43+
// TODO: test icon option
44+
45+
// TODO: test menus option
46+
47+
// TODO: test position option
48+
49+
test( "{ role: 'menu' } ", function() {
4450
var element = $( "#menu1" ).menu(),
4551
items = element.find( "li" );
4652
expect( 2 + 5 * items.length );
@@ -55,16 +61,37 @@ test( "{ role: 'menu' } ", function () {
5561
});
5662
});
5763

58-
test( "{ role: 'listbox' } ", function () {
64+
test( "{ role: 'listbox' } ", function() {
5965
var element = $( "#menu1" ).menu({
6066
role: "listbox"
6167
}),
6268
items = element.find( "li" );
63-
expect( 2 + items.length );
69+
expect( 2 + 5 * items.length );
6470
equal( element.attr( "role" ), "listbox" );
6571
ok( items.length > 0, "number of menu items" );
6672
items.each(function( item ) {
73+
ok( $( this ).hasClass( "ui-menu-item" ), "menu item ("+ item + ") class for item" );
74+
equal( $( this ).attr( "role" ), "presentation", "menu item ("+ item + ") role" );
6775
equal( $( "a", this ).attr( "role" ), "option", "menu item ("+ item + ") role" );
76+
ok( $( "a", this ).hasClass( "ui-corner-all" ), "a element class for menu item ("+ item + ")" );
77+
equal( $( "a", this ).attr( "tabindex" ), "-1", "a element tabindex for menu item ("+ item + ")" );
78+
});
79+
});
80+
81+
test( "{ role: null }", function() {
82+
var element = $( "#menu1" ).menu({
83+
role: null
84+
}),
85+
items = element.find( "li" );
86+
expect( 2 + 5 * items.length );
87+
strictEqual( element.attr( "role" ), undefined );
88+
ok( items.length > 0, "number of menu items" );
89+
items.each(function( item ) {
90+
ok( $( this ).hasClass( "ui-menu-item" ), "menu item ("+ item + ") class for item" );
91+
equal( $( this ).attr( "role" ), "presentation", "menu item ("+ item + ") role" );
92+
equal( $( "a", this ).attr( "role" ), undefined, "menu item ("+ item + ") role" );
93+
ok( $( "a", this ).hasClass( "ui-corner-all" ), "a element class for menu item ("+ item + ")" );
94+
equal( $( "a", this ).attr( "tabindex" ), "-1", "a element tabindex for menu item ("+ item + ")" );
6895
});
6996
});
7097

0 commit comments

Comments
 (0)