Accordion: Rewrote the resize method test; fails in IE8, probably related to :visible issue

jzaefferer · jzaefferer · commit 7d1c9cd3484b · 2010-04-03T17:21:31.000+02:00
diff --git a/tests/unit/accordion/accordion.html b/tests/unit/accordion/accordion.html
@@ -39,7 +39,7 @@
 	
 	<style>
 		#main { font-size: 10pt; font-family: 'trebuchet ms', verdana, arial; }
-		#navigation * { margin: 0; padding: 0; font-size: 12px; }
+		#list1 *, #navigation * { margin: 0; padding: 0; font-size: 12px; }
 	</style>
 </head>
 <body>
diff --git a/tests/unit/accordion/accordion_methods.js b/tests/unit/accordion/accordion_methods.js
@@ -114,7 +114,9 @@ test("activate, jQuery or DOM element", function() {
 });
 
 test("resize", function() {
-	var expected = $('#list1').accordion();
+	var expected = $('#list1').accordion({
+		fillSpace: true
+	});
 	
 	var sizes = [];
 	expected.find(".ui-accordion-content").each(function() {
@@ -128,10 +130,15 @@ test("resize", function() {
 	expected.find(".ui-accordion-content").each(function() {
 		sizes2.push($(this).outerHeight());
 	});
-	same(sizes, sizes2);
+	same(sizes, [38, 38, 38]);
 	
-	expected.find(".ui-accordion-content:first").height(500)
-	var sizes3 = [];
+	expected.parent().height(500);
+	expected.accordion("resize");
+	var sizes2 = [];
+	expected.find(".ui-accordion-content").each(function() {
+		sizes2.push($(this).outerHeight());
+	});
+	same(sizes2, [446, 446, 446]);
 });
 
 })(jQuery);
