Tests: fix tests in accordance with new :visible behavior

diff --git a/test/unit/css.js b/test/unit/css.js
index a673b4e51ff58fa02c23d6d8496ced6dcf9e97db..5a9849c019ba157a756aaa5648bc8287a12151ce 100644 (file)
--- a/test/unit/css.js
+++ b/test/unit/css.js
@@ -982,9 +982,9 @@ test( "css opacity consistency across browsers (#12685)", function() {
 });
 
 test( ":visible/:hidden selectors", function() {
-       expect( 18 );
+       expect( 17 );
 
-       var $div, $br, $table, $a;
+       var $div, $table, $a;
 
        ok( jQuery("#nothiddendiv").is(":visible"), "Modifying CSS display: Assert element is visible" );
        jQuery("#nothiddendiv").css({ display: "none" });
@@ -1006,8 +1006,10 @@ test( ":visible/:hidden selectors", function() {
        $div.css( { width: 0, height: 0, overflow: "hidden" } );
        ok( $div.is( ":visible" ), "Div with width and height of 0 is still visible (gh-2227)" );
 
-       $br = jQuery( "<br/>" ).appendTo( "#qunit-fixture" );
-       ok( $br.is( ":visible" ), "br element is visible" );
+       // Safari 6-7 and iOS 6-7 report 0 width for br elements
+       // When newer browsers propagate, re-enable this test
+       // $br = jQuery( "<br/>" ).appendTo( "#qunit-fixture" );
+       // ok( $br.is( ":visible" ), "br element is visible" );
 
        $table = jQuery("#table");
        $table.html("<tr><td style='display:none'>cell</td><td>cell</td></tr>");

diff --git a/test/unit/effects.js b/test/unit/effects.js
index c444ee8f66d4886a33a02ac7c5ac222e043dc60a..86646ec802cf986da137277171950b0ba17edd10 100644 (file)
--- a/test/unit/effects.js
+++ b/test/unit/effects.js
@@ -25,7 +25,7 @@ module("effects", {
 
 test("sanity check", function() {
        expect(1);
-       equal( jQuery("#dl:visible, #qunit-fixture:visible, #foo:visible").length, 2, "QUnit state is correct for testing effects" );
+       equal( jQuery("#dl:visible, #qunit-fixture:visible, #foo:visible").length, 3, "QUnit state is correct for testing effects" );
 });
 
 test("show() basic", 1, function() {