From 8df348e9eb3681ff5f09b1e52b72573e2eee3105 Mon Sep 17 00:00:00 2001 From: Richard Gibson Date: Tue, 15 Oct 2013 15:48:53 -0400 Subject: [PATCH] No ticket: Test cleanup --- test/unit/css.js | 30 +++++++++--------------------- 1 file changed, 9 insertions(+), 21 deletions(-) diff --git a/test/unit/css.js b/test/unit/css.js index cd70adbfc..2c9fae0e7 100644 --- a/test/unit/css.js +++ b/test/unit/css.js @@ -3,7 +3,7 @@ if ( jQuery.css ) { module("css", { teardown: moduleTeardown }); test("css(String|Hash)", function() { - expect( 40 ); + expect( 42 ); equal( jQuery("#qunit-fixture").css("display"), "block", "Check for css property \"display\"" ); @@ -13,7 +13,7 @@ test("css(String|Hash)", function() { notEqual( $child.css("width"), "20px", "Retrieving a width percentage on the child of a hidden div returns percentage" ); notEqual( $child.css("height"), "20px", "Retrieving a height percentage on the child of a hidden div returns percentage" ); - div = jQuery( "
" ); + div = jQuery( "
" ); // These should be "auto" (or some better value) // temporarily provide "0px" for backwards compat @@ -42,7 +42,7 @@ test("css(String|Hash)", function() { equal( parseFloat(jQuery("#nothiddendiv").css("width")), 0, "Test negative width set to 0"); equal( parseFloat(jQuery("#nothiddendiv").css("height")), 0, "Test negative height set to 0"); - equal( jQuery("
").css("display"), "none", "Styles on disconnected nodes"); + equal( jQuery("
").css("display"), "none", "Styles on disconnected nodes"); jQuery("#floatTest").css({"float": "right"}); equal( jQuery("#floatTest").css("float"), "right", "Modified CSS float using \"float\": Assert float is right"); @@ -108,6 +108,12 @@ test("css(String|Hash)", function() { // Test null child.css("font-size", null); equal( child[0].style.fontSize, old, "Make sure font-size isn't changed on null." ); + + strictEqual( child.css( "x-fake" ), undefined, "Make sure undefined is returned from css(nonexistent)." ); + + div = jQuery( "
" ).css({ position: "absolute", "z-index": 1000 }).appendTo( "#qunit-fixture" ); + strictEqual( div.css( "z-index" ), "1000", + "Make sure that a string z-index is returned from css('z-index') (#14432)." ); }); test( "css() explicit and relative values", 29, function() { @@ -197,16 +203,6 @@ test( "css() explicit and relative values", 29, function() { equal( $elem.css("opacity"), "1", "'+=0.5' on opacity (params)" ); }); -test("css(String) where values are z-index", function() { - expect(1); - - var $elem = jQuery( "
" ).appendTo( "#qunit-fixture" ); - - $elem.css({ "position": "absolute", "z-index": "1000" }); - strictEqual( $elem.css( "z-index" ), "1000" ); -}); - - test("css(String, Object)", function() { expect( 19 ); var j, div, display, ret, success; @@ -363,14 +359,6 @@ test("css(Object) where values are Functions", function() { jQuery("#cssFunctionTest").remove(); }); -test("css(String) where values are undefined", function() { - expect(1); - - var $elem = jQuery( "#nothiddendiv" ); - - strictEqual( $elem.css( "test" ), undefined ); -}); - test("css(Object) where values are Functions with incoming values", function() { expect(3); -- 2.39.5