Skip to content

Commit 7af03b7

Browse files
committed
Autocomplete tests: Cleanup.
1 parent 72f82df commit 7af03b7

File tree

2 files changed

+32
-18
lines changed

2 files changed

+32
-18
lines changed

tests/unit/autocomplete/autocomplete.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,8 @@ <h2 id="qunit-userAgent"></h2>
3939
<ol id="qunit-tests"></ol>
4040
<div id="qunit-fixture">
4141

42-
<div id="ac-wrap1" class="ac-wrap"></div>
43-
<div id="ac-wrap2" class="ac-wrap"><input id="autocomplete" class="foo"></div>
42+
<div id="autocomplete-wrap1" class="autocomplete-wrap"></div>
43+
<div id="autocomplete-wrap2" class="autocomplete-wrap"><input id="autocomplete" class="foo"></div>
4444
<div id="autocomplete-contenteditable" contenteditable="" tabindex=0></div>
4545
<textarea id="autocomplete-textarea"></textarea>
4646

tests/unit/autocomplete/autocomplete_options.js

Lines changed: 30 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -2,46 +2,55 @@
22

33
module( "autocomplete: options" );
44

5-
var data = [ "c++", "java", "php", "coldfusion", "javascript", "asp", "ruby", "python", "c", "scala", "groovy", "haskell", "perl" ];
5+
var data = [ "c++", "java", "php", "coldfusion", "javascript", "asp", "ruby",
6+
"python", "c", "scala", "groovy", "haskell", "perl" ];
67

78
test( "appendTo", function() {
89
expect( 8 );
910
var detached = $( "<div>" ),
1011
element = $( "#autocomplete" ).autocomplete();
11-
equal( element.autocomplete( "widget" ).parent()[0], document.body, "defaults to body" );
12+
equal( element.autocomplete( "widget" ).parent()[ 0 ], document.body,
13+
"defaults to body" );
1214
element.autocomplete( "destroy" );
1315

1416
element.autocomplete({
15-
appendTo: ".ac-wrap"
17+
appendTo: ".autocomplete-wrap"
1618
});
17-
equal( element.autocomplete( "widget" ).parent()[0], $( "#ac-wrap1" )[0], "first found element" );
18-
equal( $( "#ac-wrap2 .ui-autocomplete" ).length, 0, "only appends to one element" );
19+
equal( element.autocomplete( "widget" ).parent()[ 0 ],
20+
$( "#autocomplete-wrap1" )[ 0 ], "first found element" );
21+
equal( $( "#autocomplete-wrap2 .ui-autocomplete" ).length, 0,
22+
"only appends to one element" );
1923
element.autocomplete( "destroy" );
2024

21-
$( "#ac-wrap2" ).addClass( "ui-front" );
25+
$( "#autocomplete-wrap2" ).addClass( "ui-front" );
2226
element.autocomplete();
23-
equal( element.autocomplete( "widget" ).parent()[0], $( "#ac-wrap2" )[0], "null, inside .ui-front" );
27+
equal( element.autocomplete( "widget" ).parent()[ 0 ],
28+
$( "#autocomplete-wrap2" )[ 0 ], "null, inside .ui-front" );
2429
element.autocomplete( "destroy" );
25-
$( "#ac-wrap2" ).removeClass( "ui-front" );
30+
$( "#autocomlete-wrap2" ).removeClass( "ui-front" );
2631

27-
element.autocomplete().autocomplete( "option", "appendTo", "#ac-wrap1" );
28-
equal( element.autocomplete( "widget" ).parent()[0], $( "#ac-wrap1" )[0], "modified after init" );
32+
element.autocomplete().autocomplete( "option", "appendTo", "#autocomplete-wrap1" );
33+
equal( element.autocomplete( "widget" ).parent()[ 0 ],
34+
$( "#autocomplete-wrap1" )[ 0 ], "modified after init" );
2935
element.autocomplete( "destroy" );
3036

3137
element.autocomplete({
3238
appendTo: detached
3339
});
34-
equal( element.autocomplete( "widget" ).parent()[0], detached[0], "detached jQuery object" );
40+
equal( element.autocomplete( "widget" ).parent()[ 0 ], detached[ 0 ],
41+
"detached jQuery object" );
3542
element.autocomplete( "destroy" );
3643

3744
element.autocomplete({
38-
appendTo: detached[0]
45+
appendTo: detached[ 0 ]
3946
});
40-
equal( element.autocomplete( "widget" ).parent()[0], detached[0], "detached DOM element" );
47+
equal( element.autocomplete( "widget" ).parent()[ 0 ], detached[ 0 ],
48+
"detached DOM element" );
4149
element.autocomplete( "destroy" );
4250

4351
element.autocomplete().autocomplete( "option", "appendTo", detached );
44-
equal( element.autocomplete( "widget" ).parent()[0], detached[0], "detached DOM element via option()" );
52+
equal( element.autocomplete( "widget" ).parent()[ 0 ], detached[ 0 ],
53+
"detached DOM element via option()" );
4554
element.autocomplete( "destroy" );
4655
});
4756

@@ -51,8 +60,13 @@ function autoFocusTest( afValue, focusedLength ) {
5160
delay: 0,
5261
source: data,
5362
open: function() {
54-
equal( element.autocomplete( "widget" ).children( ".ui-menu-item:first" ).find( ".ui-state-focus" ).length,
55-
focusedLength, "first item is " + (afValue ? "" : "not") + " auto focused" );
63+
equal(
64+
element.autocomplete( "widget" )
65+
.children( ".ui-menu-item:first" )
66+
.find( ".ui-state-focus" )
67+
.length,
68+
focusedLength,
69+
"first item is " + (afValue ? "" : "not") + " auto focused" );
5670
start();
5771
}
5872
});

0 commit comments

Comments
 (0)