]> source.dussan.org Git - jquery-ui.git/commitdiff
Draggable Tests: Style Sweep
authorMike Sherov <mike.sherov@gmail.com>
Mon, 4 Mar 2013 06:32:23 +0000 (01:32 -0500)
committerMike Sherov <mike.sherov@gmail.com>
Mon, 4 Mar 2013 06:32:23 +0000 (01:32 -0500)
tests/unit/draggable/draggable_options.js

index 0ab41b1f16fd8b098db168a95b9d125f446857fa..b996c644ca3d751ff72d6611a62085eed55de54d 100644 (file)
-/*
- * draggable_options.js
- */
-(function($) {
+(function( $ ) {
 
-module("draggable: options");
+module( "draggable: options" );
 
 // TODO: This doesn't actually test whether append happened, possibly remove
-test("{ appendTo: 'parent' }, default, no clone", function() {
+test( "{ appendTo: 'parent' }, default, no clone", function() {
        expect( 2 );
-       var el = $("#draggable2").draggable({ appendTo: "parent" });
-       TestHelpers.draggable.shouldMove(el);
-
-       el = $("#draggable1").draggable({ appendTo: "parent" });
-       TestHelpers.draggable.shouldMove(el);
+       var element = $( "#draggable2" ).draggable({ appendTo: "parent" });
+       TestHelpers.draggable.shouldMove( element );
 
+       element = $( "#draggable1" ).draggable({ appendTo: "parent" });
+       TestHelpers.draggable.shouldMove( element );
 });
 
 // TODO: This doesn't actually test whether append happened, possibly remove
-test("{ appendTo: Element }, no clone", function() {
+test( "{ appendTo: Element }, no clone", function() {
        expect( 2 );
-       var el = $("#draggable2").draggable({appendTo: $("#draggable2").parent()[0] });
+       var element = $( "#draggable2" ).draggable({ appendTo: $( "#draggable2" ).parent()[ 0 ] });
 
-       TestHelpers.draggable.shouldMove(el);
+       TestHelpers.draggable.shouldMove( element );
 
-       el = $("#draggable1").draggable({ appendTo: $("#draggable2").parent()[0] });
-       TestHelpers.draggable.shouldMove(el);
+       element = $( "#draggable1" ).draggable({ appendTo: $( "#draggable2" ).parent()[ 0 ] });
+       TestHelpers.draggable.shouldMove( element );
 });
 
 // TODO: This doesn't actually test whether append happened, possibly remove
-test("{ appendTo: Selector }, no clone", function() {
+test( "{ appendTo: Selector }, no clone", function() {
        expect( 2 );
-       var el = $("#draggable2").draggable({ appendTo: "#main" });
-       TestHelpers.draggable.shouldMove(el);
+       var element = $( "#draggable2" ).draggable({ appendTo: "#main" });
+       TestHelpers.draggable.shouldMove( element );
 
-       el = $("#draggable1").draggable({ appendTo: "#main" });
-       TestHelpers.draggable.shouldMove(el);
+       element = $( "#draggable1" ).draggable({ appendTo: "#main" });
+       TestHelpers.draggable.shouldMove( element );
 });
 
-test("{ appendTo: 'parent' }, default", function() {
-
-       expect(2);
-
-       var el = $("#draggable1").draggable();
+test( "{ appendTo: 'parent' }, default", function() {
+       expect( 2 );
 
-       TestHelpers.draggable.trackAppendedParent(el);
+       var element = $( "#draggable1" ).draggable();
 
-       equal( el.draggable( "option", "appendTo" ), "parent" );
+       TestHelpers.draggable.trackAppendedParent( element );
 
-       TestHelpers.draggable.move(el, 1, 1);
-       equal( el.data("last_dragged_parent"), $("#qunit-fixture")[0] );
+       equal( element.draggable( "option", "appendTo" ), "parent" );
 
+       TestHelpers.draggable.move( element, 1, 1 );
+       equal( element.data( "last_dragged_parent" ), $( "#qunit-fixture" )[ 0 ] );
 });
 
-test("{ appendTo: Element }", function() {
-
-       expect(1);
-
-       var appendTo = $("#draggable2").parent()[0],
-               el = $("#draggable1").draggable({ appendTo: appendTo });
+test( "{ appendTo: Element }", function() {
+       expect( 1 );
 
-       TestHelpers.draggable.trackAppendedParent(el);
+       var appendTo = $( "#draggable2" ).parent()[ 0 ],
+               element = $( "#draggable1" ).draggable({ appendTo: appendTo });
 
-       TestHelpers.draggable.move(el, 1, 1);
-       equal( el.data("last_dragged_parent"), appendTo );
+       TestHelpers.draggable.trackAppendedParent( element );
 
+       TestHelpers.draggable.move( element, 1, 1 );
+       equal( element.data( "last_dragged_parent" ), appendTo );
 });
 
-test("{ appendTo: jQuery }", function() {
-
-       expect(1);
-
-       var appendTo = $("#draggable2").parent(),
-               el = $("#draggable1").draggable({ appendTo: appendTo });
+test( "{ appendTo: jQuery }", function() {
+       expect( 1 );
 
-       TestHelpers.draggable.trackAppendedParent(el);
+       var appendTo = $( "#draggable2" ).parent(),
+               element = $( "#draggable1" ).draggable({ appendTo: appendTo });
 
-       TestHelpers.draggable.move(el, 1, 1);
-       equal( el.data("last_dragged_parent"), appendTo[0] );
+       TestHelpers.draggable.trackAppendedParent( element );
 
+       TestHelpers.draggable.move( element, 1, 1 );
+       equal( element.data( "last_dragged_parent" ), appendTo[ 0 ] );
 });
-test("{ appendTo: Selector }", function() {
-
-       expect(1);
 
+test( "{ appendTo: Selector }", function() {
+       expect( 1 );
 
        var appendTo = "#main",
-               el = $("#draggable1").draggable({ appendTo: appendTo });
+               element = $( "#draggable1" ).draggable({ appendTo: appendTo });
 
-       TestHelpers.draggable.trackAppendedParent(el);
-
-       TestHelpers.draggable.move(el, 1, 1);
-       equal( el.data("last_dragged_parent"), $(appendTo)[0] );
+       TestHelpers.draggable.trackAppendedParent( element );
 
+       TestHelpers.draggable.move( element, 1, 1 );
+       equal( element.data( "last_dragged_parent" ), $(appendTo)[ 0 ] );
 });
 
+test( "appendTo, default, switching after initialization", function() {
+       expect( 2 );
 
-test("appendTo, default, switching after initialization", function() {
-
-       expect(2);
-
-       var el = $("#draggable1").draggable({ helper : "clone" });
-
-       TestHelpers.draggable.trackAppendedParent(el);
+       var element = $( "#draggable1" ).draggable({ helper : "clone" });
 
-       // Move and make sure el was appended to fixture
-       TestHelpers.draggable.move(el, 1, 1);
-       equal( el.data("last_dragged_parent"), $("#qunit-fixture")[0] );
+       TestHelpers.draggable.trackAppendedParent( element );
 
-       // Move and make sure el was appended to main
-       el.draggable( "option", "appendTo", $("#main") );
-       TestHelpers.draggable.move(el, 2, 2);
-       equal( el.data("last_dragged_parent"), $("#main")[0] );
+       // Move and make sure element was appended to fixture
+       TestHelpers.draggable.move( element, 1, 1 );
+       equal( element.data( "last_dragged_parent" ), $( "#qunit-fixture" )[ 0 ] );
 
+       // Move and make sure element was appended to main
+       element.draggable( "option", "appendTo", $( "#main" ) );
+       TestHelpers.draggable.move( element, 2, 2 );
+       equal( element.data( "last_dragged_parent" ), $( "#main" )[ 0 ] );
 });
 
-test("{ axis: false }, default", function() {
+test( "{ axis: false }, default", function() {
        expect( 1 );
-       var el = $("#draggable2").draggable({ axis: false });
-       TestHelpers.draggable.shouldMove(el);
+       var element = $( "#draggable2" ).draggable({ axis: false });
+       TestHelpers.draggable.shouldMove( element );
 });
 
-test("{ axis: 'x' }", function() {
+test( "{ axis: 'x' }", function() {
        expect( 1 );
-       var el = $("#draggable2").draggable({ axis: "x" });
-       TestHelpers.draggable.testDrag(el, el, 50, 50, 50, 0);
+       var element = $( "#draggable2" ).draggable({ axis: "x" });
+       TestHelpers.draggable.testDrag( element, element, 50, 50, 50, 0 );
 });
 
-test("{ axis: 'y' }", function() {
+test( "{ axis: 'y' }", function() {
        expect( 1 );
-       var el = $("#draggable2").draggable({ axis: "y" });
-       TestHelpers.draggable.testDrag(el, el, 50, 50, 0, 50);
+       var element = $( "#draggable2" ).draggable({ axis: "y" });
+       TestHelpers.draggable.testDrag( element, element, 50, 50, 0, 50 );
 });
 
-test("{ axis: ? }, unexpected", function() {
-       var el,
+test( "{ axis: ? }, unexpected", function() {
+       var element,
                unexpected = {
                        "true": true,
                        "{}": {},
@@ -146,63 +131,61 @@ test("{ axis: ? }, unexpected", function() {
        expect( 6 );
 
        $.each(unexpected, function(key, val) {
-               el = $("#draggable2").draggable({ axis: val });
-               TestHelpers.draggable.testDrag(el, el, 50, 50, 50, 50, "axis: " + key);
-               el.draggable("destroy");
+               element = $( "#draggable2" ).draggable({ axis: val });
+               TestHelpers.draggable.testDrag( element, element, 50, 50, 50, 50, "axis: " + key );
+               element.draggable( "destroy" );
        });
 });
 
-test("axis, default, switching after initialization", function() {
-
-       expect(3);
+test( "axis, default, switching after initialization", function() {
+       expect( 3 );
 
-       var el;
+       var element = $( "#draggable1" ).draggable({ axis : false });
 
-       // Any direction
-       el = $("#draggable1").draggable({ axis : false });
-       TestHelpers.draggable.testDrag(el, el, 50, 50, 50, 50);
+       // Any Direction
+       TestHelpers.draggable.testDrag( element, element, 50, 50, 50, 50 );
 
        // Only horizontal
-       el.draggable("option", "axis", "x");
-       TestHelpers.draggable.testDrag(el, el, 50, 50, 50, 0);
+       element.draggable( "option", "axis", "x" );
+       TestHelpers.draggable.testDrag( element, element, 50, 50, 50, 0 );
 
        // Vertical only
-       el.draggable("option", "axis", "y");
-       TestHelpers.draggable.testDrag(el, el, 50, 50, 0, 50);
+       element.draggable( "option", "axis", "y" );
+       TestHelpers.draggable.testDrag( element, element, 50, 50, 0, 50 );
 
 });
 
-test("{ cancel: 'input,textarea,button,select,option' }, default", function() {
+test( "{ cancel: 'input,textarea,button,select,option' }, default", function() {
        expect( 2 );
 
-       $("<div id='draggable-option-cancel-default'><input type='text'></div>").appendTo("#main");
+       $( "<div id='draggable-option-cancel-default'><input type='text'></div>" ).appendTo( "#main" );
 
-       var el = $("#draggable-option-cancel-default").draggable({ cancel: "input,textarea,button,select,option" });
-       TestHelpers.draggable.shouldMove(el);
+       var element = $( "#draggable-option-cancel-default" ).draggable({ cancel: "input,textarea,button,select,option" });
+       TestHelpers.draggable.shouldMove( element );
 
-       el.draggable("destroy");
+       element.draggable( "destroy" );
 
-       el = $("#draggable-option-cancel-default").draggable({ cancel: "input,textarea,button,select,option" });
-       TestHelpers.draggable.testDrag(el, "#draggable-option-cancel-default input", 50, 50, 0, 0);
-       el.draggable("destroy");
+       element = $( "#draggable-option-cancel-default" ).draggable({ cancel: "input,textarea,button,select,option" });
+       TestHelpers.draggable.testDrag( element, "#draggable-option-cancel-default input", 50, 50, 0, 0 );
+       element.draggable( "destroy" );
 });
 
-test("{ cancel: 'span' }", function() {
+test( "{ cancel: 'span' }", function() {
        expect( 2 );
 
-       var el = $("#draggable2").draggable();
-       TestHelpers.draggable.testDrag(el, "#draggable2 span", 50, 50, 50, 50);
+       var element = $( "#draggable2" ).draggable();
+       TestHelpers.draggable.testDrag( element, "#draggable2 span", 50, 50, 50, 50 );
 
-       el.draggable("destroy");
+       element.draggable( "destroy" );
 
-       el = $("#draggable2").draggable({ cancel: "span" });
-       TestHelpers.draggable.testDrag(el, "#draggable2 span", 50, 50, 0, 0);
+       element = $( "#draggable2" ).draggable({ cancel: "span" });
+       TestHelpers.draggable.testDrag( element, "#draggable2 span", 50, 50, 0, 0 );
 });
 
 test( "{ cancel: ? }, unexpected", function() {
        expect( 6 );
 
-       var el,
+       var element,
                unexpected = {
                        "true": true,
                        "false": false,
@@ -213,63 +196,62 @@ test( "{ cancel: ? }, unexpected", function() {
                };
 
        $.each( unexpected, function( key, val ) {
-               el = $("#draggable2").draggable({ cancel: val });
-               TestHelpers.draggable.shouldMove( el, "cancel: " + key );
-               el.draggable("destroy");
+               element = $( "#draggable2" ).draggable({ cancel: val });
+               TestHelpers.draggable.shouldMove( element, "cancel: " + key );
+               element.draggable( "destroy" );
        });
 });
 
 /**
-test("{ cancel: Selectors }, matching parent selector", function() {
+test( "{ cancel: Selectors }, matching parent selector", function() {
 
        expect( 5 );
 
-       var el = $("#draggable2").draggable({ cancel: "span a" });
+       var element = $( "#draggable2" ).draggable({ cancel: "span a" });
 
-       $("#qunit-fixture").append( "<span id='wrapping'><a></a></span>" );
+       $( "#qunit-fixture" ).append( "<span id='wrapping'><a></a></span>" );
 
-       el.find( "span" ).append( "<a>" );
+       element.find( "span" ).append( "<a>" );
 
-       $("#wrapping a").append( el );
+       $( "#wrapping a" ).append( element );
 
-       TestHelpers.draggable.testDrag(el, "#draggable2 span", 50, 50, 50, 50, "drag span child");
-       TestHelpers.draggable.shouldNotMove( $("#draggable2 span a") );
-       TestHelpers.draggable.shouldNotMove( $("#wrapping a") );
+       TestHelpers.draggable.testDrag( element, "#draggable2 span", 50, 50, 50, 50, "drag span child" );
+       TestHelpers.draggable.shouldNotMove( $( "#draggable2 span a" ) );
+       TestHelpers.draggable.shouldNotMove( $( "#wrapping a" ) );
 
-       $("#draggable2").draggable( "option", "cancel", "span > a" );
-       $("#draggable2").find( "a" ).append( "<a>" );
+       $( "#draggable2" ).draggable( "option", "cancel", "span > a" );
+       $( "#draggable2" ).find( "a" ).append( "<a>" );
 
 
-       TestHelpers.draggable.testDrag(el, $("#draggable2 span a").last(), 50, 50, 50, 50, "drag span child");
-       TestHelpers.draggable.shouldNotMove( $("#draggable2 span a").first() );
+       TestHelpers.draggable.testDrag( element, $( "#draggable2 span a" ).last(), 50, 50, 50, 50, "drag span child" );
+       TestHelpers.draggable.shouldNotMove( $( "#draggable2 span a" ).first() );
 
 });
 */
 
-test("cancel, default, switching after initialization", function() {
+test( "cancelement, default, switching after initialization", function() {
        expect( 3 );
 
-       $("<div id='draggable-option-cancel-default'><input type='text'></div>").appendTo("#main");
+       $( "<div id='draggable-option-cancel-default'><input type='text'></div>" ).appendTo( "#main" );
 
-       var input = $("#draggable-option-cancel-default input"),
-               el = $("#draggable-option-cancel-default").draggable();
+       var input = $( "#draggable-option-cancel-default input" ),
+               element = $( "#draggable-option-cancel-default" ).draggable();
 
-       TestHelpers.draggable.testDrag(el, input, 50, 50, 0, 0);
+       TestHelpers.draggable.testDrag( element, input, 50, 50, 0, 0 );
 
-       el.draggable("option", "cancel", "textarea" );
-       TestHelpers.draggable.testDrag(el, input, 50, 50, 50, 50);
-
-       el.draggable("option", "cancel", "input" );
-       TestHelpers.draggable.testDrag(el, input, 50, 50, 0, 0);
+       element.draggable( "option", "cancel", "textarea" );
+       TestHelpers.draggable.testDrag( element, input, 50, 50, 50, 50 );
 
+       element.draggable( "option", "cancel", "input" );
+       TestHelpers.draggable.testDrag( element, input, 50, 50, 0, 0 );
 });
 
 /*
 
-test("{ connectToSortable: selector }, default", function() {
+test( "{ connectToSortable: selector }, default", function() {
        expect( 1 );
 
-       ok(false, "missing test - untested code is broken code");
+       ok(false, "missing test - untested code is broken code" );
 });
 */
 
@@ -277,184 +259,185 @@ test( "{ containment: Element }", function() {
        expect( 1 );
 
        var offsetAfter,
-               el = $( "#draggable1" ).draggable({ containment: $( "#draggable1" ).parent()[ 0 ] }),
-               p = el.parent(),
+               element = $( "#draggable1" ).draggable({ containment: $( "#draggable1" ).parent()[ 0 ] }),
+               p = element.parent(),
                po = p.offset(),
                expected = {
-                       left: po.left + TestHelpers.draggable.border(p, "left") + TestHelpers.draggable.margin(el, "left"),
-                       top: po.top + TestHelpers.draggable.border(p, "top") + TestHelpers.draggable.margin(el, "top")
+                       left: po.left + TestHelpers.draggable.border( p, "left" ) + TestHelpers.draggable.margin( element, "left" ),
+                       top: po.top + TestHelpers.draggable.border( p, "top" ) + TestHelpers.draggable.margin( element, "top" )
                };
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                dx: -100,
                dy: -100
        });
-       offsetAfter = el.offset();
-       deepEqual(offsetAfter, expected, "compare offset to parent");
+       offsetAfter = element.offset();
+       deepEqual( offsetAfter, expected, "compare offset to parent" );
 });
 
 test( "{ containment: Selector }", function() {
        expect( 1 );
 
        var offsetAfter,
-               el = $( "#draggable1" ).draggable({ containment: $( "#qunit-fixture" ) }),
-               p = el.parent(),
+               element = $( "#draggable1" ).draggable({ containment: $( "#qunit-fixture" ) }),
+               p = element.parent(),
                po = p.offset(),
                expected = {
-                       left: po.left + TestHelpers.draggable.border(p, "left") + TestHelpers.draggable.margin(el, "left"),
-                       top: po.top + TestHelpers.draggable.border(p, "top") + TestHelpers.draggable.margin(el, "top")
+                       left: po.left + TestHelpers.draggable.border( p, "left" ) + TestHelpers.draggable.margin( element, "left" ),
+                       top: po.top + TestHelpers.draggable.border( p, "top" ) + TestHelpers.draggable.margin( element, "top" )
                };
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                dx: -100,
                dy: -100
        });
-       offsetAfter = el.offset();
-       deepEqual(offsetAfter, expected, "compare offset to parent");
+       offsetAfter = element.offset();
+       deepEqual( offsetAfter, expected, "compare offset to parent" );
 });
 
 test( "{ containment: [x1, y1, x2, y2] }", function() {
        expect( 1 );
 
-       var el = $( "#draggable1" ).draggable(),
-               eo = el.offset();
+       var element = $( "#draggable1" ).draggable(),
+               eo = element.offset();
 
-       el.draggable( "option", "containment", [ eo.left, eo.top, eo.left + el.width() + 5, eo.left + el.width() + 5 ] );
+       element.draggable( "option", "containment", [ eo.left, eo.top, eo.left + element.width() + 5, eo.left + element.width() + 5 ] );
 
-       TestHelpers.draggable.testDrag( el, el, -100, -100, 0, 0 );
+       TestHelpers.draggable.testDrag( element, element, -100, -100, 0, 0 );
 });
 
-test("{ containment: 'parent' }, relative", function() {
+test( "{ containment: 'parent' }, relative", function() {
        expect( 1 );
 
        var offsetAfter,
-               el = $("#draggable1").draggable({ containment: "parent" }),
-               p = el.parent(),
+               element = $( "#draggable1" ).draggable({ containment: "parent" }),
+               p = element.parent(),
                po = p.offset(),
                expected = {
-                       left: po.left + TestHelpers.draggable.border(p, "left") + TestHelpers.draggable.margin(el, "left"),
-                       top: po.top + TestHelpers.draggable.border(p, "top") + TestHelpers.draggable.margin(el, "top")
+                       left: po.left + TestHelpers.draggable.border( p, "left" ) + TestHelpers.draggable.margin( element, "left" ),
+                       top: po.top + TestHelpers.draggable.border( p, "top" ) + TestHelpers.draggable.margin( element, "top" )
                };
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                dx: -100,
                dy: -100
        });
-       offsetAfter = el.offset();
-       deepEqual(offsetAfter, expected, "compare offset to parent");
+       offsetAfter = element.offset();
+       deepEqual( offsetAfter, expected, "compare offset to parent" );
 });
 
-test("{ containment: 'parent' }, absolute", function() {
+test( "{ containment: 'parent' }, absolute", function() {
        expect( 1 );
 
        var offsetAfter,
-               el = $("#draggable2").draggable({ containment: "parent" }),
-               p = el.parent(),
+               element = $( "#draggable2" ).draggable({ containment: "parent" }),
+               p = element.parent(),
                po = p.offset(),
                expected = {
-                       left: po.left + TestHelpers.draggable.border(p, "left") + TestHelpers.draggable.margin(el, "left"),
-                       top: po.top + TestHelpers.draggable.border(p, "top") + TestHelpers.draggable.margin(el, "top")
+                       left: po.left + TestHelpers.draggable.border( p, "left" ) + TestHelpers.draggable.margin( element, "left" ),
+                       top: po.top + TestHelpers.draggable.border( p, "top" ) + TestHelpers.draggable.margin( element, "top" )
                };
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                dx: -100,
                dy: -100
        });
-       offsetAfter = el.offset();
-       deepEqual(offsetAfter, expected, "compare offset to parent");
+       offsetAfter = element.offset();
+       deepEqual( offsetAfter, expected, "compare offset to parent" );
 });
 
-test("containment, default, switching after initialization", function() {
+test( "containment, default, switching after initialization", function() {
        expect( 2 );
 
-       var el = $("#draggable1").draggable({ containment: false });
+       var element = $( "#draggable1" ).draggable({ containment: false });
 
-       TestHelpers.draggable.testDrag(el, el, -100, -100, -100, -100);
+       TestHelpers.draggable.testDrag( element, element, -100, -100, -100, -100 );
 
-       el.draggable( "option", "containment", "parent" )
-               .css({top:0,left:0})
-               .appendTo( $("#main") );
+       element.draggable( "option", "containment", "parent" )
+               .css({
+                       top: 0,
+                       left: 0
+               })
+               .appendTo( $( "#main" ) );
 
-       TestHelpers.draggable.testDrag(el, el, -100, -100, 0, 0);
+       TestHelpers.draggable.testDrag( element, element, -100, -100, 0, 0 );
 
        // TODO: Switching back to false does not update to false
-       // el.draggable( "option", "containment", false );
-       // TestHelpers.draggable.testDrag(el, el, -100, -100, -100, -100);
-
+       // element.draggable( "option", "containment", false );
+       // TestHelpers.draggable.testDrag( element, element, -100, -100, -100, -100 );
 });
 
-test("{ cursor: 'auto' }, default", function() {
-       function getCursor() { return $("#draggable2").css("cursor"); }
+test( "{ cursor: 'auto' }, default", function() {
+       function getCursor() {
+               return $( "#draggable2" ).css( "cursor" );
+       }
 
-       expect(2);
+       expect( 2 );
 
-       var actual, before, after,
+       var actual, after,
                expected = "auto",
-               el = $("#draggable2").draggable({
+               element = $( "#draggable2" ).draggable({
                        cursor: expected,
                        start: function() {
                                actual = getCursor();
                        }
-               });
+               }),
+               before = getCursor();
 
-       before = getCursor();
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                dx: -1,
                dy: -1
        });
        after = getCursor();
 
-       equal(actual, expected, "start callback: cursor '" + expected + "'");
-       equal(after, before, "after drag: cursor restored");
-
+       equal( actual, expected, "start callback: cursor '" + expected + "'" );
+       equal( after, before, "after drag: cursor restored" );
 });
 
-test("{ cursor: 'move' }", function() {
-
-       function getCursor() { return $("body").css("cursor"); }
+test( "{ cursor: 'move' }", function() {
+       function getCursor() {
+               return $( "body" ).css( "cursor" );
+       }
 
-       expect(2);
+       expect( 2 );
 
-       var actual, before, after,
+       var actual, after,
                expected = "move",
-               el = $("#draggable2").draggable({
+               element = $( "#draggable2" ).draggable({
                        cursor: expected,
                        start: function() {
                                actual = getCursor();
                        }
-               });
+               }),
+               before = getCursor();
 
-       before = getCursor();
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                dx: -1,
                dy: -1
        });
        after = getCursor();
 
-       equal(actual, expected, "start callback: cursor '" + expected + "'");
-       equal(after, before, "after drag: cursor restored");
-
+       equal( actual, expected, "start callback: cursor '" + expected + "'" );
+       equal( after, before, "after drag: cursor restored" );
 });
 
-test("cursor, default, switching after initialization", function() {
-
-       expect(3);
-
-       var el = $("#draggable1").draggable();
+test( "cursor, default, switching after initialization", function() {
+       expect( 3 );
 
-       TestHelpers.draggable.trackMouseCss( el );
+       var element = $( "#draggable1" ).draggable();
 
-       TestHelpers.draggable.move( el, 1, 1 );
-       equal( el.data("last_dragged_cursor"), "auto" );
+       TestHelpers.draggable.trackMouseCss( element );
 
-       el.draggable( "option", "cursor", "move" );
-       TestHelpers.draggable.move( el, 1, 1 );
-       equal( el.data("last_dragged_cursor"), "move" );
+       TestHelpers.draggable.move( element, 1, 1 );
+       equal( element.data( "last_dragged_cursor" ), "auto" );
 
-       el.draggable( "option", "cursor", "ns-resize" );
-       TestHelpers.draggable.move( el, 1, 1 );
-       equal( el.data("last_dragged_cursor"), "ns-resize" );
+       element.draggable( "option", "cursor", "move" );
+       TestHelpers.draggable.move( element, 1, 1 );
+       equal( element.data( "last_dragged_cursor" ), "move" );
 
+       element.draggable( "option", "cursor", "ns-resize" );
+       TestHelpers.draggable.move( element, 1, 1 );
+       equal( element.data( "last_dragged_cursor" ), "ns-resize" );
 });
 
 test( "cursorAt", function() {
@@ -473,7 +456,7 @@ test( "cursorAt", function() {
 
        $.each( tests, function( testName, testData ) {
                $.each( [ "relative", "absolute" ], function( i, position ) {
-                       var el = $( "#draggable" + ( i + 1 ) ).draggable({
+                       var element = $( "#draggable" + ( i + 1 ) ).draggable({
                                        cursorAt: testData.cursorAt,
                                        drag: function( event, ui ) {
                                                if( !testData.cursorAt ) {
@@ -489,7 +472,7 @@ test( "cursorAt", function() {
                                        }
                        });
 
-                       el.simulate( "drag", {
+                       element.simulate( "drag", {
                                moves: 1,
                                dx: deltaX,
                                dy: deltaY
@@ -499,7 +482,6 @@ test( "cursorAt", function() {
 });
 
 test( "cursorAt, switching after initialization", function() {
-
        expect( 24 );
 
        var deltaX = -3,
@@ -515,9 +497,9 @@ test( "cursorAt, switching after initialization", function() {
 
        $.each( tests, function( testName, testData ) {
                $.each( [ "relative", "absolute" ], function( i, position ) {
-                       var el = $( "#draggable" + ( i + 1 ) );
+                       var element = $( "#draggable" + ( i + 1 ) );
 
-                       el.draggable({
+                       element.draggable({
                                        drag: function( event, ui ) {
                                                if( !testData.cursorAt ) {
                                                        equal( ui.position.left - ui.originalPosition.left, deltaX, testName + " " + position + " left" );
@@ -532,527 +514,502 @@ test( "cursorAt, switching after initialization", function() {
                                        }
                        });
 
-                       el.draggable( "option", "cursorAt", false );
-                       el.draggable( "option", "cursorAt", testData.cursorAt );
+                       element.draggable( "option", "cursorAt", false );
+                       element.draggable( "option", "cursorAt", testData.cursorAt );
 
-                       el.simulate( "drag", {
+                       element.simulate( "drag", {
                                moves: 1,
                                dx: deltaX,
                                dy: deltaY
                        });
                });
        });
-
 });
 
-
 test( "disabled", function() {
-
        expect( 3 );
 
-       var el = $("#draggable1").draggable();
+       var element = $( "#draggable1" ).draggable();
 
-       TestHelpers.draggable.shouldMove(el);
+       TestHelpers.draggable.shouldMove( element );
 
-       el.draggable( "option", "disabled", true );
-       TestHelpers.draggable.shouldNotMove(el);
-
-       el.draggable( "option", "disabled", false );
-       TestHelpers.draggable.shouldMove(el);
+       element.draggable( "option", "disabled", true );
+       TestHelpers.draggable.shouldNotMove( element );
 
+       element.draggable( "option", "disabled", false );
+       TestHelpers.draggable.shouldMove( element );
 });
 
-test("{ grid: [50, 50] }, relative", function() {
+test( "{ grid: [50, 50] }, relative", function() {
        expect( 2 );
 
-       var el = $("#draggable1").draggable({ grid: [50, 50] });
-       TestHelpers.draggable.testDrag(el, el, 24, 24, 0, 0);
-       TestHelpers.draggable.testDrag(el, el, 26, 25, 50, 50);
+       var element = $( "#draggable1" ).draggable({ grid: [ 50, 50 ] });
+       TestHelpers.draggable.testDrag( element, element, 24, 24, 0, 0 );
+       TestHelpers.draggable.testDrag( element, element, 26, 25, 50, 50 );
 });
 
-test("{ grid: [50, 50] }, absolute", function() {
+test( "{ grid: [50, 50] }, absolute", function() {
        expect( 2 );
 
-       var el = $("#draggable2").draggable({ grid: [50, 50] });
-       TestHelpers.draggable.testDrag(el, el, 24, 24, 0, 0);
-       TestHelpers.draggable.testDrag(el, el, 26, 25, 50, 50);
+       var element = $( "#draggable2" ).draggable({ grid: [ 50, 50 ] });
+       TestHelpers.draggable.testDrag( element, element, 24, 24, 0, 0 );
+       TestHelpers.draggable.testDrag( element, element, 26, 25, 50, 50 );
 });
 
-test("grid, switching after initialization", function() {
-
+test( "grid, switching after initialization", function() {
        expect( 4 );
 
-       var el = $("#draggable1").draggable();
-       // Forward
-       TestHelpers.draggable.testDrag(el, el, 24, 24, 24, 24);
-       TestHelpers.draggable.testDrag(el, el, 0, 0, 0, 0);
+       var element = $( "#draggable1" ).draggable();
 
-       el.draggable( "option", "grid", [50,50] );
+       // Forward
+       TestHelpers.draggable.testDrag( element, element, 24, 24, 24, 24 );
+       TestHelpers.draggable.testDrag( element, element, 0, 0, 0, 0 );
 
-       TestHelpers.draggable.testDrag(el, el, 24, 24, 0, 0);
-       TestHelpers.draggable.testDrag(el, el, 26, 25, 50, 50);
+       element.draggable( "option", "grid", [ 50,50 ] );
 
+       TestHelpers.draggable.testDrag( element, element, 24, 24, 0, 0 );
+       TestHelpers.draggable.testDrag( element, element, 26, 25, 50, 50 );
 });
 
-test("{ handle: 'span' }", function() {
+test( "{ handle: 'span' }", function() {
        expect( 2 );
 
-       var el = $("#draggable2").draggable({ handle: "span" });
+       var element = $( "#draggable2" ).draggable({ handle: "span" });
 
-       TestHelpers.draggable.testDrag(el, "#draggable2 span", 50, 50, 50, 50, "drag span");
-       TestHelpers.draggable.shouldNotMove(el, "drag element");
+       TestHelpers.draggable.testDrag( element, "#draggable2 span", 50, 50, 50, 50, "drag span" );
+       TestHelpers.draggable.shouldNotMove( element, "drag element" );
 });
 
 /*
-test("{ handle: Selectors }, matching parent selector", function() {
+test( "{ handle: Selectors }, matching parent selector", function() {
 
        expect( 4 );
 
-       var el = $("#draggable2").draggable({ handle: "span a" });
+       var element = $( "#draggable2" ).draggable({ handle: "span a" });
 
-       $("#qunit-fixture").append( "<span id='wrapping'><a></a></span>" );
+       $( "#qunit-fixture" ).append( "<span id='wrapping'><a></a></span>" );
 
-       el.find( "span" ).append( "<a>" );
+       element.find( "span" ).append( "<a>" );
 
-       $("#wrapping a").append( el );
+       $( "#wrapping a" ).append( element );
 
-       TestHelpers.draggable.testDrag(el, "#draggable2 span a", 50, 50, 50, 50, "drag span child");
-       TestHelpers.draggable.shouldNotMove( $("#wrapping a") );
+       TestHelpers.draggable.testDrag( element, "#draggable2 span a", 50, 50, 50, 50, "drag span child" );
+       TestHelpers.draggable.shouldNotMove( $( "#wrapping a" ) );
 
-       $("#draggable2").draggable( "option", "handle", "span > a" );
-       $("#draggable2").find( "a" ).append( "<a>" );
+       $( "#draggable2" ).draggable( "option", "handle", "span > a" );
+       $( "#draggable2" ).find( "a" ).append( "<a>" );
 
-       TestHelpers.draggable.testDrag(el, $("#draggable2 span a").first(), 50, 50, 50, 50, "drag span child");
-       TestHelpers.draggable.shouldNotMove( $("#draggable2 span a").last() );
+       TestHelpers.draggable.testDrag( element, $( "#draggable2 span a" ).first(), 50, 50, 50, 50, "drag span child" );
+       TestHelpers.draggable.shouldNotMove( $( "#draggable2 span a" ).last() );
 
 });
 */
 
-test("handle, default, switching after initialization", function() {
+test( "handle, default, switching after initialization", function() {
        expect( 6 );
 
-       var el = $("#draggable2").draggable();
+       var element = $( "#draggable2" ).draggable();
 
-       TestHelpers.draggable.testDrag(el, el, 50, 50, 50, 50);
-       TestHelpers.draggable.testDrag(el, "#draggable2 span", 100, 100, 100, 100);
+       TestHelpers.draggable.testDrag( element, element, 50, 50, 50, 50 );
+       TestHelpers.draggable.testDrag( element, "#draggable2 span", 100, 100, 100, 100 );
 
        // Switch
-       el.draggable( "option", "handle", "span" );
-       TestHelpers.draggable.testDrag(el, el, 50, 50, 0, 0);
-       TestHelpers.draggable.testDrag(el, "#draggable2 span", 100, 100, 100, 100);
+       element.draggable( "option", "handle", "span" );
+       TestHelpers.draggable.testDrag( element, element, 50, 50, 0, 0 );
+       TestHelpers.draggable.testDrag( element, "#draggable2 span", 100, 100, 100, 100 );
 
        // And back
-       el.draggable( "option", "handle", false );
-       TestHelpers.draggable.testDrag(el, el, 50, 50, 50, 50);
-       TestHelpers.draggable.testDrag(el, "#draggable2 span", 100, 100, 100, 100);
-
+       element.draggable( "option", "handle", false );
+       TestHelpers.draggable.testDrag( element, element, 50, 50, 50, 50 );
+       TestHelpers.draggable.testDrag( element, "#draggable2 span", 100, 100, 100, 100 );
 });
 
-test("helper, default, switching after initialization", function() {
+test( "helper, default, switching after initialization", function() {
        expect( 3 );
 
-       var el = $("#draggable1").draggable();
-       TestHelpers.draggable.shouldMove(el);
-
-       el.draggable( "option", "helper", "clone" );
-       TestHelpers.draggable.shouldNotMove(el);
+       var element = $( "#draggable1" ).draggable();
+       TestHelpers.draggable.shouldMove( element );
 
-       el.draggable( "option", "helper", "original" );
-       TestHelpers.draggable.shouldMove(el);
+       element.draggable( "option", "helper", "clone" );
+       TestHelpers.draggable.shouldNotMove( element );
 
+       element.draggable( "option", "helper", "original" );
+       TestHelpers.draggable.shouldMove( element );
 });
 
-test("{ helper: 'clone' }, relative", function() {
+test( "{ helper: 'clone' }, relative", function() {
        expect( 1 );
 
-       var el = $("#draggable1").draggable({ helper: "clone" });
-       TestHelpers.draggable.shouldNotMove(el);
+       var element = $( "#draggable1" ).draggable({ helper: "clone" });
+       TestHelpers.draggable.shouldNotMove( element );
 });
 
-test("{ helper: 'clone' }, absolute", function() {
+test( "{ helper: 'clone' }, absolute", function() {
        expect( 1 );
 
-       var el = $("#draggable2").draggable({ helper: "clone" });
-       TestHelpers.draggable.shouldNotMove(el);
+       var element = $( "#draggable2" ).draggable({ helper: "clone" });
+       TestHelpers.draggable.shouldNotMove( element );
 });
 
-test("{ helper: 'original' }, relative, with scroll offset on parent", function() {
+test( "{ helper: 'original' }, relative, with scroll offset on parent", function() {
        expect( 3 );
 
-       var el = $("#draggable1").draggable({ helper: "original" });
+       var element = $( "#draggable1" ).draggable({ helper: "original" });
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.testScroll(el, "relative");
+       TestHelpers.draggable.testScroll( element, "relative" );
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.testScroll(el, "static");
+       TestHelpers.draggable.testScroll( element, "static" );
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.testScroll(el, "absolute");
+       TestHelpers.draggable.testScroll( element, "absolute" );
 
        TestHelpers.draggable.restoreScroll();
-
 });
 
-test("{ helper: 'original' }, relative, with scroll offset on root", function() {
+test( "{ helper: 'original' }, relative, with scroll offset on root", function() {
        expect( 3 );
 
-       var el = $("#draggable1").draggable({ helper: "original" });
-
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "relative");
+       var element = $( "#draggable1" ).draggable({ helper: "original" });
 
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "static");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "relative" );
 
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "absolute");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "static" );
 
-       TestHelpers.draggable.restoreScroll("root");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "absolute" );
 
+       TestHelpers.draggable.restoreScroll( "root" );
 });
 
-test("{ helper: 'original' }, relative, with scroll offset on root and parent", function() {
-
-       expect(3);
+test( "{ helper: 'original' }, relative, with scroll offset on root and parent", function() {
+       expect( 3 );
 
-       var el = $("#draggable1").draggable({ helper: "original" });
+       var element = $( "#draggable1" ).draggable({ helper: "original" });
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "relative");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "relative" );
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "static");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "static" );
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "absolute");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "absolute" );
 
        TestHelpers.draggable.restoreScroll();
-       TestHelpers.draggable.restoreScroll("root");
-
+       TestHelpers.draggable.restoreScroll( "root" );
 });
 
-test("{ helper: 'original' }, absolute, with scroll offset on parent", function() {
-
-       expect(3);
+test( "{ helper: 'original' }, absolute, with scroll offset on parent", function() {
+       expect( 3 );
 
-       var el = $("#draggable1").css({ position: "absolute", top: 0, left: 0 }).draggable({ helper: "original" });
+       var element = $( "#draggable1" ).css({ position: "absolute", top: 0, left: 0 }).draggable({ helper: "original" });
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.testScroll(el, "relative");
+       TestHelpers.draggable.testScroll( element, "relative" );
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.testScroll(el, "static");
+       TestHelpers.draggable.testScroll( element, "static" );
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.testScroll(el, "absolute");
+       TestHelpers.draggable.testScroll( element, "absolute" );
 
        TestHelpers.draggable.restoreScroll();
-
 });
 
-test("{ helper: 'original' }, absolute, with scroll offset on root", function() {
-
-       expect(3);
-
-       var el = $("#draggable1").css({ position: "absolute", top: 0, left: 0 }).draggable({ helper: "original" });
+test( "{ helper: 'original' }, absolute, with scroll offset on root", function() {
+       expect( 3 );
 
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "relative");
+       var element = $( "#draggable1" ).css({ position: "absolute", top: 0, left: 0 }).draggable({ helper: "original" });
 
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "static");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "relative" );
 
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "absolute");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "static" );
 
-       TestHelpers.draggable.restoreScroll("root");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "absolute" );
 
+       TestHelpers.draggable.restoreScroll( "root" );
 });
 
-test("{ helper: 'original' }, absolute, with scroll offset on root and parent", function() {
-
-       expect(3);
+test( "{ helper: 'original' }, absolute, with scroll offset on root and parent", function() {
+       expect( 3 );
 
-       var el = $("#draggable1").css({ position: "absolute", top: 0, left: 0 }).draggable({ helper: "original" });
+       var element = $( "#draggable1" ).css({ position: "absolute", top: 0, left: 0 }).draggable({ helper: "original" });
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "relative");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "relative" );
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "static");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "static" );
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "absolute");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "absolute" );
 
        TestHelpers.draggable.restoreScroll();
-       TestHelpers.draggable.restoreScroll("root");
-
+       TestHelpers.draggable.restoreScroll( "root" );
 });
 
-test("{ helper: 'original' }, fixed, with scroll offset on parent", function() {
-
-       expect(3);
+test( "{ helper: 'original' }, fixed, with scroll offset on parent", function() {
+       expect( 3 );
 
-       var el = $("#draggable1").css({ position: "fixed", top: 0, left: 0 }).draggable({ helper: "original" });
+       var element = $( "#draggable1" ).css({ position: "fixed", top: 0, left: 0 }).draggable({ helper: "original" });
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.testScroll(el, "relative");
+       TestHelpers.draggable.testScroll( element, "relative" );
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.testScroll(el, "static");
+       TestHelpers.draggable.testScroll( element, "static" );
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.testScroll(el, "absolute");
+       TestHelpers.draggable.testScroll( element, "absolute" );
 
        TestHelpers.draggable.restoreScroll();
-
 });
 
-test("{ helper: 'original' }, fixed, with scroll offset on root", function() {
-
-       expect(3);
+test( "{ helper: 'original' }, fixed, with scroll offset on root", function() {
+       expect( 3 );
 
-       var el = $("#draggable1").css({ position: "fixed", top: 0, left: 0 }).draggable({ helper: "original" });
+       var element = $( "#draggable1" ).css({ position: "fixed", top: 0, left: 0 }).draggable({ helper: "original" });
 
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "relative");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "relative" );
 
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "static");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "static" );
 
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "absolute");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "absolute" );
 
-       TestHelpers.draggable.restoreScroll("root");
+       TestHelpers.draggable.restoreScroll( "root" );
 });
 
-test("{ helper: 'original' }, fixed, with scroll offset on root and parent", function() {
-
-       expect(3);
+test( "{ helper: 'original' }, fixed, with scroll offset on root and parent", function() {
+       expect( 3 );
 
-       var el = $("#draggable1").css({ position: "fixed", top: 0, left: 0 }).draggable({ helper: "original" });
+       var element = $( "#draggable1" ).css({ position: "fixed", top: 0, left: 0 }).draggable({ helper: "original" });
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "relative");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "relative" );
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "static");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "static" );
 
        TestHelpers.draggable.setScroll();
-       TestHelpers.draggable.setScroll("root");
-       TestHelpers.draggable.testScroll(el, "absolute");
+       TestHelpers.draggable.setScroll( "root" );
+       TestHelpers.draggable.testScroll( element, "absolute" );
 
        TestHelpers.draggable.restoreScroll();
-       TestHelpers.draggable.restoreScroll("root");
-
+       TestHelpers.draggable.restoreScroll( "root" );
 });
 
-test("{ helper: 'clone' }, absolute", function() {
-
-       expect(1);
+test( "{ helper: 'clone' }, absolute", function() {
+       expect( 1 );
 
        var helperOffset = null,
-               origOffset = $("#draggable1").offset(),
-               el = $("#draggable1").draggable({ helper: "clone", drag: function(event, ui) {
+               origOffset = $( "#draggable1" ).offset(),
+               element = $( "#draggable1" ).draggable({ helper: "clone", drag: function( event, ui) {
                        helperOffset = ui.helper.offset();
                } });
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                dx: 1,
                dy: 1
        });
-       deepEqual({ top: helperOffset.top-1, left: helperOffset.left-1 }, origOffset, "dragged[1, 1] ");
-
+       deepEqual({ top: helperOffset.top - 1, left: helperOffset.left - 1 }, origOffset, "dragged[1, 1]" );
 });
 
-test("{ helper: 'clone' }, absolute with scroll offset on parent", function() {
-
-       expect(3);
+test( "{ helper: 'clone' }, absolute with scroll offset on parent", function() {
+       expect( 3 );
 
        TestHelpers.draggable.setScroll();
        var helperOffset = null,
                origOffset = null,
-               el = $("#draggable1").draggable({ helper: "clone", drag: function(event, ui) {
-                       helperOffset = ui.helper.offset();
-               } });
+               element = $( "#draggable1" ).draggable({
+                       helper: "clone",
+                       drag: function( event, ui) {
+                               helperOffset = ui.helper.offset();
+                       }
+               });
 
-       $("#main").css("position", "relative");
-       origOffset = $("#draggable1").offset();
-       el.simulate( "drag", {
+       $( "#main" ).css( "position", "relative" );
+       origOffset = $( "#draggable1" ).offset();
+       element.simulate( "drag", {
                dx: 1,
                dy: 1
        });
-       deepEqual({ top: helperOffset.top-1, left: helperOffset.left-1 }, origOffset, "dragged[1, 1] ");
+       deepEqual({ top: helperOffset.top - 1, left: helperOffset.left - 1 }, origOffset, "dragged[1, 1]" );
 
-       $("#main").css("position", "static");
-       origOffset = $("#draggable1").offset();
-       el.simulate( "drag", {
+       $( "#main" ).css( "position", "static" );
+       origOffset = $( "#draggable1" ).offset();
+       element.simulate( "drag", {
                dx: 1,
                dy: 1
        });
-       deepEqual({ top: helperOffset.top-1, left: helperOffset.left-1 }, origOffset, "dragged[1, 1] ");
+       deepEqual({ top: helperOffset.top - 1, left: helperOffset.left - 1 }, origOffset, "dragged[1, 1]" );
 
-       $("#main").css("position", "absolute");
-       origOffset = $("#draggable1").offset();
-       el.simulate( "drag", {
+       $( "#main" ).css( "position", "absolute" );
+       origOffset = $( "#draggable1" ).offset();
+       element.simulate( "drag", {
                dx: 1,
                dy: 1
        });
-       deepEqual({ top: helperOffset.top-1, left: helperOffset.left-1 }, origOffset, "dragged[1, 1] ");
+       deepEqual({ top: helperOffset.top - 1, left: helperOffset.left - 1 }, origOffset, "dragged[1, 1]" );
 
        TestHelpers.draggable.restoreScroll();
-
 });
 
-test("{ helper: 'clone' }, absolute with scroll offset on root", function() {
-
-       expect(3);
+test( "{ helper: 'clone' }, absolute with scroll offset on root", function() {
+       expect( 3 );
 
-       TestHelpers.draggable.setScroll("root");
+       TestHelpers.draggable.setScroll( "root" );
        var helperOffset = null,
                origOffset = null,
-               el = $("#draggable1").draggable({ helper: "clone", drag: function(event, ui) {
-                       helperOffset = ui.helper.offset();
-               } });
+               element = $( "#draggable1" ).draggable({
+                       helper: "clone",
+                       drag: function( event, ui) {
+                               helperOffset = ui.helper.offset();
+                       }
+               });
 
-       $("#main").css("position", "relative");
-       origOffset = $("#draggable1").offset();
-       el.simulate( "drag", {
+       $( "#main" ).css( "position", "relative" );
+       origOffset = $( "#draggable1" ).offset();
+       element.simulate( "drag", {
                dx: 1,
                dy: 1
        });
-       deepEqual({ top: helperOffset.top-1, left: helperOffset.left-1 }, origOffset, "dragged[1, 1] ");
+       deepEqual({ top: helperOffset.top - 1, left: helperOffset.left - 1 }, origOffset, "dragged[1, 1]" );
 
-       $("#main").css("position", "static");
-       origOffset = $("#draggable1").offset();
-       el.simulate( "drag", {
+       $( "#main" ).css( "position", "static" );
+       origOffset = $( "#draggable1" ).offset();
+       element.simulate( "drag", {
                dx: 1,
                dy: 1
        });
-       deepEqual({ top: helperOffset.top-1, left: helperOffset.left-1 }, origOffset, "dragged[1, 1] ");
+       deepEqual({ top: helperOffset.top - 1, left: helperOffset.left - 1 }, origOffset, "dragged[1, 1]" );
 
-       $("#main").css("position", "absolute");
-       origOffset = $("#draggable1").offset();
-       el.simulate( "drag", {
+       $( "#main" ).css( "position", "absolute" );
+       origOffset = $( "#draggable1" ).offset();
+       element.simulate( "drag", {
                dx: 1,
                dy: 1
        });
-       deepEqual({ top: helperOffset.top-1, left: helperOffset.left-1 }, origOffset, "dragged[1, 1] ");
-
-       TestHelpers.draggable.restoreScroll("root");
+       deepEqual({ top: helperOffset.top - 1, left: helperOffset.left - 1 }, origOffset, "dragged[1, 1]" );
 
+       TestHelpers.draggable.restoreScroll( "root" );
 });
 
-test("{ helper: 'clone' }, absolute with scroll offset on root and parent", function() {
-
-       expect(3);
+test( "{ helper: 'clone' }, absolute with scroll offset on root and parent", function() {
+       expect( 3 );
 
-       TestHelpers.draggable.setScroll("root");
+       TestHelpers.draggable.setScroll( "root" );
        TestHelpers.draggable.setScroll();
 
        var helperOffset = null,
                origOffset = null,
-               el = $("#draggable1").draggable({ helper: "clone", drag: function(event, ui) {
-                       helperOffset = ui.helper.offset();
-               } });
+               element = $( "#draggable1" ).draggable({
+                       helper: "clone",
+                       drag: function( event, ui) {
+                               helperOffset = ui.helper.offset();
+                       }
+               });
 
-       $("#main").css("position", "relative");
-       origOffset = $("#draggable1").offset();
-       el.simulate( "drag", {
+       $( "#main" ).css( "position", "relative" );
+       origOffset = $( "#draggable1" ).offset();
+       element.simulate( "drag", {
                dx: 1,
                dy: 1
        });
-       deepEqual({ top: helperOffset.top-1, left: helperOffset.left-1 }, origOffset, "dragged[1, 1] ");
+       deepEqual({ top: helperOffset.top - 1, left: helperOffset.left - 1 }, origOffset, "dragged[1, 1]" );
 
-       $("#main").css("position", "static");
-       origOffset = $("#draggable1").offset();
-       el.simulate( "drag", {
+       $( "#main" ).css( "position", "static" );
+       origOffset = $( "#draggable1" ).offset();
+       element.simulate( "drag", {
                dx: 1,
                dy: 1
        });
-       deepEqual({ top: helperOffset.top-1, left: helperOffset.left-1 }, origOffset, "dragged[1, 1] ");
+       deepEqual({ top: helperOffset.top - 1, left: helperOffset.left - 1 }, origOffset, "dragged[1, 1]" );
 
-       $("#main").css("position", "absolute");
-       origOffset = $("#draggable1").offset();
-       el.simulate( "drag", {
+       $( "#main" ).css( "position", "absolute" );
+       origOffset = $( "#draggable1" ).offset();
+       element.simulate( "drag", {
                dx: 1,
                dy: 1
        });
-       deepEqual({ top: helperOffset.top-1, left: helperOffset.left-1 }, origOffset, "dragged[1, 1] ");
+       deepEqual({ top: helperOffset.top - 1, left: helperOffset.left - 1 }, origOffset, "dragged[1, 1]" );
 
-       TestHelpers.draggable.restoreScroll("root");
+       TestHelpers.draggable.restoreScroll( "root" );
        TestHelpers.draggable.restoreScroll();
-
 });
 
-test("{ opacity: 0.5 }", function() {
-
-       expect(1);
+test( "{ opacity: 0.5 }", function() {
+       expect( 1 );
 
        var opacity = null,
-               el = $("#draggable2").draggable({
+               element = $( "#draggable2" ).draggable({
                        opacity: 0.5,
                        start: function() {
-                               opacity = $(this).css("opacity");
+                               opacity = $(this).css( "opacity" );
                        }
                });
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                dx: -1,
                dy: -1
        });
 
-       equal(opacity, 0.5, "start callback: opacity is");
-
+       equal( opacity, 0.5, "start callback: opacity is" );
 });
 
-test("opacity, default, switching after initialization", function() {
-
-       expect(3);
+test( "opacity, default, switching after initialization", function() {
+       expect( 3 );
 
        var opacity = null,
-               el = $("#draggable2").draggable({
+               element = $( "#draggable2" ).draggable({
                        start: function() {
-                               opacity = $(this).css("opacity");
+                               opacity = $(this).css( "opacity" );
                        }
                });
 
-       TestHelpers.draggable.move( el, 1, 1 );
+       TestHelpers.draggable.move( element, 1, 1 );
        equal( opacity, 1 );
 
-       el.draggable( "option", "opacity", 0.5 );
-       TestHelpers.draggable.move( el, 2, 1 );
+       element.draggable( "option", "opacity", 0.5 );
+       TestHelpers.draggable.move( element, 2, 1 );
        equal( opacity, 0.5 );
 
-       el.draggable( "option", "opacity", false );
-       TestHelpers.draggable.move( el, 3, 1 );
+       element.draggable( "option", "opacity", false );
+       TestHelpers.draggable.move( element, 3, 1 );
        equal( opacity, 1 );
-
 });
 
 asyncTest( "revert and revertDuration", function() {
        expect( 4 );
 
-       var el = $( "#draggable2" ).draggable({
+       var element = $( "#draggable2" ).draggable({
                revert: true,
                revertDuration: 0
        });
-       TestHelpers.draggable.shouldNotMove( el, "revert: true, revertDuration: 0 should revert immediately" );
+       TestHelpers.draggable.shouldNotMove( element, "revert: true, revertDuration: 0 should revert immediately" );
 
        $( "#draggable2" ).draggable( "option", "revert", "invalid" );
-       TestHelpers.draggable.shouldNotMove( el, "revert: invalid, revertDuration: 0 should revert immediately" );
+       TestHelpers.draggable.shouldNotMove( element, "revert: invalid, revertDuration: 0 should revert immediately" );
 
        $( "#draggable2" ).draggable( "option", "revert", false );
-       TestHelpers.draggable.shouldMove( el, "revert: false should allow movement" );
+       TestHelpers.draggable.shouldMove( element, "revert: false should allow movement" );
 
        $( "#draggable2" ).draggable( "option", {
                revert: true,
@@ -1061,8 +1018,9 @@ asyncTest( "revert and revertDuration", function() {
                        start();
                }
        });
+
        // animation are async, so test for it asynchronously
-       TestHelpers.draggable.move( el, 50, 50 );
+       TestHelpers.draggable.move( element, 50, 50 );
        setTimeout( function() {
                ok( $( "#draggable2" ).is( ":animated" ), "revert: true with revertDuration should animate" );
        });
@@ -1071,20 +1029,20 @@ asyncTest( "revert and revertDuration", function() {
 test( "revert: valid", function() {
        expect( 1 );
 
-       var el = $( "#draggable2" ).draggable({
+       var element = $( "#draggable2" ).draggable({
                        revert: "valid",
                        revertDuration: 0
                });
 
        $( "#droppable" ).droppable();
 
-       TestHelpers.draggable.testDrag( el, el, 100, 100, 0, 0, "revert: valid reverts when dropped on a droppable" );
+       TestHelpers.draggable.testDrag( element, element, 100, 100, 0, 0, "revert: valid reverts when dropped on a droppable" );
 });
 
 test( "scope", function() {
        expect( 2 );
 
-       var el = $( "#draggable2" ).draggable({
+       var element = $( "#draggable2" ).draggable({
                scope: "tasks",
                revert: "valid",
                revertDuration: 0
@@ -1092,27 +1050,27 @@ test( "scope", function() {
 
        $( "#droppable" ).droppable({ scope: "tasks" });
 
-       TestHelpers.draggable.testDrag( el, el, 100, 100, 0, 0, "revert: valid reverts when dropped on a droppable" );
+       TestHelpers.draggable.testDrag( element, element, 100, 100, 0, 0, "revert: valid reverts when dropped on a droppable" );
 
-       $( "#droppable" ).droppable("destroy").droppable({ scope: "nottasks" });
+       $( "#droppable" ).droppable( "destroy" ).droppable({ scope: "nottasks" });
 
-       TestHelpers.draggable.testDrag( el, el, 100, 100, 100, 100, "revert: valid reverts when dropped on a droppable" );
+       TestHelpers.draggable.testDrag( element, element, 100, 100, 100, 100, "revert: valid reverts when dropped on a droppable" );
 });
 
 test( "scroll, scrollSensitivity, and scrollSpeed", function() {
        expect( 2 );
 
        var viewportHeight = $( window ).height(),
-               el = $( "#draggable1" ).draggable({ scroll: true }),
-               scrollSensitivity = el.draggable( "option", "scrollSensitivity" ),
-               scrollSpeed = el.draggable( "option", "scrollSpeed" );
+               element = $( "#draggable1" ).draggable({ scroll: true }),
+               scrollSensitivity = element.draggable( "option", "scrollSensitivity" ),
+               scrollSpeed = element.draggable( "option", "scrollSpeed" );
 
-       el.offset({
+       element.offset({
                top: viewportHeight - scrollSensitivity - 1,
                left: 1
        });
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                dx: 1,
                y: viewportHeight - scrollSensitivity - 1,
                moves: 1
@@ -1120,14 +1078,14 @@ test( "scroll, scrollSensitivity, and scrollSpeed", function() {
 
        ok( $( window ).scrollTop() === 0, "scroll: true doesn't scroll when the element is dragged outside of scrollSensitivity" );
 
-       el.draggable( "option", "scrollSensitivity", scrollSensitivity + 10 );
+       element.draggable( "option", "scrollSensitivity", scrollSensitivity + 10 );
 
-       el.offset({
+       element.offset({
                top: viewportHeight - scrollSensitivity - 1,
                left: 1
        });
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                dx: 1,
                y: viewportHeight - scrollSensitivity - 1,
                moves: 1
@@ -1143,119 +1101,119 @@ test( "snap, snapMode, and snapTolerance", function() {
 
        var newX, newY,
                snapTolerance = 15,
-               el = $( "#draggable1" ).draggable({
+               element = $( "#draggable1" ).draggable({
                        snap: true,
                        snapMode: "both",
                        snapTolerance: snapTolerance
                }),
                element2 = $( "#draggable2" ).draggable();
 
-       el.offset({
+       element.offset({
                top: 1,
                left: 1
        });
 
-       newX = element2.offset().left - el.outerWidth() - snapTolerance - 1;
+       newX = element2.offset().left - element.outerWidth() - snapTolerance - 1;
        newY = element2.offset().top;
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                handle: "corner",
                x: newX,
                y: newY,
                moves: 1
        });
 
-       deepEqual( el.offset(), { top: newY, left: newX }, "doesn't snap outside the snapTolerance" );
+       deepEqual( element.offset(), { top: newY, left: newX }, "doesn't snap outside the snapTolerance" );
 
        newX += 2;
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                handle: "corner",
                x: newX,
                y: newY,
                moves: 1
        });
 
-       notDeepEqual( el.offset(), { top: newY, left: newX }, "snaps inside the snapTolerance" );
+       notDeepEqual( element.offset(), { top: newY, left: newX }, "snaps inside the snapTolerance" );
 
-       el.draggable( "option", "snap", "#draggable2" );
+       element.draggable( "option", "snap", "#draggable2" );
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                handle: "corner",
                x: newX,
                y: newY,
                moves: 1
        });
 
-       notDeepEqual( el.offset(), { top: newY, left: newX }, "snaps based on selector" );
+       notDeepEqual( element.offset(), { top: newY, left: newX }, "snaps based on selector" );
 
-       el.draggable( "option", "snap", "#draggable3" );
+       element.draggable( "option", "snap", "#draggable3" );
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                handle: "corner",
                x: newX,
                y: newY,
                moves: 1
        });
 
-       deepEqual( el.offset(), { top: newY, left: newX }, "doesn't snap based on invalid selector" );
+       deepEqual( element.offset(), { top: newY, left: newX }, "doesn't snap based on invalid selector" );
 
-       el.draggable( "option", "snap", true );
-       el.draggable( "option", "snapTolerance", snapTolerance - 2 );
-       el.simulate( "drag", {
+       element.draggable( "option", "snap", true );
+       element.draggable( "option", "snapTolerance", snapTolerance - 2 );
+       element.simulate( "drag", {
                handle: "corner",
                x: newX,
                y: newY,
                moves: 1
        });
 
-       deepEqual( el.offset(), { top: newY, left: newX }, "doesn't snap outside the modified snapTolerance" );
+       deepEqual( element.offset(), { top: newY, left: newX }, "doesn't snap outside the modified snapTolerance" );
 
-       el.draggable( "option", "snapTolerance", snapTolerance );
-       el.draggable( "option", "snapMode", "inner" );
+       element.draggable( "option", "snapTolerance", snapTolerance );
+       element.draggable( "option", "snapMode", "inner" );
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                handle: "corner",
                x: newX,
                y: newY,
                moves: 1
        });
 
-       deepEqual( el.offset(), { top: newY, left: newX }, "doesn't snap inside the outer snapTolerance area when snapMode is inner" );
+       deepEqual( element.offset(), { top: newY, left: newX }, "doesn't snap inside the outer snapTolerance area when snapMode is inner" );
 
        newX = element2.offset().left - snapTolerance - 1;
        newY = element2.offset().top;
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                handle: "corner",
                x: newX,
                y: newY,
                moves: 1
        });
 
-       deepEqual( el.offset(), { top: newY, left: newX }, "doesn't snap inside the outer snapTolerance area when snapMode is inner" );
+       deepEqual( element.offset(), { top: newY, left: newX }, "doesn't snap inside the outer snapTolerance area when snapMode is inner" );
 
        newX++;
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                handle: "corner",
                x: newX,
                y: newY,
                moves: 1
        });
 
-       notDeepEqual( el.offset(), { top: newY, left: newX }, "snaps inside the inner snapTolerance area when snapMode is inner" );
+       notDeepEqual( element.offset(), { top: newY, left: newX }, "snaps inside the inner snapTolerance area when snapMode is inner" );
 
-       el.draggable( "option", "snapMode", "outer" );
+       element.draggable( "option", "snapMode", "outer" );
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                handle: "corner",
                x: newX,
                y: newY,
                moves: 1
        });
 
-       deepEqual( el.offset(), { top: newY, left: newX }, "doesn't snap on the inner snapTolerance area when snapMode is outer" );
+       deepEqual( element.offset(), { top: newY, left: newX }, "doesn't snap on the inner snapTolerance area when snapMode is outer" );
 });
 
 test( "stack", function() {
@@ -1275,51 +1233,51 @@ test( "stack", function() {
        equal( element2.css( "zIndex" ), "3", "stack increments zIndex correctly" );
 });
 
-test("{ zIndex: 10 }", function() {
-       expect(1);
+test( "{ zIndex: 10 }", function() {
+       expect( 1 );
 
        var actual,
                expected = 10,
-               el = $("#draggable2").draggable({
+               element = $( "#draggable2" ).draggable({
                        zIndex: expected,
                        start: function() {
-                               actual = $(this).css("zIndex");
+                               actual = $(this).css( "zIndex" );
                        }
                });
 
-       el.simulate( "drag", {
+       element.simulate( "drag", {
                dx: -1,
                dy: -1
        });
 
-       equal(actual, expected, "start callback: zIndex is");
+       equal( actual, expected, "start callback: zIndex is" );
 
 });
 
-test("zIndex, default, switching after initialization", function() {
+test( "zIndex, default, switching after initialization", function() {
 
-       expect(3);
+       expect( 3 );
 
        var zindex = null,
-               el = $("#draggable2").draggable({
+               element = $( "#draggable2" ).draggable({
                        start: function() {
-                               zindex = $(this).css("z-index");
+                               zindex = $(this).css( "z-index" );
                        }
                });
 
-       el.css( "z-index", 1 );
+       element.css( "z-index", 1 );
 
-       TestHelpers.draggable.move( el, 1, 1 );
+       TestHelpers.draggable.move( element, 1, 1 );
        equal( zindex, 1 );
 
-       el.draggable( "option", "zIndex", 5 );
-       TestHelpers.draggable.move( el, 2, 1 );
+       element.draggable( "option", "zIndex", 5 );
+       TestHelpers.draggable.move( element, 2, 1 );
        equal( zindex, 5 );
 
-       el.draggable( "option", "zIndex", false );
-       TestHelpers.draggable.move( el, 3, 1 );
+       element.draggable( "option", "zIndex", false );
+       TestHelpers.draggable.move( element, 3, 1 );
        equal( zindex, 1 );
 
 });
 
-})(jQuery);
+})( jQuery );