testrunner: fixing tests that were broken due to recent updates to qunit.
authorAriel Flesler <aflesler@gmail.com>
Sat, 18 Oct 2008 21:59:09 +0000 (21:59 +0000)
committerAriel Flesler <aflesler@gmail.com>
Sat, 18 Oct 2008 21:59:09 +0000 (21:59 +0000)
test/unit/core.js

index 002b130..686899e 100644 (file)
@@ -1361,25 +1361,24 @@ test("addClass(String)", function() {
 \r
 test("removeClass(String) - simple", function() {\r
        expect(4);\r
-       var div = jQuery("div").addClass("test").removeClass("test"),\r
-               pass = true;\r
-       for ( var i = 0; i < div.size(); i++ ) {\r
-               if ( div.get(i).className.indexOf("test") != -1 ) pass = false;\r
-       }\r
-       ok( pass, "Remove Class" );\r
+       \r
+       var $divs = jQuery('div');\r
+       \r
+       $divs.addClass("test").removeClass("test");\r
+               \r
+       ok( !$divs.is('.test'), "Remove Class" );\r
 \r
        reset();\r
-       var div = jQuery("div").addClass("test").addClass("foo").addClass("bar");\r
-       div.removeClass("test").removeClass("bar").removeClass("foo");\r
-       var pass = true;\r
-       for ( var i = 0; i < div.size(); i++ ) {\r
-        if ( div.get(i).className.match(/test|bar|foo/) ) pass = false;\r
-       }\r
-       ok( pass, "Remove multiple classes" );\r
+       \r
+       $divs.addClass("test").addClass("foo").addClass("bar");\r
+       $divs.removeClass("test").removeClass("bar").removeClass("foo");\r
+       \r
+       ok( !$divs.is('.test,.bar,.foo'), "Remove multiple classes" );\r
 \r
        reset();\r
-       var div = jQuery("div:eq(0)").addClass("test").removeClass("");\r
-       ok( div.is('.test'), "Empty string passed to removeClass" );\r
+       \r
+       $divs.eq(0).addClass("test").removeClass("");\r
+       ok( $divs.eq(0).is('.test'), "Empty string passed to removeClass" );\r
 \r
        // using contents will get regular, text, and comment nodes\r
        var j = jQuery("#nonnodes").contents();\r