From: Jörn Zaefferer <joern.zaefferer@gmail.com>
Date: Sat, 17 Oct 2015 19:12:26 +0000 (-0400)
Subject: Draggable: Fix options tests
X-Git-Tag: 1.12.0-beta.1~35
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=refs%2Fpull%2F1618%2Fhead;p=jquery-ui.git

Draggable: Fix options tests

These failed when running through tests/unit/all.html, due to the smaller
iframe. Reducing a browser window enough triggered the same issue. Making
the top offset much smaller fixes that.

The rest is fixing bad indent and adds a missing module call.
---

diff --git a/tests/unit/draggable/draggable.html b/tests/unit/draggable/draggable.html
index 0971b2325..53468a3a9 100644
--- a/tests/unit/draggable/draggable.html
+++ b/tests/unit/draggable/draggable.html
@@ -48,7 +48,7 @@
 	}
 	.sortable {
 		position: relative;
-		top: 8000px;
+		top: 800px;
 		left: 10px;
 		width: 300px;
 		padding: 0;
@@ -81,17 +81,17 @@
 	<div style="width: 1px; height: 1000px;"></div>
 	<div style="position: absolute; width: 1px; height: 2000px;"></div>
 	<ul id="sortable" class="sortable">
-		    <li id="draggableSortable">Item 0</li>
-		    <li id="draggableSortable2">Item 1</li>
-		    <li>Item 2</li>
-		    <li>Item 3</li>
-		  </ul>
-		  <ul id="sortable2" class="sortable">
-		    <li id="draggableSortableClone" class="sortable2Item">Item 0</li>
-		    <li>Item 1</li>
-		    <li>Item 2</li>
-		    <li>Item 3</li>
-		  </ul>
+		<li id="draggableSortable">Item 0</li>
+		<li id="draggableSortable2">Item 1</li>
+		<li>Item 2</li>
+		<li>Item 3</li>
+	</ul>
+	<ul id="sortable2" class="sortable">
+		<li id="draggableSortableClone" class="sortable2Item">Item 0</li>
+		<li>Item 1</li>
+		<li>Item 2</li>
+		<li>Item 3</li>
+	</ul>
 </div>
 
 </body>
diff --git a/tests/unit/draggable/options.js b/tests/unit/draggable/options.js
index 2c8de5a0a..2aa5a1ae1 100644
--- a/tests/unit/draggable/options.js
+++ b/tests/unit/draggable/options.js
@@ -6,6 +6,8 @@ define( [
 	"ui/widgets/sortable"
 ], function( $, testHelper ) {
 
+module( "draggable: options" );
+
 // TODO: This doesn't actually test whether append happened, possibly remove
 test( "{ appendTo: 'parent' }, default, no clone", function() {
 	expect( 4 );