1 files changed, 66 insertions, 37 deletions

diff --git a/tests/unit/menu/menu_methods.js b/tests/unit/menu/menu_methods.js
index c17986816..510ddb1d7 100644
--- a/tests/unit/menu/menu_methods.js
+++ b/tests/unit/menu/menu_methods.js
@@ -1,58 +1,87 @@
-/*
- * menu_methods.js
- */
-(function($) {
+(function( $ ) {
 
 var log = TestHelpers.menu.log,
+	logOutput = TestHelpers.menu.logOutput,
 	click = TestHelpers.menu.click;
 
-module("menu: methods");
+module( "menu: methods", {
+	setup: function() {
+		TestHelpers.menu.clearLog();
+	}
+});
 
 test( "enable/disable", function() {
 	expect( 3 );
-	var menu = $( "#menu1" ).menu({
-		select: function(event, ui) {
+	var element = $( "#menu1" ).menu({
+		select: function( event, ui ) {
 			log();
 		}
 	});
-	menu.menu("disable");
-	ok(menu.is(".ui-state-disabled"),"Missing ui-state-disabled class");
-	log("click",true);
-	click(menu,"1");
-	log("afterclick");
-	menu.menu("enable");
-	ok(menu.not(".ui-state-disabled"),"Has ui-state-disabled class");
-	log("click");
-	click(menu,"1");
-	log("afterclick");
-	equal( $("#log").html(), "afterclick,1,click,afterclick,click,", "Click order not valid.");
+	element.menu( "disable" );
+	ok( element.is( ".ui-state-disabled" ), "Missing ui-state-disabled class" );
+	log( "click", true );
+	click( element, "1" );
+	log( "afterclick" );
+	element.menu( "enable" );
+	ok( element.not( ".ui-state-disabled" ), "Has ui-state-disabled class" );
+	log( "click" );
+	click( element, "1" );
+	log( "afterclick" );
+	equal( logOutput(), "click,afterclick,click,1,afterclick", "Click order not valid." );
 });
 
 test( "refresh", function() {
 	expect( 5 );
-	var menu = $( "#menu1" ).menu();
-	equal(menu.find(".ui-menu-item").length,5,"Incorrect number of menu items");
-	menu.append("<li><a href='#'>test item</a></li>").menu("refresh");
-	equal(menu.find(".ui-menu-item").length,6,"Incorrect number of menu items");
-	menu.find(".ui-menu-item:last").remove().end().menu("refresh");
-	equal(menu.find(".ui-menu-item").length,5,"Incorrect number of menu items");
-	menu.append("<li>---</li>").menu("refresh");
-	equal(menu.find(".ui-menu-item").length,5,"Incorrect number of menu items");
-	menu.children(":last").remove().end().menu("refresh");
-	equal(menu.find(".ui-menu-item").length,5,"Incorrect number of menu items");
+	var element = $( "#menu1" ).menu();
+	equal( element.find( ".ui-menu-item" ).length, 5, "Incorrect number of menu items" );
+	element.append( "<li><a href='#'>test item</a></li>" ).menu( "refresh" );
+	equal( element.find( ".ui-menu-item" ).length, 6, "Incorrect number of menu items" );
+	element.find( ".ui-menu-item:last" ).remove().end().menu( "refresh" );
+	equal( element.find( ".ui-menu-item" ).length, 5, "Incorrect number of menu items" );
+	element.append( "<li>---</li>" ).menu( "refresh" );
+	equal( element.find( ".ui-menu-item" ).length, 5, "Incorrect number of menu items" );
+	element.children( ":last" ).remove().end().menu( "refresh" );
+	equal( element.find( ".ui-menu-item" ).length, 5, "Incorrect number of menu items" );
 });
 
-test("destroy", function() {
-	domEqual("#menu1", function() {
-		$("#menu1").menu().menu("destroy");
+// TODO: test focus method
+
+// TODO: test blur method
+
+// TODO: test collapseAll method
+
+// TODO: test collapse method
+
+// TODO: test expand method
+
+// TODO: test next method
+
+// TODO: test prev method
+
+// TODO: test isFirstItem method
+
+// TODO: test isLastItem method
+
+// TODO: test nextPage method
+
+// TODO: test prevPage method
+
+// TODO: test select method
+
+test( "destroy", function() {
+	expect( 4 );
+	domEqual( "#menu1", function() {
+		$( "#menu1" ).menu().menu( "destroy" );
+	});
+	domEqual( "#menu2", function() {
+		$( "#menu2" ).menu().menu( "destroy" );
 	});
-	domEqual("#menu5", function() {
-		$("#menu5").menu().menu("destroy");
+	domEqual( "#menu5", function() {
+		$( "#menu5").menu().menu( "destroy" );
 	});
-	domEqual("#menu6", function() {
-		$("#menu6").menu().menu("destroy");
+	domEqual( "#menu6", function() {
+		$( "#menu6" ).menu().menu( "destroy" );
 	});
 });
 
-
-})(jQuery);
+})( jQuery );