Skip to content
Permalink
Browse files

No need for the try/finally logic in the css tests, won't even work a…

…s expected.
  • Loading branch information
jeresig committed Jan 7, 2010
1 parent 22ff8e9 commit 0bb4fd0af72846467a4c86a47f9cfa72874c21ae
Showing with 43 additions and 49 deletions.
  1. +43 −49 test/unit/css.js
@@ -120,69 +120,63 @@ if(jQuery.browser.msie) {
}

test("css(String, Function)", function() {
try {
expect(3);
expect(3);

var sizes = ["10px", "20px", "30px"];
var sizes = ["10px", "20px", "30px"];

jQuery("<div id='cssFunctionTest'><div class='cssFunction'></div>" +
"<div class='cssFunction'></div>" +
"<div class='cssFunction'></div></div>")
.appendTo("body");
jQuery("<div id='cssFunctionTest'><div class='cssFunction'></div>" +
"<div class='cssFunction'></div>" +
"<div class='cssFunction'></div></div>")
.appendTo("body");

var index = 0;
var index = 0;

jQuery("#cssFunctionTest div").css("font-size", function() {
var size = sizes[index];
index++;
return size;
});

index = 0;

jQuery("#cssFunctionTest div").each(function() {
var computedSize = jQuery(this).css("font-size")
var expectedSize = sizes[index]
equals( computedSize, expectedSize, "Div #" + index + " should be " + expectedSize );
index++;
});
jQuery("#cssFunctionTest div").css("font-size", function() {
var size = sizes[index];
index++;
return size;
});

} finally {
jQuery("#cssFunctionTest").remove();
}
index = 0;

jQuery("#cssFunctionTest div").each(function() {
var computedSize = jQuery(this).css("font-size")
var expectedSize = sizes[index]
equals( computedSize, expectedSize, "Div #" + index + " should be " + expectedSize );
index++;
});

jQuery("#cssFunctionTest").remove();
});

test("css(Object) where values are Functions", function() {
try {
expect(3);
expect(3);

var sizes = ["10px", "20px", "30px"];

jQuery("<div id='cssFunctionTest'><div class='cssFunction'></div>" +
"<div class='cssFunction'></div>" +
"<div class='cssFunction'></div></div>")
.appendTo("body");
var sizes = ["10px", "20px", "30px"];

var index = 0;
jQuery("<div id='cssFunctionTest'><div class='cssFunction'></div>" +
"<div class='cssFunction'></div>" +
"<div class='cssFunction'></div></div>")
.appendTo("body");

var index = 0;

jQuery("#cssFunctionTest div").css({fontSize: function() {
var size = sizes[index];
index++;
return size;
}});
jQuery("#cssFunctionTest div").css({fontSize: function() {
var size = sizes[index];
index++;
return size;
}});

index = 0;
index = 0;

jQuery("#cssFunctionTest div").each(function() {
var computedSize = jQuery(this).css("font-size")
var expectedSize = sizes[index]
equals( computedSize, expectedSize, "Div #" + index + " should be " + expectedSize );
index++;
});
jQuery("#cssFunctionTest div").each(function() {
var computedSize = jQuery(this).css("font-size")
var expectedSize = sizes[index]
equals( computedSize, expectedSize, "Div #" + index + " should be " + expectedSize );
index++;
});

} finally {
jQuery("#cssFunctionTest").remove();
}
jQuery("#cssFunctionTest").remove();
});

test("jQuery.css(elem, 'height') doesn't clear radio buttons (bug #1095)", function () {

0 comments on commit 0bb4fd0

Please sign in to comment.
You can’t perform that action at this time.