From b6bdbb421177660c9f36cfbdb66430406f615427 Mon Sep 17 00:00:00 2001
From: Jörn Zaefferer <joern.zaefferer@gmail.com>
Date: Fri, 3 Jul 2009 21:05:22 +0000
Subject: accordion fix for #4653: added tests, adopted no-auto-height demo to
 also cover navigation option; fixed an unrelated bug, where
 ui-accordion-content-active was never removed

---
 tests/unit/accordion/accordion.html       |  6 +++---
 tests/unit/accordion/accordion_options.js | 24 ++++++++++++++++--------
 2 files changed, 19 insertions(+), 11 deletions(-)

(limited to 'tests/unit/accordion')

diff --git a/tests/unit/accordion/accordion.html b/tests/unit/accordion/accordion.html
index bce2e8cd7..fbb3a85d0 100644
--- a/tests/unit/accordion/accordion.html
+++ b/tests/unit/accordion/accordion.html
@@ -58,7 +58,7 @@
 
 	<ul id="navigation">
 		<li>
-			<a href="?p=1.1.1">Guitar</a>
+			<h2><a href="?p=1.1.1">Guitar</a></h2>
 			<ul>
 				<li><a href="?p=1.1.1.1">Electric</a></li>
 				<li><a href="?p=1.1.1.2">Acoustic</a></li>
@@ -68,7 +68,7 @@
 			</ul> 
 		</li>
 		<li>
-			<a href="?p=1.1.2"><span>Bass</span></a>
+			<h2><a href="?p=1.1.2"><span>Bass</span></a></h2>
 			<ul>
 				<li><a href="?p=1.1.2.1">Electric</a></li>
 				<li><a href="?p=1.1.2.2">Acoustic</a></li>
@@ -80,7 +80,7 @@
 			</ul> 
 		</li>
 		<li>
-			<a href="?p=1.1.3">Drums</a>
+			<h2><a href="?p=1.1.3">Drums</a></h2>
 			<ul>
 				<li><a href="?p=1.1.3.2">Acoustic Drums</a></li>
 				<li><a href="?p=1.1.3.3">Electronic Drums</a></li>
diff --git a/tests/unit/accordion/accordion_options.js b/tests/unit/accordion/accordion_options.js
index 1ef6542a3..95edae597 100644
--- a/tests/unit/accordion/accordion_options.js
+++ b/tests/unit/accordion/accordion_options.js
@@ -136,16 +136,24 @@ test("{ icons: false }", function() {
 	icons(false);
 });
 
-test("{ navigation: false }, default", function() {
-	ok(false, 'missing test - untested code is broken code');
-});
-
-test("{ navigation: true }", function() {
-	ok(false, 'missing test - untested code is broken code');
+test("{ navigation: true, navigationFilter: header }", function() {
+	$("#navigation").accordion({
+		navigation: true,
+		navigationFilter: function() {
+			return /\?p=1\.1\.3$/.test(this.href);
+		}
+	});
+	equals( $("#navigation .ui-accordion-content:eq(2)").size(), 1, "third content active" );
 });
 
-test("{ navigationFilter: Function }, default", function() {
-	ok(false, 'missing test - untested code is broken code');
+test("{ navigation: true, navigationFilter: content }", function() {
+	$("#navigation").accordion({
+		navigation: true,
+		navigationFilter: function() {
+			return /\?p=1\.1\.3\.2$/.test(this.href);
+		}
+	});
+	equals( $("#navigation .ui-accordion-content:eq(2)").size(), 1, "third content active" );
 });
 
 })(jQuery);
-- 
cgit v1.2.3