Selectmenu Tests: cleanup unit tests

jquery · Apr 29, 2013 · c73f7e8 · c73f7e8
1 parent 4d986a4
commit c73f7e8
Show file tree

Hide file tree

Showing 3 changed files with 45 additions and 12 deletions.
diff --git a/tests/unit/selectmenu/selectmenu_common.js b/tests/unit/selectmenu/selectmenu_common.js
@@ -10,6 +10,7 @@ TestHelpers.commonWidgetTests( "selectmenu", {
 			at: "left bottom",
 			collision: "none"
 		},
+
 		// callbacks
 		create: null,
 		change: null,

diff --git a/tests/unit/selectmenu/selectmenu_core.js b/tests/unit/selectmenu/selectmenu_core.js
@@ -15,17 +15,25 @@ test("accessibility", function () {
 
 	equal( button.attr( "role" ), "combobox", "button link role" );
 	equal( button.attr( "aria-haspopup" ), "true", "button link aria-haspopup" );
-	equal( button.attr( "aria-expanded" ), "false", "button link  aria-expanded" );
-	equal( button.attr( "aria-autocomplete" ), "list", "button link  aria-autocomplete" );
+	equal( button.attr( "aria-expanded" ), "false", "button link aria-expanded" );
+	equal( button.attr( "aria-autocomplete" ), "list", "button link aria-autocomplete" );
 	equal( button.attr( "aria-owns" ), menu.attr("id"), "button link aria-owns" );
-	equal( button.attr( "aria-labelledby" ), links.eq( element[0].selectedIndex ).attr( "id" ), "button link aria-labelledby" );
+	equal( 
+		button.attr( "aria-labelledby" ), 
+		links.eq( element[0].selectedIndex ).attr( "id" ), 
+		"button link aria-labelledby" 
+	);
 	equal( button.attr( "tabindex" ), 0, "button link tabindex" );
 
 	equal( menu.attr( "role" ), "listbox", "menu role" );
 	equal( menu.attr( "aria-labelledby" ), button.attr( "id" ), "menu aria-labelledby" );
 	equal( menu.attr( "aria-hidden" ), "true", "menu aria-hidden" );
 	equal( menu.attr( "tabindex" ), 0, "menu tabindex" );
-	equal( menu.attr( "aria-activedescendant" ), links.eq( element[ 0 ].selectedIndex ).attr( "id" ), "menu aria-activedescendant" );
+	equal( 
+		menu.attr( "aria-activedescendant" ), 
+		links.eq( element[ 0 ].selectedIndex ).attr( "id" ), 
+		"menu aria-activedescendant" 
+	);
 	$.each( links, function( index ){
 		equal( $( this ).attr( "role" ), "option", "menu link #" + index +" role" );
 		equal( $( this ).attr( "tabindex" ), -1, "menu link #" + index +" tabindex" );
@@ -56,9 +64,21 @@ $.each([
 		links = menu.find("li.ui-menu-item a");
 
 		button.simulate( "keydown", { keyCode: $.ui.keyCode.DOWN } );
-		equal( menu.attr( "aria-activedescendant" ), links.eq( element[ 0 ].selectedIndex ).attr( "id" ), "menu aria-activedescendant" );
-		equal( button.attr( "aria-activedescendant" ), links.eq( element[ 0 ].selectedIndex ).attr( "id" ), "button aria-activedescendant" );
-		equal( element.find( "option:selected" ).val(), selected.next( "option" ).val() , "original select state" );
+		equal( 
+			menu.attr( "aria-activedescendant" ), 
+			links.eq( element[ 0 ].selectedIndex ).attr( "id" ), 
+			"menu aria-activedescendant" 
+		);
+		equal( 
+			button.attr( "aria-activedescendant" ), 
+			links.eq( element[ 0 ].selectedIndex ).attr( "id" ), 
+			"button aria-activedescendant" 
+		);
+		equal( 
+			element.find( "option:selected" ).val(), 
+			selected.next( "option" ).val() , 
+			"original select state" 
+		);
 		equal( button.text(), selected.next( "option" ).text(), "button text" );
 	});
 
@@ -75,13 +95,25 @@ $.each([
 
 		button.simulate( "click" );
 		menu.find( "a" ).last().simulate( "mouseover" ).trigger( "click" );
-		equal( menu.attr( "aria-activedescendant" ), links.eq( element[ 0 ].selectedIndex ).attr( "id" ), "menu aria-activedescendant" );
-		equal( button.attr( "aria-activedescendant" ), links.eq( element[ 0 ].selectedIndex ).attr( "id" ), "button aria-activedescendant" );
-		equal( element.find( "option:selected" ).val(), element.find( "option" ).last().val(), "original select state" );
+		equal( 
+			menu.attr( "aria-activedescendant" ), 
+			links.eq( element[ 0 ].selectedIndex ).attr( "id" ), 
+			"menu aria-activedescendant" 
+		);
+		equal( 
+			button.attr( "aria-activedescendant" ), 
+			links.eq( element[ 0 ].selectedIndex ).attr( "id" ), 
+			"button aria-activedescendant" 
+		);
+		equal( 
+			element.find( "option:selected" ).val(), 
+			element.find( "option" ).last().val(), 
+			"original select state"
+		);
 		equal( button.text(), element.find( "option" ).last().text(), "button text" );
 	});
 
-	test( "state synchronization - after focus item and keydown on button  - " + settings.type, function () {
+	test( "state synchronization - after focus item and keydown on button - " + settings.type, function () {
 		expect(4);
 
 		var links,

diff --git a/tests/unit/selectmenu/selectmenu_events.js b/tests/unit/selectmenu/selectmenu_events.js
@@ -1,4 +1,4 @@
-(function ($) {
+(function ( $ ) {
 
 module( "selectmenu: events", {
 	setup: function () {