Skip to content
Permalink
Browse files

Handle two more cases where comments were missing in the test case, i…

…n Blackberry 4.6.
  • Loading branch information...
jeresig committed Aug 27, 2010
1 parent ea8b158 commit 43fbe3b33aeaa73c151ddb59632c85bc7f8a87b9
Showing with 7 additions and 1 deletion.
  1. +7 −1 test/unit/manipulation.js
@@ -984,6 +984,11 @@ test("html(Function) with incoming value", function() {
equals( val, old[i], "Make sure the incoming value is correct." );
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();
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" );

// 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]();
equals( jQuery("#nonnodes").contents().length, 0, "Check node,textnode,comment remove works" );

0 comments on commit 43fbe3b

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