diff options
| author | Mike Sherov <mike.sherov@gmail.com> | 2012-10-23 09:36:42 -0500 |
|---|---|---|
| committer | Corey Frang <gnarf@gnarf.net> | 2012-10-23 09:37:46 -0500 |
| commit | e1230997aa14dae6f35326c8ba20bfe2428507c2 (patch) | |
| tree | f00984f5fbbf12f175c0994c497513a2d8e8ee26 /tests/unit/effects | |
| parent | 7af1ec727bcca8367e804cea77b9dd238b1c0d69 (diff) | |
| download | jquery-ui-e1230997aa14dae6f35326c8ba20bfe2428507c2.tar.gz jquery-ui-e1230997aa14dae6f35326c8ba20bfe2428507c2.zip | |
Build: Enable "unused" option in jshint - Remove unused variables from codebase. - Closes gh-788
Squashed commit of the following:
commit 7f19f92c646f180bc067bb24123175251a64a9d6
Author: Mike Sherov <mike.sherov@gmail.com>
Date: Tue Oct 23 10:34:28 2012 -0400
put back in fake args for signatures that we want to keep
commit 257505a9e69da0c53e3a989dab87a13112045a29
Author: Mike Sherov <mike.sherov@gmail.com>
Date: Tue Oct 23 08:10:20 2012 -0400
changes per @scott_gonzalez
commit 12725480cb58e70865e5aa6e735009b6b035c8f3
Author: Mike Sherov <mike.sherov@gmail.com>
Date: Mon Oct 22 22:54:05 2012 -0400
clean up unused vars in ui directory
commit 563595e7aee5d4a5c096b2d1de655abdf920aacd
Author: Mike Sherov <mike.sherov@gmail.com>
Date: Mon Oct 22 22:37:42 2012 -0400
clean up unused vars in grunt and tests
Diffstat (limited to 'tests/unit/effects')
| -rw-r--r-- | tests/unit/effects/effects_core.js | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/tests/unit/effects/effects_core.js b/tests/unit/effects/effects_core.js index c4e5c24f9..26390b7d3 100644 --- a/tests/unit/effects/effects_core.js +++ b/tests/unit/effects/effects_core.js @@ -12,10 +12,7 @@ function notPresent( value, array, message ) { var minDuration = 15, // duration is used for "long" animates where we plan on testing properties during animation - duration = 200, - - // mid is used for testing in the "middle" of the "duration" animations - mid = duration / 2; + duration = 200; module( "effects.core" ); @@ -58,8 +55,7 @@ test( "createWrapper and removeWrapper retain focused elements (#7595)", functio module( "effects.core: animateClass" ); asyncTest( "animateClass works with borderStyle", function() { - var test = $("div.animateClass"), - count = 0; + var test = $("div.animateClass"); expect(3); test.toggleClass("testAddBorder", minDuration, function() { test.toggleClass("testAddBorder", minDuration, function() { @@ -73,8 +69,8 @@ asyncTest( "animateClass works with borderStyle", function() { asyncTest( "animateClass works with colors", function() { var test = $("div.animateClass"), - count = 0, oldStep = jQuery.fx.step.backgroundColor; + expect(2); // we want to catch the first frame of animation @@ -102,7 +98,7 @@ asyncTest( "animateClass works with colors", function() { asyncTest( "animateClass calls step option", 1, function() { var test = jQuery( "div.animateClass" ), - step = function( fx ) { + step = function() { ok( true, "Step Function Called" ); test.stop(); start(); @@ -216,7 +212,7 @@ $.each( $.effects.effect, function( effect ) { equal( hidden.css("display"), "block", "Hidden is shown after .show(\"" +effect+ "\", time)" ); })).queue( queueTest() ).hide( effect, minDuration, queueTest(function() { equal( hidden.css("display"), "none", "Back to hidden after .hide(\"" +effect+ "\", time)" ); - })).queue( queueTest(function(next) { + })).queue( queueTest(function() { deepEqual( hidden.queue(), ["inprogress"], "Only the inprogress sentinel remains"); start(); })); |