Tests: fix tests in accordance with new :visible behavior

diff --git a/test/unit/css.js b/test/unit/css.js
index a3be0d405efb355f01a6213096e0bbfec78e55ae..4d0e11b3b812c4f333065d63a9ce96717a2f1a14 100644 (file)
--- a/test/unit/css.js
+++ b/test/unit/css.js
@@ -924,9 +924,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" });
@@ -948,8 +948,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 f23342c5600b8a2563843060d717e373a6912bbe..571fda0316d30fe89bb98df71d39bfe9d7fbb5b5 100644 (file)
--- a/test/unit/effects.js
+++ b/test/unit/effects.js
@@ -27,7 +27,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() {