Make work wrapInner(fn) work consistently. Fixes #5799.
[jquery.git] / test / unit / manipulation.js
index b0a1c71..bbf9a3a 100644 (file)
@@ -140,22 +140,29 @@ test("wrapAll(String|Element)", function() {
 });
 
 var testWrapInner = function(val) {
-       expect(8);
+       expect(11);
        var num = jQuery("#first").children().length;
-       var result = jQuery('#first').wrapInner('<div class="red"><div id="tmp"></div></div>');
+       var result = jQuery('#first').wrapInner(val('<div class="red"><div id="tmp"></div></div>'));
+       equals( jQuery("#first").children().length, 1, "Only one child" );
+       ok( jQuery("#first").children().is(".red"), "Verify Right Element" );
+       equals( jQuery("#first").children().children().children().length, num, "Verify Elements Intact" );
+
+       reset();
+       var num = jQuery("#first").html("foo<div>test</div><div>test2</div>").children().length;
+       var result = jQuery('#first').wrapInner(val('<div class="red"><div id="tmp"></div></div>'));
        equals( jQuery("#first").children().length, 1, "Only one child" );
        ok( jQuery("#first").children().is(".red"), "Verify Right Element" );
        equals( jQuery("#first").children().children().children().length, num, "Verify Elements Intact" );
 
        reset();
        var num = jQuery("#first").children().length;
-       var result = jQuery('#first').wrapInner(document.getElementById('empty'));
+       var result = jQuery('#first').wrapInner(val(document.getElementById('empty')));
        equals( jQuery("#first").children().length, 1, "Only one child" );
        ok( jQuery("#first").children().is("#empty"), "Verify Right Element" );
        equals( jQuery("#first").children().children().length, num, "Verify Elements Intact" );
 
        var div = jQuery("<div/>");
-       div.wrapInner("<span></span>");
+       div.wrapInner(val("<span></span>"));
        equals(div.children().length, 1, "The contents were wrapped.");
        equals(div.children()[0].nodeName.toLowerCase(), "span", "A span was inserted.");
 }
@@ -164,10 +171,9 @@ test("wrapInner(String|Element)", function() {
        testWrapInner(bareObj);
 });
 
-// TODO: wrapInner uses wrapAll -- get wrapAll working with Function
-// test("wrapInner(Function)", function() {
-//     testWrapInner(functionReturningObj)
-// })
+test("wrapInner(Function)", function() {
+       testWrapInner(functionReturningObj)
+});
 
 test("unwrap()", function() {
        expect(9);