From 56d6ddc2baf3331e6fb13ccbdda9d7029718ecd6 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Jo=CC=88rn=20Zaefferer?= <joern.zaefferer@gmail.com>
Date: Sat, 17 Oct 2015 15:12:26 -0400
Subject: [PATCH] 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.
---
 tests/unit/draggable/draggable.html | 24 ++++++++++++------------
 tests/unit/draggable/options.js     |  2 ++
 2 files changed, 14 insertions(+), 12 deletions(-)

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 );
-- 
2.39.5