Updating #7397 unit tests to correctly test for a persisted display value
authorrwldrn <waldron.rick@gmail.com>
Wed, 10 Nov 2010 15:23:48 +0000 (10:23 -0500)
committerrwldrn <waldron.rick@gmail.com>
Wed, 10 Nov 2010 15:23:48 +0000 (10:23 -0500)
test/unit/effects.js

index aca9263..9690796 100644 (file)
@@ -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('<div id="show-tests"><span style="position:absolute;"></span></div>');
+       jQuery("#main").append('<div id="show-tests"><span style="position:absolute;">foo</span></div>');
   
        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();
       });