Permalink
Browse files

Draggable Tests: add back in unreliable offset hack for cursorAt tests

  • Loading branch information...
1 parent 46925b8 commit ecb7b6ec1217e06bc2a26a7da694ce0a0fd84221 @mikesherov mikesherov committed Dec 22, 2012
Showing with 6 additions and 5 deletions.
  1. +6 −5 tests/unit/draggable/draggable_options.js
@@ -281,7 +281,8 @@ test("{ cursorAt: false}, default", function() {
test( "{ cursorAt: left, top }", function() {
expect( 16 );
- var deltaX = -3, deltaY = -3,
+ var deltaX = -3,
+ deltaY = -3,
tests = {
"{ left: -5, top: -5 }": { x: -5, y: -5, cursorAt : { left: -5, top: -5 } },
"[ 10, 20 ]": { x: 10, y: 20, cursorAt : [ 10, 20 ] },
@@ -295,11 +296,11 @@ test( "{ cursorAt: left, top }", function() {
cursorAt: testData.cursorAt,
drag: function( event, ui ) {
if( testData.cursorAt.right ) {
- equal( ui.helper.width() - ( event.clientX - ui.offset.left ), testData.x, testName + " " + position + " left" );
- equal( ui.helper.height() - ( event.clientY - ui.offset.top ), testData.y, testName + position + " top" );
+ equal( ui.helper.width() - ( event.clientX - ui.offset.left ), testData.x + TestHelpers.draggable.unreliableOffset, testName + " " + position + " left" );
+ equal( ui.helper.height() - ( event.clientY - ui.offset.top ), testData.y + TestHelpers.draggable.unreliableOffset, testName + position + " top" );
} else {
- equal( event.clientX - ui.offset.left, testData.x, testName + " " + position + " left" );
- equal( event.clientY - ui.offset.top, testData.y, testName + " " + position + " top" );
+ equal( event.clientX - ui.offset.left, testData.x + TestHelpers.draggable.unreliableOffset, testName + " " + position + " left" );
+ equal( event.clientY - ui.offset.top, testData.y + TestHelpers.draggable.unreliableOffset, testName + " " + position + " top" );
}
}
});

3 comments on commit ecb7b6e

Owner

jzaefferer replied Dec 27, 2012

This test fails in IE7, reproducible through BrowserStack: http://swarm.jquery.org/result/802822

Member

mikesherov replied Dec 27, 2012

I think I meant -, not +.

Please sign in to comment.