]> source.dussan.org Git - jquery-ui.git/commitdiff
Tests: Lint.
authorScott González <scott.gonzalez@gmail.com>
Thu, 19 Apr 2012 02:46:39 +0000 (22:46 -0400)
committerScott González <scott.gonzalez@gmail.com>
Thu, 19 Apr 2012 02:46:39 +0000 (22:46 -0400)
tests/unit/progressbar/progressbar_events.js
tests/unit/resizable/resizable_methods.js
tests/unit/selectable/selectable_methods.js
tests/unit/selectable/selectable_options.js
tests/unit/slider/slider_methods.js
tests/unit/slider/slider_options.js
tests/unit/sortable/sortable_events.js
tests/unit/sortable/sortable_methods.js
tests/unit/sortable/sortable_tickets.js

index 8d7b8868dcab573edbf4d37eae590ca40334ea32..f3713171b45aeb5eaa6825b9151838b083769f77 100644 (file)
@@ -15,7 +15,7 @@ test("create", function() {
                change: function() {
                        ok(false, 'create() has triggered change()');
                }
-       })
+       });
 });
 
 test("change", function() {
index 03e91d44013bd9164f235d215b09db3361454a9c..2b4c4256cb164fc8886d058cf4b1a7cbada9823e 100644 (file)
@@ -20,7 +20,7 @@ test("init", function() {
        $('<div></div>').resizable().resizable("foo").remove();
        ok(true, 'arbitrary method called after init');
 
-       el = $('<div></div>').resizable()
+       el = $('<div></div>').resizable();
        var foo = el.resizable("option", "foo");
        el.remove();
        ok(true, 'arbitrary option getter after init');
index 32dfe00ed1780fed98b440811f86dd371e3278e9..dbc88f26982f865062f6e2b785e612f5ccea08d2 100644 (file)
@@ -20,7 +20,7 @@ test("init", function() {
        $("<div></div>").selectable().selectable("foo").remove();
        ok(true, 'arbitrary method called after init');
 
-       el = $("<div></div>").selectable()
+       el = $("<div></div>").selectable();
        var foo = el.selectable("option", "foo");
        el.remove();
        ok(true, 'arbitrary option getter after init');
@@ -49,7 +49,8 @@ test("destroy", function() {
 
 test("enable", function() {
        expect(3);
-       var fired = false;
+       var expected, actual,
+               fired = false;
 
        el = $("#selectable1");
        el.selectable({
@@ -63,14 +64,15 @@ test("enable", function() {
        equal(fired, true, "start fired");
        el.selectable("destroy");
 
-       var expected = $('<div></div>').selectable(),
-               actual = expected.selectable('enable');
+       expected = $('<div></div>').selectable();
+       actual = expected.selectable('enable');
        equal(actual, expected, 'enable is chainable');
 });
 
 test("disable", function() {
        expect(3);
-       var fired = false;
+       var expected, actual,
+               fired = false;
 
        el = $("#selectable1");
        el.selectable({
@@ -85,8 +87,8 @@ test("disable", function() {
        equal(fired, false, "start fired");
        el.selectable("destroy");
 
-       var expected = $('<div></div>').selectable(),
-               actual = expected.selectable('disable');
+       expected = $('<div></div>').selectable();
+       actual = expected.selectable('disable');
        equal(actual, expected, 'disable is chainable');
 });
 
index 2bc44e6e00dd71e642c2d31d4bf2fafc75a57055..be9fdf5feb6b3cce1479c1d3583917a5258f6616 100644 (file)
@@ -8,7 +8,7 @@ module("selectable: options");
 test("autoRefresh", function() {
        expect(3);
        el = $("#selectable1");
-       var actual, sel = $("*", el), selected = function() { actual += 1 };
+       var actual, sel = $("*", el), selected = function() { actual += 1; };
 
        actual = 0;
        el = $("#selectable1").selectable({ autoRefresh: false, selected: selected });
@@ -33,12 +33,12 @@ test("autoRefresh", function() {
 test("filter", function() {
        expect(2);
        el = $("#selectable1");
-       var actual, sel = $("*", el), selected = function() { actual += 1 };
+       var actual, sel = $("*", el), selected = function() { actual += 1; };
 
        actual = 0;
        el = $("#selectable1").selectable({ filter: '.special', selected: selected });
        drag(1000, 1000);
-       ok(sel.length != 1, "this test assumes more than 1 selectee");
+       ok(sel.length !== 1, "this test assumes more than 1 selectee");
        equal(actual, 1);
        el.selectable("destroy");
 });
index b462220025bf460d2f5a66c903d9b2630e9d7bdb..79f80e8683af36411f9aada25fc6986dde6c1efb 100644 (file)
@@ -17,8 +17,8 @@ test("init", function() {
        $('<div></div>').slider().remove();
        ok(true, '.slider() called on disconnected DOMElement');
 
-       var el = $('<div></div>').slider();
-       var foo = el.slider("option", "foo");
+       var el = $('<div></div>').slider(),
+               foo = el.slider("option", "foo");
        el.remove();
        ok(true, 'arbitrary option getter after init');
 
@@ -42,11 +42,12 @@ test("destroy", function() {
 });
 
 test("enable", function() {
-       var expected = $('<div></div>').slider(),
+       var el,
+               expected = $('<div></div>').slider(),
                actual = expected.slider('enable');
        equal(actual, expected, 'enable is chainable');
 
-       var el = $('<div></div>').slider({ disabled: true });
+       el = $('<div></div>').slider({ disabled: true });
        ok(el.hasClass('ui-disabled'), 'slider has ui-disabled class before enable method call');
        ok(el.hasClass('ui-slider-disabled'), 'slider has ui-slider-disabled class before enable method call');
        el.slider('enable');
@@ -55,11 +56,12 @@ test("enable", function() {
 });
 
 test("disable", function() {
-       var expected = $('<div></div>').slider(),
+       var el,
+               expected = $('<div></div>').slider(),
                actual = expected.slider('disable');
        equal(actual, expected, 'disable is chainable');
 
-       var el = $('<div></div>').slider({ disabled: false });
+       el = $('<div></div>').slider({ disabled: false });
        ok(!el.hasClass('ui-disabled'), 'slider does not have ui-disabled class before disabled method call');
        ok(!el.hasClass('ui-slider-disabled'), 'slider does not have ui-slider-disabled class before disable method call');
        el.slider('disable');
index 7d9899284bb7cfd103d6dc585289ee6e212130d1..09067a3e9630708355360f2f1b73d646d4b8c3a7 100644 (file)
@@ -23,8 +23,8 @@ test("max", function() {
        };
 
        el.slider(options);
-       ok(el.slider("option", "value") == options.value, "value option is not contained by max");
-       ok(el.slider("value") == options.max, "value method is contained by max");
+       ok(el.slider("option", "value") === options.value, "value option is not contained by max");
+       ok(el.slider("value") === options.max, "value method is contained by max");
        el.slider('destroy');
 
 });
@@ -41,8 +41,8 @@ test("min", function() {
        };
 
        el.slider(options);
-       ok(el.slider("option", "value") == options.value, "value option is not contained by min");
-       ok(el.slider("value") == options.min, "value method is contained by min");
+       ok(el.slider("option", "value") === options.value, "value option is not contained by min");
+       ok(el.slider("value") === options.min, "value method is contained by min");
        el.slider('destroy');
 
 });
@@ -73,7 +73,7 @@ test("orientation", function() {
                value: -1
        };
 
-       var percentVal = (options.value - options.min) / (options.max - options.min) * 100;
+       percentVal = (options.value - options.min) / (options.max - options.min) * 100;
 
        el.slider(options).slider("option", "orientation", "vertical");
        ok(el.is('.ui-slider-vertical'), "vertical slider has class .ui-slider-vertical");
index cb4ff4ff3506d8c73aa6becd7347f354d981d53a..6ed54e1b6c5ff2ff7593a04e642ce8a44cd33351 100644 (file)
@@ -6,13 +6,13 @@
 module("sortable: events");
 
 test("start", function() {
-       
+
        var hash;
        $("#sortable")
                .sortable({ start: function(e, ui) { hash = ui; } })
                .find('li:eq(0)').simulate("drag", { dx: 0, dy: 10 });
-       
-       ok(hash, 'start event triggered');      
+
+       ok(hash, 'start event triggered');
        ok(hash.helper, 'UI hash includes: helper');
        ok(hash.placeholder, 'UI hash includes: placeholder');
        ok(hash.position && (hash.position.top && hash.position.left), 'UI hash includes: position');
@@ -20,104 +20,104 @@ test("start", function() {
        ok(hash.item, 'UI hash includes: item');
        ok(!hash.sender, 'UI hash does not include: sender');
 
-                       
+
 });
 
 test("sort", function() {
-       
+
        var hash;
        $("#sortable")
                .sortable({ sort: function(e, ui) { hash = ui; } })
                .find('li:eq(0)').simulate("drag", { dx: 0, dy: 10 });
-       
-       ok(hash, 'sort event triggered');       
+
+       ok(hash, 'sort event triggered');
        ok(hash.helper, 'UI hash includes: helper');
        ok(hash.placeholder, 'UI hash includes: placeholder');
        ok(hash.position && (hash.position.top && hash.position.left), 'UI hash includes: position');
        ok(hash.offset && (hash.offset.top && hash.offset.left), 'UI hash includes: offset');
        ok(hash.item, 'UI hash includes: item');
        ok(!hash.sender, 'UI hash does not include: sender');
-       
+
 });
 
 test("change", function() {
-       
+
        var hash;
        $("#sortable")
                .sortable({ change: function(e, ui) { hash = ui; } })
                .find('li:eq(0)').simulate("drag", { dx: 1, dy: 1 });
-       
+
        ok(!hash, '1px drag, change event should not be triggered');
-       
+
        $("#sortable")
                .sortable({ change: function(e, ui) { hash = ui; } })
-               .find('li:eq(0)').simulate("drag", { dx: 0, dy: 20 });  
-               
-       ok(hash, 'change event triggered');     
+               .find('li:eq(0)').simulate("drag", { dx: 0, dy: 20 });
+
+       ok(hash, 'change event triggered');
        ok(hash.helper, 'UI hash includes: helper');
        ok(hash.placeholder, 'UI hash includes: placeholder');
        ok(hash.position && (hash.position.top && hash.position.left), 'UI hash includes: position');
        ok(hash.offset && (hash.offset.top && hash.offset.left), 'UI hash includes: offset');
        ok(hash.item, 'UI hash includes: item');
        ok(!hash.sender, 'UI hash does not include: sender');
-       
+
 });
 
 test("beforeStop", function() {
-       
+
        var hash;
        $("#sortable")
                .sortable({ beforeStop: function(e, ui) { hash = ui; } })
-               .find('li:eq(0)').simulate("drag", { dx: 0, dy: 20 });  
-               
-       ok(hash, 'beforeStop event triggered'); 
+               .find('li:eq(0)').simulate("drag", { dx: 0, dy: 20 });
+
+       ok(hash, 'beforeStop event triggered');
        ok(hash.helper, 'UI hash includes: helper');
        ok(hash.placeholder, 'UI hash includes: placeholder');
        ok(hash.position && (hash.position.top && hash.position.left), 'UI hash includes: position');
        ok(hash.offset && (hash.offset.top && hash.offset.left), 'UI hash includes: offset');
        ok(hash.item, 'UI hash includes: item');
        ok(!hash.sender, 'UI hash does not include: sender');
-       
+
 });
 
 test("stop", function() {
-       
+
        var hash;
        $("#sortable")
                .sortable({ stop: function(e, ui) { hash = ui; } })
-               .find('li:eq(0)').simulate("drag", { dx: 0, dy: 20 });  
-               
-       ok(hash, 'stop event triggered');       
+               .find('li:eq(0)').simulate("drag", { dx: 0, dy: 20 });
+
+       ok(hash, 'stop event triggered');
        ok(!hash.helper, 'UI should not include: helper');
        ok(hash.placeholder, 'UI hash includes: placeholder');
        ok(hash.position && (hash.position.top && hash.position.left), 'UI hash includes: position');
        ok(hash.offset && (hash.offset.top && hash.offset.left), 'UI hash includes: offset');
        ok(hash.item, 'UI hash includes: item');
        ok(!hash.sender, 'UI hash does not include: sender');
-       
+
 });
 
 test("update", function() {
-       
+
        var hash;
        $("#sortable")
                .sortable({ update: function(e, ui) { hash = ui; } })
                .find('li:eq(0)').simulate("drag", { dx: 1, dy: 1 });
-       
+
        ok(!hash, '1px drag, update event should not be triggered');
-       
+
        $("#sortable")
                .sortable({ update: function(e, ui) { hash = ui; } })
-               .find('li:eq(0)').simulate("drag", { dx: 0, dy: 20 });  
-               
-       ok(hash, 'update event triggered');     
+               .find('li:eq(0)').simulate("drag", { dx: 0, dy: 20 });
+
+       ok(hash, 'update event triggered');
        ok(!hash.helper, 'UI hash should not include: helper');
        ok(hash.placeholder, 'UI hash includes: placeholder');
        ok(hash.position && (hash.position.top && hash.position.left), 'UI hash includes: position');
        ok(hash.offset && (hash.offset.top && hash.offset.left), 'UI hash includes: offset');
        ok(hash.item, 'UI hash includes: item');
        ok(!hash.sender, 'UI hash does not include: sender');
-       
+
 });
 
 test("receive", function() {
index 2446d27558a09023679cc19728d31b578941e59e..c2a0b95481da3c862e0695891389945b74049ce4 100644 (file)
@@ -5,7 +5,7 @@
 
 var el, offsetBefore, offsetAfter, dragged;
 
-var drag = function(handle, dx, dy) {
+function drag(handle, dx, dy) {
        offsetBefore = $(handle).offset();
        $(handle).simulate("drag", {
                dx: dx || 0,
@@ -15,7 +15,7 @@ var drag = function(handle, dx, dy) {
        offsetAfter = $(handle).offset();
 }
 
-var sort = function(handle, dx, dy, index, msg) {
+function sort(handle, dx, dy, index, msg) {
        drag(handle, dx, dy);
        equal($(handle).parent().children().index(handle), index, msg);
 }
index 6ee503a58cbe86766dca01d8e9cefcb51f0f0d88..22ad61baf8fdf50079c5b6e3aa9886a4b2ad6269 100644 (file)
@@ -5,7 +5,7 @@
 
 var el, offsetBefore, offsetAfter, dragged;
 
-var drag = function(handle, dx, dy) {
+function drag(handle, dx, dy) {
        offsetBefore = $(handle).offset();
        $(handle).simulate("drag", {
                dx: dx || 0,
@@ -15,7 +15,7 @@ var drag = function(handle, dx, dy) {
        offsetAfter = $(handle).offset();
 }
 
-var sort = function(handle, dx, dy, index, msg) {
+function sort(handle, dx, dy, index, msg) {
        drag(handle, dx, dy);
        equal($(handle).parent().children().index(handle), index, msg);
 }