library-mirrors/jquery-ui
1
0
Fork 0
mirror of https://github.com/jquery/jquery-ui.git synced 2025-01-07 20:34:24 +00:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Draggable Tests: make sure offset tests are actually testing offsets

Browse Source
This commit is contained in:
Mike Sherov 2013-10-19 15:47:13 -04:00
parent 67fd40eda7
commit 82edd58743
3 changed files with 14 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
tests/unit/draggable/draggable.html
View File

@ -46,10 +46,11 @@
<h2 id="qunit-userAgent"></h2>
<ol id="qunit-tests"></ol>
<div id="qunit-fixture">
<div id="main"></div>
<div id="draggable1" style="background: green; width: 200px; height: 100px;">Relative</div>
<div id="draggable2" style="background: green; width: 200px; height: 100px; position: absolute; top: 10px; left: 10px;"><span><em>Absolute</em></span></div>
<div id="droppable" style="background: green; width: 200px; height: 100px; position: absolute; top: 110px; left: 110px;"><span>Absolute</span></div>
<div id="main">
<div id="draggable1" style="background: green; width: 200px; height: 100px;">Relative</div>
<div id="draggable2" style="background: green; width: 200px; height: 100px; position: absolute; top: 10px; left: 10px;"><span><em>Absolute</em></span></div>
<div id="droppable" style="background: green; width: 200px; height: 100px; position: absolute; top: 110px; left: 110px;"><span>Absolute</span></div>
</div>
<div style="width: 1px; height: 1000px;"></div>
<div style="position: absolute; width: 1px; height: 2000px;"></div>
</div>

8
tests/unit/draggable/draggable_options.js
View File

@ -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() {

9
tests/unit/draggable/draggable_test_helpers.js
View File

@ -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 );
}