Permalink
Browse files

More unit tests and a $.hasData that works for JS objects too.

  • Loading branch information...
csnover committed Dec 22, 2010
1 parent f5d4bf8 commit e199ead4cba3687beca2444eea4bb8abc20f55b6
Showing with 17 additions and 7 deletions.
  1. +5 −1 src/data.js
  2. +12 −6 test/unit/data.js
@@ -22,7 +22,11 @@ jQuery.extend({
},

hasData: function( elem ) {
return !elem.nodeType || (!!elem[ jQuery.expando ] && !jQuery.isEmptyObject(jQuery.cache[ elem[jQuery.expando] ]));
if (elem.nodeType) {
elem = jQuery.cache[ elem[jQuery.expando] ];
}

return !!elem && !jQuery.isEmptyObject(elem);
},

data: function( elem, name, data ) {
@@ -79,12 +79,18 @@ test("jQuery.data", function() {
});

test("jQuery.hasData", function() {
var div = document.createElement( "div" );
equals( jQuery.hasData(div), false, "No data exists" );
jQuery.data( div, "foo", "bar" );
equals( jQuery.hasData(div), true, "Data exists" );
jQuery.removeData( div, "foo" );
equals( jQuery.hasData(div), false, "Data was removed" );
expect(6);

function testData(obj) {
equals( jQuery.hasData(obj), false, "No data exists" );
jQuery.data( obj, "foo", "bar" );
equals( jQuery.hasData(obj), true, "Data exists" );
jQuery.removeData( obj, "foo" );
equals( jQuery.hasData(obj), false, "Data was removed" );
}

testData(document.createElement('div'));
testData({});
});

test(".data()", function() {

0 comments on commit e199ead

Please sign in to comment.