jquery core: Simplifying isEmptyObject() and adding tests.

flesler · flesler · commit a38a5cd531a3 · 2009-07-16T15:16:44.000Z
diff --git a/src/core.js b/src/core.js
@@ -292,9 +292,9 @@ jQuery.extend({
 	},
 
 	isEmptyObject: function( obj ) {
-		var name = "";
-		for(name in obj) break;
-		return !name;
+		for(var name in obj)
+			return false;
+		return true;
 	},
 
 	// check if an element is in a (or is an) XML document
diff --git a/test/unit/core.js b/test/unit/core.js
@@ -599,3 +599,13 @@ test("jQuery.makeArray", function(){
 
 	ok( jQuery.makeArray(document.getElementById('form')).length >= 13, "Pass makeArray a form (treat as elements)" );
 });
+
+test("jQuery.isEmptyObject", function(){
+	expect(2);
+	
+	equals(true, jQuery.isEmptyObject({}), "isEmptyObject on empty object literal" );
+	equals(false, jQuery.isEmptyObject({a:1}), "isEmptyObject on non-empty object literal" );
+	
+	// What about this ?
+	// equals(true, jQuery.isEmptyObject(null), "isEmptyObject on null" );
+});
