Skip to content
Permalink
Browse files

Autocomplete: added unit tests for autoFocus. Fixed #7032 - Autocompl…

…ete: Add option to automatically highlight the first result
  • Loading branch information
rdworth committed Mar 15, 2011
1 parent 2f5eade commit 56b7ec134d8e1203f700f99dabcdad8c50a9b0c2
Showing with 27 additions and 0 deletions.
  1. +27 −0 tests/unit/autocomplete/autocomplete_options.js
@@ -97,6 +97,33 @@ test( "appendTo", function() {
ac.autocomplete( "destroy" );
});

test( "autoFocus: false", function() {
var ac = $( "#autocomplete" ).autocomplete({
autoFocus: false,
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" );
start();
}
});
ac.val( "ja" ).keydown();
stop();
});

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();
});

test("delay", function() {
var ac = $("#autocomplete").autocomplete({

2 comments on commit 56b7ec1

@jzaefferer

This comment has been minimized.

Copy link
Member

@jzaefferer jzaefferer replied Mar 15, 2011

Could you refactor these? The test should be just test("autoFocus: true/false", function() { autoFocusTest(true/false, 1/0); });

@rdworth

This comment has been minimized.

Copy link
Contributor Author

@rdworth rdworth replied Mar 15, 2011

Done

Please sign in to comment.
You can’t perform that action at this time.