From: Richard Worth <rdworth@gmail.com>
Date: Tue, 15 Mar 2011 13:36:36 +0000 (-0400)
Subject: Autocomplete: refactored unit tests for autoFocus option
X-Git-Tag: 1.8.11~3
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=1fe58dbf525c614dd1b490a66dad43a1711181ee;p=jquery-ui.git

Autocomplete: refactored unit tests for autoFocus option
(cherry picked from commit c1a0f2bf6c7155e73e373818c44b06dcbe18c847)
---

diff --git a/tests/unit/autocomplete/autocomplete_options.js b/tests/unit/autocomplete/autocomplete_options.js
index 09cfb6b5f..be2f28391 100644
--- a/tests/unit/autocomplete/autocomplete_options.js
+++ b/tests/unit/autocomplete/autocomplete_options.js
@@ -97,32 +97,26 @@ test( "appendTo", function() {
 	ac.autocomplete( "destroy" );
 });
 
-test( "autoFocus: false", function() {
+function autoFocusTest( afValue, focusedLength ) {
 	var ac = $( "#autocomplete" ).autocomplete({
-		autoFocus: false,
+		autoFocus: afValue,
 		delay: 0,
 		source: data,
 		open: function( event, ui ) {
-			equals( 0, ac.autocomplete( "widget" ).children( ".ui-menu-item:first .ui-state-focus" ).length, "first item is not auto focused" );
+			equals( focusedLength, ac.autocomplete( "widget" ).children( ".ui-menu-item:first .ui-state-focus" ).length, "first item is " + afValue ? "" : "not" + " auto focused" );
 			start();			
 		}
 	});
 	ac.val( "ja" ).keydown();
 	stop();
+}
+
+test( "autoFocus: false", function() {
+	autoFocusTest( false, 0 );
 });
 
 test( "autoFocus: true", function() {
-	var ac = $( "#autocomplete" ).autocomplete({
-		autoFocus: true,
-		delay: 0,
-		source: data,
-		open: function( event, ui ) {
-			equals( 1, ac.autocomplete( "widget" ).children( ".ui-menu-item:first .ui-state-focus" ).length, "first item is auto focused" );
-			start();			
-		}
-	});
-	ac.val( "ja" ).keydown();
-	stop();
+	autoFocusTest( true, 1 );
 });
 
 test("delay", function() {