Skip to content

Commit 9ea5b5d

Browse files
committed
Fix menu unit tests (regression fron Hans' branch)
1 parent 5c47098 commit 9ea5b5d

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

tests/unit/menu/menu_core.js

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,23 +13,26 @@ test("accessibility", function () {
1313
var item0 = $("li:eq(0) a");
1414

1515
ok( ac.hasClass("ui-menu ui-widget ui-widget-content ui-corner-all"), "menu class");
16-
equals( ac.attr("role"), "listbox", "main role");
16+
equals( ac.attr("role"), "menu", "main role");
1717
ok( !ac.attr("aria-activedescendant"), "aria attribute not yet active");
18+
1819
var item = ac.find( "li:first" ).find( "a" ).attr( "id", "xid" ).end();
1920
ac.menu( "focus", $.Event(), item );
2021
equals( ac.attr("aria-activedescendant"), "xid", "aria attribute, id from dom");
22+
2123
var item = ac.find( "li:last" );
2224
ac.menu( "focus", $.Event(), item );
23-
equals( ac.attr("aria-activedescendant"), "menu1-activedescendant", "aria attribute, generated id");
25+
equals( ac.attr("aria-activedescendant"), "menu1-4", "aria attribute, generated id");
2426
});
2527

2628
test("items class and role", function () {
2729
var ac = $('#menu1').menu();
28-
expect(1 + 4 * $("li",ac).length);
30+
expect(1 + 5 * $("li",ac).length);
2931
ok( ($("li",ac).length > 0 ), "number of menu items");
3032
$("li",ac).each(function(item) {
3133
ok( $(this).hasClass("ui-menu-item"), "menu item ("+ item + ") class for item");
32-
equals( $(this).attr("role"), "menuitem", "menu item ("+ item + ") role");
34+
equals( $(this).attr("role"), "presentation", "menu item ("+ item + ") role");
35+
equals( $("a", this).attr("role"), "menuitem", "menu item ("+ item + ") role");
3336
ok( $("a",this).hasClass("ui-corner-all"), "a element class for menu item ("+ item + ") ");
3437
equals( $("a",this).attr("tabindex"), "-1", "a element tabindex for menu item ("+ item + ") ");
3538
});

0 commit comments

Comments
 (0)