Made sure that .remove now uses multiFilter. Fixes bug #4205.
[jquery.git] / test / unit / core.js
index aec334e..9d91c4f 100644 (file)
@@ -615,11 +615,11 @@ test("css(String|Hash)", function() {
 
        equals( jQuery('#main').css("display"), 'none', 'Check for css property "display"');
 
-       ok( jQuery('#foo').is(':visible'), 'Modifying CSS display: Assert element is visible');
-       jQuery('#foo').css({display: 'none'});
-       ok( !jQuery('#foo').is(':visible'), 'Modified CSS display: Assert element is hidden');
-       jQuery('#foo').css({display: 'block'});
-       ok( jQuery('#foo').is(':visible'), 'Modified CSS display: Assert element is visible');
+       ok( jQuery('#nothiddendiv').is(':visible'), 'Modifying CSS display: Assert element is visible');
+       jQuery('#nothiddendiv').css({display: 'none'});
+       ok( !jQuery('#nothiddendiv').is(':visible'), 'Modified CSS display: Assert element is hidden');
+       jQuery('#nothiddendiv').css({display: 'block'});
+       ok( jQuery('#nothiddendiv').is(':visible'), 'Modified CSS display: Assert element is visible');
 
        jQuery('#floatTest').css({styleFloat: 'right'});
        equals( jQuery('#floatTest').css('styleFloat'), 'right', 'Modified CSS float using "styleFloat": Assert float is right');
@@ -642,11 +642,11 @@ test("css(String|Hash)", function() {
 
 test("css(String, Object)", function() {
        expect(21);
-       ok( jQuery('#foo').is(':visible'), 'Modifying CSS display: Assert element is visible');
-       jQuery('#foo').css('display', 'none');
-       ok( !jQuery('#foo').is(':visible'), 'Modified CSS display: Assert element is hidden');
-       jQuery('#foo').css('display', 'block');
-       ok( jQuery('#foo').is(':visible'), 'Modified CSS display: Assert element is visible');
+       ok( jQuery('#nothiddendiv').is(':visible'), 'Modifying CSS display: Assert element is visible');
+       jQuery('#nothiddendiv').css("display", 'none');
+       ok( !jQuery('#nothiddendiv').is(':visible'), 'Modified CSS display: Assert element is hidden');
+       jQuery('#nothiddendiv').css("display", 'block');
+       ok( jQuery('#nothiddendiv').is(':visible'), 'Modified CSS display: Assert element is visible');
 
        jQuery('#floatTest').css('styleFloat', 'left');
        equals( jQuery('#floatTest').css('styleFloat'), 'left', 'Modified CSS float using "styleFloat": Assert float is left');
@@ -694,26 +694,27 @@ test("jQuery.css(elem, 'height') doesn't clear radio buttons (bug #1095)", funct
 });
 
 test("width()", function() {
-       expect(8);
+       expect(6);
 
        var $div = jQuery("#nothiddendiv");
        $div.width(30);
        equals($div.width(), 30, "Test set to 30 correctly");
+       $div.hide();
+       equals($div.width(), 30, "Test hidden div");
+       $div.show();
        $div.width(-1); // handle negative numbers by ignoring #1599
        equals($div.width(), 30, "Test negative width ignored");
        $div.css("padding", "20px");
        equals($div.width(), 30, "Test padding specified with pixels");
        $div.css("border", "2px solid #fff");
        equals($div.width(), 30, "Test border specified with pixels");
-       $div.css("padding", "2em");
-       equals($div.width(), 30, "Test padding specified with ems");
-       $div.css("border", "1em solid #fff");
+       //$div.css("padding", "2em");
+       //equals($div.width(), 30, "Test padding specified with ems");
+       //$div.css("border", "1em solid #fff");
        //DISABLED - Opera 9.6 fails this test, returns 8
        //equals($div.width(), 30, "Test border specified with ems");
-       $div.css("padding", "2%");
-       equals($div.width(), 30, "Test padding specified with percent");
-       $div.hide();
-       equals($div.width(), 30, "Test hidden div");
+       //$div.css("padding", "2%");
+       //equals($div.width(), 30, "Test padding specified with percent");
 
        $div.css({ display: "", border: "", padding: "" });
 
@@ -723,26 +724,27 @@ test("width()", function() {
 });
 
 test("height()", function() {
-       expect(7);
+       expect(5);
 
        var $div = jQuery("#nothiddendiv");
        $div.height(30);
        equals($div.height(), 30, "Test set to 30 correctly");
+       $div.hide();
+       equals($div.height(), 30, "Test hidden div");
+       $div.show();
        $div.height(-1); // handle negative numbers by ignoring #1599
        equals($div.height(), 30, "Test negative height ignored");
        $div.css("padding", "20px");
        equals($div.height(), 30, "Test padding specified with pixels");
        $div.css("border", "2px solid #fff");
        equals($div.height(), 30, "Test border specified with pixels");
-       $div.css("padding", "2em");
-       equals($div.height(), 30, "Test padding specified with ems");
-       $div.css("border", "1em solid #fff");
+       //$div.css("padding", "2em");
+       //equals($div.height(), 30, "Test padding specified with ems");
+       //$div.css("border", "1em solid #fff");
        //DISABLED - Opera 9.6 fails this test, returns 8
        //equals($div.height(), 30, "Test border specified with ems");
-       $div.css("padding", "2%");
-       equals($div.height(), 30, "Test padding specified with percent");
-       $div.hide();
-       equals($div.height(), 30, "Test hidden div");
+       //$div.css("padding", "2%");
+       //equals($div.height(), 30, "Test padding specified with percent");
 
        $div.css({ display: "", border: "", padding: "", height: "1px" });
 });
@@ -840,9 +842,7 @@ test("append(String|Element|Array<Element>|jQuery)", function() {
        equals( expected, jQuery('#sap').text(), "Check for appending of array of elements" );
 
        reset();
-       expected = document.querySelectorAll ?
-               "This link has class=\"blog\": Simon Willison's WeblogYahooTry them out:" :
-               "This link has class=\"blog\": Simon Willison's WeblogTry them out:Yahoo";
+       expected = "This link has class=\"blog\": Simon Willison's WeblogYahooTry them out:";
        jQuery('#sap').append(jQuery("#first, #yahoo"));
        equals( expected, jQuery('#sap').text(), "Check for appending of jQuery object" );
 
@@ -909,7 +909,7 @@ test("append(String|Element|Array<Element>|jQuery)", function() {
 });
 
 test("appendTo(String|Element|Array<Element>|jQuery)", function() {
-       expect(7);
+       expect(12);
        var defaultText = 'Try them out:'
        jQuery('<b>buga</b>').appendTo('#first');
        equals( jQuery("#first").text(), defaultText + 'buga', 'Check if text appending works' );
@@ -929,15 +929,34 @@ test("appendTo(String|Element|Array&lt;Element&gt;|jQuery)", function() {
        ok( jQuery(document.createElement("script")).appendTo("body").length, "Make sure a disconnected script can be appended." );
 
        reset();
-       expected = document.querySelectorAll ?
-               "This link has class=\"blog\": Simon Willison's WeblogYahooTry them out:" :
-               "This link has class=\"blog\": Simon Willison's WeblogTry them out:Yahoo";
+       expected = "This link has class=\"blog\": Simon Willison's WeblogYahooTry them out:";
        jQuery("#first, #yahoo").appendTo('#sap');
        equals( expected, jQuery('#sap').text(), "Check for appending of jQuery object" );
 
        reset();
        jQuery('#select1').appendTo('#foo');
        t( 'Append select', '#foo select', ['select1'] );
+
+       reset();
+       var div = jQuery("<div/>").click(function(){
+               ok(true, "Running a cloned click.");
+       });
+       div.appendTo("#main, #moretests");
+
+       jQuery("#main div:last").click();
+       jQuery("#moretests div:last").click();
+
+       reset();
+       var div = jQuery("<div/>").appendTo("#main, #moretests");
+
+       equals( div.length, 2, "appendTo returns the inserted elements" );
+       
+       div.addClass("test");
+
+       ok( jQuery("#main div:last").hasClass("test"), "appendTo element was modified after the insertion" );
+       ok( jQuery("#moretests div:last").hasClass("test"), "appendTo element was modified after the insertion" );
+
+       reset();
 });
 
 test("prepend(String|Element|Array&lt;Element&gt;|jQuery)", function() {
@@ -958,9 +977,7 @@ test("prepend(String|Element|Array&lt;Element&gt;|jQuery)", function() {
        equals( expected, jQuery('#sap').text(), "Check for prepending of array of elements" );
 
        reset();
-       expected = document.querySelectorAll ?
-               "YahooTry them out:This link has class=\"blog\": Simon Willison's Weblog" :
-               "Try them out:YahooThis link has class=\"blog\": Simon Willison's Weblog";
+       expected = "YahooTry them out:This link has class=\"blog\": Simon Willison's Weblog";
        jQuery('#sap').prepend(jQuery("#first, #yahoo"));
        equals( expected, jQuery('#sap').text(), "Check for prepending of jQuery object" );
 });
@@ -979,12 +996,12 @@ test("prependTo(String|Element|Array&lt;Element&gt;|jQuery)", function() {
 
        reset();
        expected = "Try them out:YahooThis link has class=\"blog\": Simon Willison's Weblog";
-       jQuery([document.getElementById('yahoo'), document.getElementById('first')]).prependTo('#sap');
+       jQuery([document.getElementById('first'), document.getElementById('yahoo')]).prependTo('#sap');
        equals( expected, jQuery('#sap').text(), "Check for prepending of array of elements" );
 
        reset();
-       expected = "Try them out:YahooThis link has class=\"blog\": Simon Willison's Weblog";
-       jQuery("#yahoo, #first").prependTo('#sap');
+       expected = "YahooTry them out:This link has class=\"blog\": Simon Willison's Weblog";
+       jQuery("#first, #yahoo").prependTo('#sap');
        equals( expected, jQuery('#sap').text(), "Check for prepending of jQuery object" );
 
        reset();
@@ -1011,9 +1028,7 @@ test("before(String|Element|Array&lt;Element&gt;|jQuery)", function() {
        equals( expected, jQuery('#en').text(), "Insert array of elements before" );
 
        reset();
-       expected = document.querySelectorAll ?
-               "This is a normal link: diveintomarkTry them out:Yahoo" :
-               "This is a normal link: Try them out:diveintomarkYahoo";
+       expected = "This is a normal link: diveintomarkTry them out:Yahoo";
        jQuery('#yahoo').before(jQuery("#first, #mark"));
        equals( expected, jQuery('#en').text(), "Insert jQuery before" );
 });
@@ -1035,9 +1050,7 @@ test("insertBefore(String|Element|Array&lt;Element&gt;|jQuery)", function() {
        equals( expected, jQuery('#en').text(), "Insert array of elements before" );
 
        reset();
-       expected = document.querySelectorAll ?
-               "This is a normal link: diveintomarkTry them out:Yahoo" :
-               "This is a normal link: Try them out:diveintomarkYahoo";
+       expected = "This is a normal link: diveintomarkTry them out:Yahoo";
        jQuery("#first, #mark").insertBefore('#yahoo');
        equals( expected, jQuery('#en').text(), "Insert jQuery before" );
 });
@@ -1059,9 +1072,7 @@ test("after(String|Element|Array&lt;Element&gt;|jQuery)", function() {
        equals( expected, jQuery('#en').text(), "Insert array of elements after" );
 
        reset();
-       expected = document.querySelectorAll ?
-               "This is a normal link: YahoodiveintomarkTry them out:" :
-               "This is a normal link: YahooTry them out:diveintomark";
+       expected = "This is a normal link: YahoodiveintomarkTry them out:";
        jQuery('#yahoo').after(jQuery("#first, #mark"));
        equals( expected, jQuery('#en').text(), "Insert jQuery after" );
 });
@@ -1079,12 +1090,12 @@ test("insertAfter(String|Element|Array&lt;Element&gt;|jQuery)", function() {
 
        reset();
        expected = "This is a normal link: YahooTry them out:diveintomark";
-       jQuery([document.getElementById('mark'), document.getElementById('first')]).insertAfter('#yahoo');
+       jQuery([document.getElementById('first'), document.getElementById('mark')]).insertAfter('#yahoo');
        equals( expected, jQuery('#en').text(), "Insert array of elements after" );
 
        reset();
-       expected = "This is a normal link: YahooTry them out:diveintomark";
-       jQuery("#mark, #first").insertAfter('#yahoo');
+       expected = "This is a normal link: YahoodiveintomarkTry them out:";
+       jQuery("#first, #mark").insertAfter('#yahoo');
        equals( expected, jQuery('#en').text(), "Insert jQuery after" );
 });
 
@@ -1156,7 +1167,7 @@ test("find(String)", function() {
 });
 
 test("clone()", function() {
-       expect(20);
+       expect(28);
        equals( 'This is a normal link: Yahoo', jQuery('#en').text(), 'Assert text for #en' );
        var clone = jQuery('#yahoo').clone();
        equals( 'Try them out:Yahoo', jQuery('#first').append(clone).text(), 'Check for clone' );
@@ -1176,6 +1187,31 @@ test("clone()", function() {
        // using contents will get comments regular, text, and comment nodes
        var cl = jQuery("#nonnodes").contents().clone();
        ok( cl.length >= 2, "Check node,textnode,comment clone works (some browsers delete comments on clone)" );
+
+       var div = jQuery("<div><ul><li>test</li></ul></div>").click(function(){
+               ok( true, "Bound event still exists." );
+       });
+
+       div = div.clone(true).clone(true);
+       equals( div.length, 1, "One element cloned" );
+       equals( div[0].nodeName.toUpperCase(), "DIV", "DIV element cloned" );
+       div.trigger("click");
+
+       div = jQuery("<div/>").append([ document.createElement("table"), document.createElement("table") ]);
+       div.find("table").click(function(){
+               ok( true, "Bound event still exists." );
+       });
+
+       div = div.clone(true);
+       equals( div.length, 1, "One element cloned" );
+       equals( div[0].nodeName.toUpperCase(), "DIV", "DIV element cloned" );
+       div.find("table:last").trigger("click");
+
+       div = jQuery("<div/>").html('<object height="355" width="425">  <param name="movie" value="http://www.youtube.com/v/JikaHBDoV3k&amp;hl=en">  <param name="wmode" value="transparent"> </object>');
+
+       div = div.clone(true);
+       equals( div.length, 1, "One element cloned" );
+       equals( div[0].nodeName.toUpperCase(), "DIV", "DIV element cloned" );
 });
 
 if (!isLocal) {
@@ -1428,7 +1464,7 @@ test("closest()", function() {
        isSet( jQuery("#main").closest("span,#html").get(), q("html"), "closest(span,#html)" );
 
        isSet( jQuery("div:eq(1)").closest("div:first").get(), [], "closest(div:first)" );
-       isSet( jQuery("div").closest("body:first div:last").get(), q("divWithNoTabIndex"), "closest(body:first div:last)" );
+       isSet( jQuery("div").closest("body:first div:last").get(), q("fx-tests"), "closest(body:first div:last)" );
 });
 
 test("not()", function() {
@@ -1454,7 +1490,7 @@ test("andSelf()", function() {
        isSet( jQuery("#en").siblings().andSelf().get(), q("sndp", "sap","en"), "Check for siblings and self" );
        isSet( jQuery("#foo").children().andSelf().get(), q("sndp", "en", "sap", "foo"), "Check for children and self" );
        isSet( jQuery("#sndp, #en").parent().andSelf().get(), q("foo","sndp","en"), "Check for parent and self" );
-       isSet( jQuery("#groups").parents("p, div").andSelf().get(), q("ap", "main", "groups"), "Check for parents and self" );
+       isSet( jQuery("#groups").parents("p, div").andSelf().get(), q("main", "ap", "groups"), "Check for parents and self" );
 });
 
 test("siblings([String])", function() {
@@ -1462,8 +1498,8 @@ test("siblings([String])", function() {
        isSet( jQuery("#en").siblings().get(), q("sndp", "sap"), "Check for siblings" );
        isSet( jQuery("#sndp").siblings(":has(code)").get(), q("sap"), "Check for filtered siblings (has code child element)" );
        isSet( jQuery("#sndp").siblings(":has(a)").get(), q("en", "sap"), "Check for filtered siblings (has anchor child element)" );
-       isSet( jQuery("#foo").siblings("form, b").get(), q("form", "lengthtest", "name-tests", "testForm", "floatTest"), "Check for multiple filters" );
-       var set = document.querySelectorAll ? q("en", "sap", "sndp") : q("sndp", "sap", "en");
+       isSet( jQuery("#foo").siblings("form, b").get(), q("form", "floatTest", "lengthtest", "name-tests", "testForm"), "Check for multiple filters" );
+       var set = q("en", "sap", "sndp");
        isSet( jQuery("#en, #sndp").siblings().get(), set, "Check for unique results from siblings" );
 });
 
@@ -1488,7 +1524,7 @@ test("parents([String])", function() {
        equals( jQuery("#groups").parents()[0].id, "ap", "Simple parents check" );
        equals( jQuery("#groups").parents("p")[0].id, "ap", "Filtered parents check" );
        equals( jQuery("#groups").parents("div")[0].id, "main", "Filtered parents check2" );
-       isSet( jQuery("#groups").parents("p, div").get(), q("ap", "main"), "Check for multiple filters" );
+       isSet( jQuery("#groups").parents("p, div").get(), q("main", "ap"), "Check for multiple filters" );
        isSet( jQuery("#en, #sndp").parents().get(), q("foo", "main", "dl", "body", "html"), "Check for unique results from parents" );
 });
 
@@ -1510,13 +1546,16 @@ test("prev([String])", function() {
 
 test("show()", function() {
        expect(15);
-       var pass = true, div = jQuery("div");
+       var pass = true, div = jQuery("#main div");
        div.show().each(function(){
                if ( this.style.display == "none" ) pass = false;
        });
        ok( pass, "Show" );
 
        jQuery("#main").append('<div id="show-tests"><div><p><a href="#"></a></p><code></code><pre></pre><span></span></div><table><thead><tr><th></th></tr></thead><tbody><tr><td></td></tr></tbody></table><ul><li></li></ul></div>');
+
+       var old = jQuery("#show-tests table").show().css("display") !== "table";
+
        var test = {
                "div"      : "block",
                "p"        : "block",
@@ -1524,14 +1563,14 @@ test("show()", function() {
                "code"     : "inline",
                "pre"      : "block",
                "span"     : "inline",
-               "table"    : jQuery.browser.msie ? "block" : "table",
-               "thead"    : jQuery.browser.msie ? "block" : "table-header-group",
-               "tbody"    : jQuery.browser.msie ? "block" : "table-row-group",
-               "tr"       : jQuery.browser.msie ? "block" : "table-row",
-               "th"       : jQuery.browser.msie ? "block" : "table-cell",
-               "td"       : jQuery.browser.msie ? "block" : "table-cell",
+               "table"    : old ? "block" : "table",
+               "thead"    : old ? "block" : "table-header-group",
+               "tbody"    : old ? "block" : "table-row-group",
+               "tr"       : old ? "block" : "table-row",
+               "th"       : old ? "block" : "table-cell",
+               "td"       : old ? "block" : "table-cell",
                "ul"       : "block",
-               "li"       : jQuery.browser.msie ? "block" : "list-item"
+               "li"       : old ? "block" : "list-item"
        };
 
        jQuery.each(test, function(selector, expected) {
@@ -1557,7 +1596,7 @@ test("addClass(String)", function() {
 });
 
 test("removeClass(String) - simple", function() {
-       expect(4);
+       expect(5);
        
        var $divs = jQuery('div');
        
@@ -1566,13 +1605,17 @@ test("removeClass(String) - simple", function() {
        ok( !$divs.is('.test'), "Remove Class" );
 
        reset();
-       
+
        $divs.addClass("test").addClass("foo").addClass("bar");
        $divs.removeClass("test").removeClass("bar").removeClass("foo");
        
        ok( !$divs.is('.test,.bar,.foo'), "Remove multiple classes" );
 
        reset();
+
+       // Make sure that a null value doesn't cause problems
+       $divs.eq(0).addClass("test").removeClass(null);
+       ok( $divs.eq(0).is('.test'), "Null value passed to removeClass" );
        
        $divs.eq(0).addClass("test").removeClass("");
        ok( $divs.eq(0).is('.test'), "Empty string passed to removeClass" );
@@ -1670,7 +1713,7 @@ test("jQuery.className", function() {
 });
 
 test("remove()", function() {
-       expect(6);
+       expect(7);
        jQuery("#ap").children().remove();
        ok( jQuery("#ap").text().length > 10, "Check text is not removed" );
        equals( jQuery("#ap").children().length, 0, "Check remove" );
@@ -1680,6 +1723,9 @@ test("remove()", function() {
        ok( jQuery("#ap").text().length > 10, "Check text is not removed" );
        equals( jQuery("#ap").children().length, 1, "Check filtered remove" );
 
+       jQuery("#ap").children().remove("a, code");
+       equals( jQuery("#ap").children().length, 0, "Check multi-filtered remove" );
+
        // using contents will get comments regular, text, and comment nodes
        equals( jQuery("#nonnodes").contents().length, 3, "Check node,textnode,comment remove works" );
        jQuery("#nonnodes").contents().remove();