diff options
| author | Mike Sherov <mike.sherov@gmail.com> | 2013-03-31 12:19:43 -0400 |
|---|---|---|
| committer | Scott González <scott.gonzalez@gmail.com> | 2013-04-17 15:07:08 -0400 |
| commit | 3f29430f8a282c80756af65996a69442f429f2ac (patch) | |
| tree | c465eaf766e9fdc9d408e07bc832ead8d86693a9 /tests | |
| parent | 90e97320324f8a769399874c7a54944fcaa125e1 (diff) | |
| download | jquery-ui-3f29430f8a282c80756af65996a69442f429f2ac.tar.gz jquery-ui-3f29430f8a282c80756af65996a69442f429f2ac.zip | |
Draggable Tests: Fix IE8 test failures by accounting for the IE8 testswarm IFRAME positioning bug.(cherry picked from commit 204a0423be63e0032f72b2ba1830a9b8dcebb9ee)
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/unit/draggable/draggable_options.js | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/tests/unit/draggable/draggable_options.js b/tests/unit/draggable/draggable_options.js index 8e1fde0bd..6a0cd593b 100644 --- a/tests/unit/draggable/draggable_options.js +++ b/tests/unit/draggable/draggable_options.js @@ -1112,7 +1112,8 @@ test( "#6817: auto scroll goes double distance when dragging", function() { scroll: true, stop: function( e, ui ) { equal( ui.offset.top, newY, "offset of item matches pointer position after scroll" ); - equal( ui.offset.top - offsetBefore.top, distance, "offset of item only moves expected distance after scroll" ); + // TODO: fix IE8 testswarm IFRAME positioning bug so closeEnough can be turned back to equal + closeEnough( ui.offset.top - offsetBefore.top, distance, 1, "offset of item only moves expected distance after scroll" ); } }), scrollSensitivity = element.draggable( "option", "scrollSensitivity" ), @@ -1137,7 +1138,7 @@ test( "#6817: auto scroll goes double distance when dragging", function() { }); test( "snap, snapMode, and snapTolerance", function() { - expect( 9 ); + expect( 10 ); var newX, newY, snapTolerance = 15, @@ -1163,7 +1164,9 @@ test( "snap, snapMode, and snapTolerance", function() { moves: 1 }); - deepEqual( element.offset(), { top: newY, left: newX }, "doesn't snap outside the snapTolerance" ); + // TODO: fix IE8 testswarm IFRAME positioning bug so closeEnough can be turned back to equal + closeEnough( element.offset().left, newX, 1, "doesn't snap outside the snapTolerance" ); + closeEnough( element.offset().top, newY, 1, "doesn't snap outside the snapTolerance" ); newX += 3; @@ -1257,7 +1260,7 @@ test( "snap, snapMode, and snapTolerance", function() { }); test( "#8459: element can snap to an element that was removed during drag", function() { - expect( 1 ); + expect( 2 ); var newX, newY, snapTolerance = 15, @@ -1286,7 +1289,9 @@ test( "#8459: element can snap to an element that was removed during drag", func moves: 1 }); - deepEqual( element.offset(), { top: newY, left: newX }, "doesn't snap to a removed element" ); + // TODO: fix IE8 testswarm IFRAME positioning bug so closeEnough can be turned back to equal + closeEnough( element.offset().left, newX, 1, "doesn't snap to a removed element" ); + closeEnough( element.offset().top, newY, 1, "doesn't snap to a removed element" ); }); test( "#8165: Snapping large rectangles to small rectangles doesn't snap properly", function() { |