]> source.dussan.org Git - jquery.git/commitdiff
Disabling .add(form.elements) unit test.
authorJohn Resig <jeresig@gmail.com>
Fri, 22 Apr 2011 04:51:23 +0000 (00:51 -0400)
committerJohn Resig <jeresig@gmail.com>
Fri, 22 Apr 2011 04:51:23 +0000 (00:51 -0400)
test/unit/traversing.js

index b5130dbf71fd27c367f36dc93967e6029c45b4cc..5216ae7526ea02c956bed374bda1c5f11b8a428e 100644 (file)
@@ -480,10 +480,15 @@ test("contents()", function() {
 });
 
 test("add(String|Element|Array|undefined)", function() {
-       expect(17);
+       expect(16);
        same( jQuery("#sndp").add("#en").add("#sap").get(), q("sndp", "en", "sap"), "Check elements from document" );
        same( jQuery("#sndp").add( jQuery("#en")[0] ).add( jQuery("#sap") ).get(), q("sndp", "en", "sap"), "Check elements from document" );
-       ok( jQuery([]).add(jQuery("#form")[0].elements).length >= 13, "Check elements from array" );
+
+       // We no longer support .add(form.elements), unfortunately.
+       // There is no way, in browsers, to reliably determine the difference
+       // between form.elements and form - and doing .add(form) and having it
+       // add the form elements is way to unexpected, so this gets the boot.
+       // ok( jQuery([]).add(jQuery("#form")[0].elements).length >= 13, "Check elements from array" );
 
        // For the time being, we're discontinuing support for jQuery(form.elements) since it's ambiguous in IE
        // use jQuery([]).add(form.elements) instead.