deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Handle two more cases where comments were missing in the test case, in Blackberry 4.6.

Browse source
This commit is contained in:
John Resig 2010-08-27 14:16:27 -04:00
parent ea8b1584b1
commit 43fbe3b33a
1 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
test/unit/manipulation.js
View file

@ -984,6 +984,11 @@ test("html(Function) with incoming value", function() {
equals( val, old[i], "Make sure the incoming value is correct." ); equals( val, old[i], "Make sure the incoming value is correct." );
return "<b>bold</b>"; return "<b>bold</b>";
}); });
// Handle the case where no comment is in the document
if ( j.length === 2 ) {
equals( null, null, "Make sure the incoming value is correct." );
}
j.find('b').removeData(); j.find('b').removeData();
equals( j.html().replace(/ xmlns="[^"]+"/g, "").toLowerCase(), "<b>bold</b>", "Check node,textnode,comment with html()" ); equals( j.html().replace(/ xmlns="[^"]+"/g, "").toLowerCase(), "<b>bold</b>", "Check node,textnode,comment with html()" );
@ -1038,7 +1043,8 @@ var testRemove = function(method) {
equals( jQuery("#ap").children().length, 0, "Check multi-filtered remove" ); equals( jQuery("#ap").children().length, 0, "Check multi-filtered remove" );
// using contents will get comments regular, text, and comment nodes // using contents will get comments regular, text, and comment nodes
equals( jQuery("#nonnodes").contents().length, 3, "Check node,textnode,comment remove works" ); // Handle the case where no comment is in the document
ok( jQuery("#nonnodes").contents().length >= 2, "Check node,textnode,comment remove works" );
jQuery("#nonnodes").contents()[method](); jQuery("#nonnodes").contents()[method]();
equals( jQuery("#nonnodes").contents().length, 0, "Check node,textnode,comment remove works" ); equals( jQuery("#nonnodes").contents().length, 0, "Check node,textnode,comment remove works" );