From 0cdd5591be15b2d605191656f3468f36f7d5a94d Mon Sep 17 00:00:00 2001
From: Jörn Zaefferer <joern.zaefferer@gmail.com>
Date: Wed, 24 Oct 2012 09:50:08 -0400
Subject: Accordion: Remove deprecated height options. Fixes #5868 - Accordion:
 Remove clearStyle option. Fixes #5872 - Accordion: Remove autoHeight and
 fillSpace options.

---
 .../unit/accordion/accordion_common_deprecated.js  |  5 +-
 tests/unit/accordion/accordion_deprecated.js       | 74 ----------------------
 2 files changed, 1 insertion(+), 78 deletions(-)

(limited to 'tests')

diff --git a/tests/unit/accordion/accordion_common_deprecated.js b/tests/unit/accordion/accordion_common_deprecated.js
index 22d5e1289..23324a0ad 100644
--- a/tests/unit/accordion/accordion_common_deprecated.js
+++ b/tests/unit/accordion/accordion_common_deprecated.js
@@ -3,14 +3,11 @@ TestHelpers.commonWidgetTests( "accordion", {
 		active: 0,
 		animate: null,
 		animated: "slide",
-		autoHeight: true,
-		clearStyle: false,
 		collapsible: false,
 		disabled: false,
 		event: "click",
-		fillSpace: false,
 		header: "> li > :first-child,> :not(li):even",
-		heightStyle: null,
+		heightStyle: "auto",
 		icons: {
 			"activeHeader": null,
 			"header": "ui-icon-triangle-1-e",
diff --git a/tests/unit/accordion/accordion_deprecated.js b/tests/unit/accordion/accordion_deprecated.js
index 2068fef95..b8f499542 100644
--- a/tests/unit/accordion/accordion_deprecated.js
+++ b/tests/unit/accordion/accordion_deprecated.js
@@ -104,80 +104,6 @@ test( "{ active: jQuery Object }", function() {
 
 
 
-
-module( "accordion (deprecated) - height options", setupTeardown() );
-
-test( "{ autoHeight: true }, default", function() {
-	expect( 3 );
-	equalHeight( $( "#navigation" ).accordion({ autoHeight: true }), 105 );
-});
-
-test( "{ autoHeight: false }", function() {
-	expect( 3 );
-	var element = $( "#navigation" ).accordion({ autoHeight: false }),
-		sizes = [];
-	element.find( ".ui-accordion-content" ).each(function() {
-		sizes.push( $(this).height() );
-	});
-	ok( sizes[0] >= 70 && sizes[0] <= 105, "was " + sizes[0] );
-	ok( sizes[1] >= 98 && sizes[1] <= 126, "was " + sizes[1] );
-	ok( sizes[2] >= 42 && sizes[2] <= 54, "was " + sizes[2] );
-});
-
-test( "{ fillSpace: true }", function() {
-	expect( 3 );
-	$( "#navigationWrapper" ).height( 500 );
-	var element = $( "#navigation" ).accordion({ fillSpace: true });
-	equalHeight( element, 455 );
-});
-
-test( "{ fillSapce: true } with sibling", function() {
-	expect( 3 );
-	$( "#navigationWrapper" ).height( 500 );
-	$( "<p>Lorem Ipsum</p>" )
-		.css({
-			height: 50,
-			marginTop: 20,
-			marginBottom: 30
-		})
-		.prependTo( "#navigationWrapper" );
-	var element = $( "#navigation" ).accordion({ fillSpace: true });
-	equalHeight( element , 355 );
-});
-
-test( "{ fillSpace: true } with multiple siblings", function() {
-	expect( 3 );
-	$( "#navigationWrapper" ).height( 500 );
-	$( "<p>Lorem Ipsum</p>" )
-		.css({
-			height: 50,
-			marginTop: 20,
-			marginBottom: 30
-		})
-		.prependTo( "#navigationWrapper" );
-	$( "<p>Lorem Ipsum</p>" )
-		.css({
-			height: 50,
-			marginTop: 20,
-			marginBottom: 30,
-			position: "absolute"
-		})
-		.prependTo( "#navigationWrapper" );
-	$( "<p>Lorem Ipsum</p>" )
-		.css({
-			height: 25,
-			marginTop: 10,
-			marginBottom: 15
-		})
-		.prependTo( "#navigationWrapper" );
-	var element = $( "#navigation" ).accordion({ fillSpace: true });
-	equalHeight( element, 305 );
-});
-
-
-
-
-
 module( "accordion (deprecated) - icons", setupTeardown() );
 
 test( "icons, headerSelected", function() {
-- 
cgit v1.2.3