Permalink
Browse files

Tests: fix tests in accordance with new :visible behavior

  • Loading branch information...
timmywil committed May 12, 2015
1 parent 79bcb29 commit 16713fb6097849727de2ba1b489c98e242a168c3
Showing with 7 additions and 5 deletions.
  1. +6 −4 test/unit/css.js
  2. +1 −1 test/unit/effects.js
View
@@ -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>");
View
@@ -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() {

0 comments on commit 16713fb

Please sign in to comment.