Renamed isObjectLiteral to isPlainObject (makes more sense, per the suggestion of...
[jquery.git] / test / unit / core.js
index 3a3b42f..df610eb 100644 (file)
@@ -29,7 +29,7 @@ test("jQuery()", function() {
 
 
        var main = jQuery("#main");
-       isSet( jQuery("div p", main).get(), q("sndp", "en", "sap"), "Basic selector with jQuery object as context" );
+       same( jQuery("div p", main).get(), q("sndp", "en", "sap"), "Basic selector with jQuery object as context" );
 
 /*
        // disabled since this test was doing nothing. i tried to fix it but i'm not sure
@@ -189,6 +189,62 @@ test("noConflict", function() {
        jQuery = $$;
 });
 
+test("trim", function() {
+  expect(4);
+
+  var nbsp = String.fromCharCode(160);
+
+  equals( jQuery.trim("hello  "), "hello", "trailing space" );
+  equals( jQuery.trim("  hello"), "hello", "leading space" );
+  equals( jQuery.trim("  hello   "), "hello", "space on both sides" );
+  equals( jQuery.trim("  " + nbsp + "hello  " + nbsp + " "), "hello", " " );
+});
+
+test("isPlainObject", function() {
+       expect(7);
+
+       stop();
+
+       // The use case that we want to match
+       ok(jQuery.isPlainObject({}), "{}");
+       // Instantiated objects shouldn't be matched
+       ok(!jQuery.isPlainObject(new Date), "new Date");
+       var fn = function(){};
+       // Functions shouldn't be matched
+       ok(!jQuery.isPlainObject(fn), "fn");
+       // Again, instantiated objects shouldn't be matched
+       ok(!jQuery.isPlainObject(new fn), "new fn (no methods)");
+       // Makes the function a little more realistic
+       // (and harder to detect, incidentally)
+       fn.prototype = {someMethod: function(){}};
+       // Again, instantiated objects shouldn't be matched
+       ok(!jQuery.isPlainObject(new fn), "new fn");
+
+       // DOM Element
+       ok(!jQuery.isPlainObject(document.createElement("div")), "DOM Element");
+       var iframe = document.createElement("iframe");
+       document.body.appendChild(iframe);
+
+       window.iframeDone = function(otherObject){
+               // Objects from other windows should be matched
+               ok(jQuery.isPlainObject(new otherObject), "new otherObject");
+               document.body.removeChild( iframe );
+               start();
+       };
+       var doc = iframe.contentDocument || iframe.contentWindow.document;
+       doc.open();
+       doc.write("<body onload='window.top.iframeDone(Object);'>");
+       doc.close();
+});
+
 test("isFunction", function() {
        expect(19);
 
@@ -310,7 +366,7 @@ test("isXMLDoc - XML", function() {
 }
 
 test("jQuery('html')", function() {
-       expect(13);
+       expect(15);
 
        reset();
        jQuery.foo = false;
@@ -339,6 +395,9 @@ test("jQuery('html')", function() {
        ok( j.length >= 2, "Check node,textnode,comment creation (some browsers delete comments)" );
 
        ok( !jQuery("<option>test</option>")[0].selected, "Make sure that options are auto-selected #2050" );
+
+       ok( jQuery("<div></div>")[0], "Create a div with closing tag." );
+       ok( jQuery("<table></table>")[0], "Create a table with closing tag." );
 });
 
 test("jQuery('html', context)", function() {
@@ -376,12 +435,12 @@ test("size()", function() {
 
 test("get()", function() {
        expect(1);
-       isSet( jQuery("p").get(), q("firstp","ap","sndp","en","sap","first"), "Get All Elements" );
+       same( jQuery("p").get(), q("firstp","ap","sndp","en","sap","first"), "Get All Elements" );
 });
 
 test("toArray()", function() {
        expect(1);
-       isSet ( jQuery("p").toArray(),
+       same( jQuery("p").toArray(),
                q("firstp","ap","sndp","en","sap","first"),
                "Convert jQuery object to an Array" )
 })
@@ -400,8 +459,8 @@ test("get(-Number)",function() {
 
 test("add(String|Element|Array|undefined)", function() {
        expect(16);
-       isSet( jQuery("#sndp").add("#en").add("#sap").get(), q("sndp", "en", "sap"), "Check elements from document" );
-       isSet( jQuery("#sndp").add( jQuery("#en")[0] ).add( jQuery("#sap") ).get(), q("sndp", "en", "sap"), "Check elements from document" );
+       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" );
 
        // For the time being, we're discontinuing support for jQuery(form.elements) since it's ambiguous in IE
@@ -436,6 +495,18 @@ test("add(String|Element|Array|undefined)", function() {
        ok( jQuery([]).add( document.getElementById('form') ).length >= 13, "Add a form (adds the elements)" );
 });
 
+test("add(String, Context)", function() {
+       expect(6);
+
+       equals( jQuery(document).add("#form").length, 2, "Make sure that using regular context document still works." );
+       equals( jQuery(document.body).add("#form").length, 2, "Using a body context." );
+       equals( jQuery(document.body).add("#html").length, 1, "Using a body context." );
+
+       equals( jQuery(document).add("#form", document).length, 2, "Use a passed in document context." );
+       equals( jQuery(document).add("#form", document.body).length, 2, "Use a passed in body context." );
+       equals( jQuery(document).add("#html", document.body).length, 1, "Use a passed in body context." );
+});
+
 test("each(Function)", function() {
        expect(1);
        var div = jQuery("div");
@@ -485,7 +556,7 @@ test("index(Object|String|undefined)", function() {
 });
 
 test("jQuery.merge()", function() {
-       expect(6);
+       expect(8);
 
        var parse = jQuery.merge;
 
@@ -499,10 +570,14 @@ test("jQuery.merge()", function() {
 
        // Fixed at [5998], #3641
        same( parse([-2,-1], [0,1,2]), [-2,-1,0,1,2], "Second array including a zero (falsy)");
+       
+       // After fixing #5527
+       same( parse([], [null, undefined]), [null, undefined], "Second array including null and undefined values");
+       same( parse({length:0}, [1,2]), {length:2, 0:1, 1:2}, "First array like");
 });
 
 test("jQuery.extend(Object, Object)", function() {
-       expect(23);
+       expect(25);
 
        var settings = { xnumber1: 5, xnumber2: 7, xstring1: "peter", xstring2: "pan" },
                options = { xnumber2: 1, xstring2: "x", xxx: "newstring" },
@@ -515,33 +590,43 @@ test("jQuery.extend(Object, Object)", function() {
                deepmerged = { foo: { bar: true, baz: true }, foo2: document };
 
        jQuery.extend(settings, options);
-       isObj( settings, merged, "Check if extended: settings must be extended" );
-       isObj( options, optionsCopy, "Check if not modified: options must not be modified" );
+       same( settings, merged, "Check if extended: settings must be extended" );
+       same( options, optionsCopy, "Check if not modified: options must not be modified" );
 
        jQuery.extend(settings, null, options);
-       isObj( settings, merged, "Check if extended: settings must be extended" );
-       isObj( options, optionsCopy, "Check if not modified: options must not be modified" );
+       same( settings, merged, "Check if extended: settings must be extended" );
+       same( options, optionsCopy, "Check if not modified: options must not be modified" );
 
        jQuery.extend(true, deep1, deep2);
-       isObj( deep1.foo, deepmerged.foo, "Check if foo: settings must be extended" );
-       isObj( deep2.foo, deep2copy.foo, "Check if not deep2: options must not be modified" );
+       same( deep1.foo, deepmerged.foo, "Check if foo: settings must be extended" );
+       same( deep2.foo, deep2copy.foo, "Check if not deep2: options must not be modified" );
        equals( deep1.foo2, document, "Make sure that a deep clone was not attempted on the document" );
 
        var empty = {};
        var optionsWithLength = { foo: { length: -1 } };
        jQuery.extend(true, empty, optionsWithLength);
-       isObj( empty.foo, optionsWithLength.foo, "The length property must copy correctly" );
+       same( empty.foo, optionsWithLength.foo, "The length property must copy correctly" );
 
        empty = {};
        var optionsWithDate = { foo: { date: new Date } };
        jQuery.extend(true, empty, optionsWithDate);
-       isObj( empty.foo, optionsWithDate.foo, "Dates copy correctly" );
+       same( empty.foo, optionsWithDate.foo, "Dates copy correctly" );
 
        var myKlass = function() {};
+       var customObject = new myKlass();
+       var optionsWithCustomObject = { foo: { date: customObject } };
        empty = {};
-       var optionsWithCustomObject = { foo: { date: new myKlass } };
        jQuery.extend(true, empty, optionsWithCustomObject);
-       isObj( empty.foo, optionsWithCustomObject.foo, "Custom objects copy correctly" );
+       ok( empty.foo && empty.foo.date === customObject, "Custom objects copy correctly (no methods)" );
+       
+       // Makes the class a little more realistic
+       myKlass.prototype = { someMethod: function(){} };
+       empty = {};
+       jQuery.extend(true, empty, optionsWithCustomObject);
+       ok( empty.foo && empty.foo.date === customObject, "Custom objects copy correctly" );
+       
+       var ret = jQuery.extend(true, { foo: 4 }, { foo: new Number(5) } );
+       ok( ret.foo == 5, "Wrapped numbers copy correctly" );
 
        var nullUndef;
        nullUndef = jQuery.extend({}, options, { xnumber2: null });
@@ -556,7 +641,7 @@ test("jQuery.extend(Object, Object)", function() {
        var target = {};
        var recursive = { foo:target, bar:5 };
        jQuery.extend(true, target, recursive);
-       isObj( target, { bar:5 }, "Check to make sure a recursive obj doesn't go never-ending loop by not copying it over" );
+       same( target, { bar:5 }, "Check to make sure a recursive obj doesn't go never-ending loop by not copying it over" );
 
        var ret = jQuery.extend(true, { foo: [] }, { foo: [0] } ); // 1907
        equals( ret.foo.length, 1, "Check to make sure a value with coersion 'false' copies over when necessary to fix #1907" );
@@ -584,10 +669,10 @@ test("jQuery.extend(Object, Object)", function() {
                merged2 = { xnumber1: 5, xnumber2: 1, xstring1: "peter", xstring2: "xx", xxx: "newstringx" };
 
        var settings = jQuery.extend({}, defaults, options1, options2);
-       isObj( settings, merged2, "Check if extended: settings must be extended" );
-       isObj( defaults, defaultsCopy, "Check if not modified: options1 must not be modified" );
-       isObj( options1, options1Copy, "Check if not modified: options1 must not be modified" );
-       isObj( options2, options2Copy, "Check if not modified: options2 must not be modified" );
+       same( settings, merged2, "Check if extended: settings must be extended" );
+       same( defaults, defaultsCopy, "Check if not modified: options1 must not be modified" );
+       same( options1, options1Copy, "Check if not modified: options1 must not be modified" );
+       same( options2, options2Copy, "Check if not modified: options2 must not be modified" );
 });
 
 test("jQuery.each(Object,Function)", function() {