Skip to content
Permalink
Browse files

Add some more tests to make sure that pulling in data- properties on …

…an element with no data set, works.
  • Loading branch information...
jeresig committed Oct 17, 2010
1 parent 97dfa0d commit 008e971943b0ce55333a65caea7f64eb82b985ac
Showing with 11 additions and 3 deletions.
  1. +11 −3 test/unit/data.js
@@ -184,9 +184,10 @@ test(".data(String) and .data(String, Object)", function() {
});

test("data-* attributes", function() {
expect(33);
expect(37);
var div = jQuery("<div>"),
child = jQuery("<div data-myobj='old data' data-ignored=\"DOM\" data-other='test'></div>");
child = jQuery("<div data-myobj='old data' data-ignored=\"DOM\" data-other='test'></div>"),
dummy = jQuery("<div data-myobj='old data' data-ignored=\"DOM\" data-other='test'></div>");

equals( div.data("attr"), undefined, "Check for non-existing data-attr attribute" );

@@ -208,10 +209,11 @@ test("data-* attributes", function() {
child.data("ignored", "cache");
equals( child.data("ignored"), "cache", "Cached data used before DOM data-* fallback");

var obj = child.data(), check = [ "myobj", "ignored", "other" ], num = 0;
var obj = child.data(), obj2 = dummy.data(), check = [ "myobj", "ignored", "other" ], num = 0, num2 = 0;

for ( var i = 0, l = check.length; i < l; i++ ) {
ok( obj[ check[i] ], "Make sure data- property exists when calling data-." );
ok( obj2[ check[i] ], "Make sure data- property exists when calling data-." );
}

for ( var prop in obj ) {
@@ -220,6 +222,12 @@ test("data-* attributes", function() {

equals( num, check.length, "Make sure that the right number of properties came through." );

for ( var prop in obj2 ) {
num2++;
}

equals( num2, check.length, "Make sure that the right number of properties came through." );

child.attr("data-other", "newvalue");

equals( child.data("other"), "test", "Make sure value was pulled in properly from a .data()." );

0 comments on commit 008e971

Please sign in to comment.
You can’t perform that action at this time.