From 2a23650a19ed4db2d031697638bdd71c06f44492 Mon Sep 17 00:00:00 2001 From: rwldrn Date: Wed, 10 Nov 2010 10:23:48 -0500 Subject: [PATCH] Updating #7397 unit tests to correctly test for a persisted display value --- test/unit/effects.js | 26 +++++++++++++++----------- 1 file changed, 15 insertions(+), 11 deletions(-) diff --git a/test/unit/effects.js b/test/unit/effects.js index aca9263..9690796 100644 --- a/test/unit/effects.js +++ b/test/unit/effects.js @@ -134,30 +134,34 @@ test("show(Number) - other displays", function() { // Supports #7397 test("Persist correct display value", function() { - expect(4); + expect(3); QUnit.reset(); stop(); // #show-tests * is set display: none in CSS - jQuery("#main").append('
'); + jQuery("#main").append('
foo
'); var $span = jQuery("#show-tests span"), - orig = $span.css("display"), - num = 0; + displayNone = $span.css("display"), + display = '', num = 0; - equal(orig, "none", "Expecting to start at display: none"); + $span.show(); + + display = $span.css("display"); + + $span.hide(); - $span.text('Saving...').fadeIn(100, function() { + $span.fadeIn(100, function() { - equal($span.css("display"), "block", "Expecting display: block"); + equals($span.css("display"), display, "Expecting display: " + display); - $span.text('Saved!').fadeOut(100, function () { + $span.fadeOut(100, function () { - equal($span.css("display"), "none", "Expecting display: none"); + equals($span.css("display"), displayNone, "Expecting display: " + displayNone); - $span.text('Saving...').fadeIn(100, function() { + $span.fadeIn(100, function() { - equal($span.css("display"), "block", "Expecting display: block"); + equals($span.css("display"), display, "Expecting display: " + display); start(); }); -- 1.7.10.4