Draggable Tests: Fix IE8 test failures by accounting for the IE8 testswarm IFRAME positioning bug.(cherry picked from commit 204a0423be)

This commit is contained in:
Mike Sherov 2013-03-31 12:19:43 -04:00 committed by Scott González
parent 90e9732032
commit 3f29430f8a

View File

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