diff --git a/tests/unit/draggable/draggable.html b/tests/unit/draggable/draggable.html
index 3d1d3326e0..17f9745a6c 100644
--- a/tests/unit/draggable/draggable.html
+++ b/tests/unit/draggable/draggable.html
@@ -46,10 +46,11 @@
-
-
Relative
-
Absolute
-
Absolute
+
+
Relative
+
Absolute
+
Absolute
+
diff --git a/tests/unit/draggable/draggable_options.js b/tests/unit/draggable/draggable_options.js
index ef551003db..27d53639f2 100644
--- a/tests/unit/draggable/draggable_options.js
+++ b/tests/unit/draggable/draggable_options.js
@@ -43,7 +43,7 @@ test( "{ appendTo: 'parent' }, default", function() {
equal( element.draggable( "option", "appendTo" ), "parent" );
TestHelpers.draggable.move( element, 1, 1 );
- equal( element.data( "last_dragged_parent" ), $( "#qunit-fixture" )[ 0 ] );
+ equal( element.data( "last_dragged_parent" ), $( "#main" )[ 0 ] );
});
test( "{ appendTo: Element }", function() {
@@ -91,12 +91,12 @@ test( "appendTo, default, switching after initialization", function() {
// Move and make sure element was appended to fixture
TestHelpers.draggable.move( element, 1, 1 );
- equal( element.data( "last_dragged_parent" ), $( "#qunit-fixture" )[ 0 ] );
+ equal( element.data( "last_dragged_parent" ), $( "#main" )[ 0 ] );
// Move and make sure element was appended to main
- element.draggable( "option", "appendTo", $( "#main" ) );
+ element.draggable( "option", "appendTo", $( "#qunit-fixture" ) );
TestHelpers.draggable.move( element, 2, 2 );
- equal( element.data( "last_dragged_parent" ), $( "#main" )[ 0 ] );
+ equal( element.data( "last_dragged_parent" ), $( "#qunit-fixture" )[ 0 ] );
});
test( "{ axis: false }, default", function() {
diff --git a/tests/unit/draggable/draggable_test_helpers.js b/tests/unit/draggable/draggable_test_helpers.js
index b365055568..00ff08d3e0 100644
--- a/tests/unit/draggable/draggable_test_helpers.js
+++ b/tests/unit/draggable/draggable_test_helpers.js
@@ -12,7 +12,10 @@ TestHelpers.draggable = {
$( handle ).simulate( "drag", {
dx: dx,
- dy: dy
+ dy: dy,
+ // moves is 1 here because simulate currently fire events synchronously
+ // so we can't faithfully test things that rely on a scroll event (which is async)
+ moves: 1
});
offsetAfter = el.offset();
@@ -43,9 +46,7 @@ TestHelpers.draggable = {
},
setScroll: function( what ) {
if( what ) {
- // TODO: currently, the draggable interaction doesn't properly account for scrolled pages,
- // uncomment the line below to make the tests fail that should when the page is scrolled
- // $( document ).scrollTop( 100 ).scrollLeft( 100 );
+ $( document ).scrollTop( 100 ).scrollLeft( 100 );
} else {
$( "#main" ).scrollTop( 100 ).scrollLeft( 100 );
}