From 2aabd427ebb307319f3203aab3f989d7305fd00d Mon Sep 17 00:00:00 2001 From: =?utf8?q?Jo=CC=88rn=20Zaefferer?= Date: Sat, 24 Jan 2015 17:10:20 +0100 Subject: [PATCH] Tests: Fix style issues with missing or multiple line breaks As pointed out by jscs, after a esformatter run. The latter doesn't address these, so fixing them manually. --- tests/unit/autocomplete/autocomplete_core.js | 2 -- tests/unit/button/button_core.js | 1 - tests/unit/button/button_methods.js | 1 - tests/unit/datepicker/datepicker_options.js | 1 - tests/unit/datepicker/datepicker_test_helpers.js | 2 +- tests/unit/dialog/dialog_events.js | 1 - tests/unit/droppable/droppable_events.js | 2 -- tests/unit/droppable/droppable_test_helpers.js | 2 +- tests/unit/menu/menu_events.js | 1 - tests/unit/resizable/resizable_options.js | 1 - tests/unit/resizable/resizable_test_helpers.js | 2 +- tests/unit/selectable/selectable_core.js | 2 +- tests/unit/selectable/selectable_options.js | 1 - tests/unit/selectmenu/selectmenu_core.js | 1 - tests/unit/selectmenu/selectmenu_options.js | 2 -- tests/unit/sortable/sortable_events.js | 2 -- tests/unit/sortable/sortable_test_helpers.js | 2 +- tests/unit/spinner/spinner_test_helpers.js | 1 - tests/unit/tabs/tabs_options.js | 1 - tests/unit/tabs/tabs_test_helpers.js | 1 - tests/unit/testsuite.js | 1 - 21 files changed, 5 insertions(+), 25 deletions(-) diff --git a/tests/unit/autocomplete/autocomplete_core.js b/tests/unit/autocomplete/autocomplete_core.js index 1208d89f1..669cefb86 100644 --- a/tests/unit/autocomplete/autocomplete_core.js +++ b/tests/unit/autocomplete/autocomplete_core.js @@ -216,8 +216,6 @@ asyncTest( "ESCAPE in multiline autocomplete", function() { }, 50 ); }); - - asyncTest( "handle race condition", function() { expect( 3 ); var count = 0, diff --git a/tests/unit/button/button_core.js b/tests/unit/button/button_core.js index f647cdc26..340f25402 100644 --- a/tests/unit/button/button_core.js +++ b/tests/unit/button/button_core.js @@ -2,7 +2,6 @@ * button_core.js */ - (function($) { module("button: core"); diff --git a/tests/unit/button/button_methods.js b/tests/unit/button/button_methods.js index 1c781c87e..d4d92cead 100644 --- a/tests/unit/button/button_methods.js +++ b/tests/unit/button/button_methods.js @@ -3,7 +3,6 @@ */ (function($) { - module("button: methods"); test("destroy", function() { diff --git a/tests/unit/datepicker/datepicker_options.js b/tests/unit/datepicker/datepicker_options.js index 11b010717..08c31f5ad 100644 --- a/tests/unit/datepicker/datepicker_options.js +++ b/tests/unit/datepicker/datepicker_options.js @@ -767,7 +767,6 @@ var beforeShowThis = null, beforeShowDayThis = null, beforeShowDayOK = true; - function beforeAll(input, inst) { beforeShowThis = this; beforeShowInput = input; diff --git a/tests/unit/datepicker/datepicker_test_helpers.js b/tests/unit/datepicker/datepicker_test_helpers.js index 34b41bbc6..82610393b 100644 --- a/tests/unit/datepicker/datepicker_test_helpers.js +++ b/tests/unit/datepicker/datepicker_test_helpers.js @@ -24,4 +24,4 @@ TestHelpers.datepicker = { }, onFocus: TestHelpers.onFocus, PROP_NAME: "datepicker" -}; \ No newline at end of file +}; diff --git a/tests/unit/dialog/dialog_events.js b/tests/unit/dialog/dialog_events.js index 2af9990a8..49b93bdd1 100644 --- a/tests/unit/dialog/dialog_events.js +++ b/tests/unit/dialog/dialog_events.js @@ -39,7 +39,6 @@ test("open", function() { element.remove(); }); - test( "focus", function() { expect( 5 ); var element, other; diff --git a/tests/unit/droppable/droppable_events.js b/tests/unit/droppable/droppable_events.js index 4b8fe5acd..0e1424245 100644 --- a/tests/unit/droppable/droppable_events.js +++ b/tests/unit/droppable/droppable_events.js @@ -35,8 +35,6 @@ test( "droppable destruction/recreation on drop event", function() { ok( !droppable2.hasClass( "active" ), "subsequent droppable no longer active" ); }); - - // todo: comment the following in when ready to actually test /* test("activate", function() { diff --git a/tests/unit/droppable/droppable_test_helpers.js b/tests/unit/droppable/droppable_test_helpers.js index ffd374529..ce972be63 100644 --- a/tests/unit/droppable/droppable_test_helpers.js +++ b/tests/unit/droppable/droppable_test_helpers.js @@ -7,4 +7,4 @@ TestHelpers.droppable = { // todo: actually implement this ok(true, "missing test - untested code is broken code"); } -}; \ No newline at end of file +}; diff --git a/tests/unit/menu/menu_events.js b/tests/unit/menu/menu_events.js index 8156abbcd..b1ade3b85 100644 --- a/tests/unit/menu/menu_events.js +++ b/tests/unit/menu/menu_events.js @@ -169,7 +169,6 @@ asyncTest( "handle submenu auto collapse: mouseleave", function() { setTimeout( menumouseleave1, 350 ); }); - asyncTest( "handle keyboard navigation on menu without scroll and without submenus", function() { expect( 12 ); var element = $( "#menu1" ).menu({ diff --git a/tests/unit/resizable/resizable_options.js b/tests/unit/resizable/resizable_options.js index c46801bc5..f038ef0fa 100644 --- a/tests/unit/resizable/resizable_options.js +++ b/tests/unit/resizable/resizable_options.js @@ -24,7 +24,6 @@ test( "alsoResize", function() { equal( other.width(), 130, "alsoResize width" ); }); - test("aspectRatio: 'preserve' (e)", function() { expect(4); diff --git a/tests/unit/resizable/resizable_test_helpers.js b/tests/unit/resizable/resizable_test_helpers.js index 7ab5aa1a5..cc0ef48ec 100644 --- a/tests/unit/resizable/resizable_test_helpers.js +++ b/tests/unit/resizable/resizable_test_helpers.js @@ -8,4 +8,4 @@ TestHelpers.resizable = { dy: dy }); } -}; \ No newline at end of file +}; diff --git a/tests/unit/selectable/selectable_core.js b/tests/unit/selectable/selectable_core.js index 9953b6c07..c3ace5370 100644 --- a/tests/unit/selectable/selectable_core.js +++ b/tests/unit/selectable/selectable_core.js @@ -1,3 +1,3 @@ /* * selectable_core.js - */ \ No newline at end of file + */ diff --git a/tests/unit/selectable/selectable_options.js b/tests/unit/selectable/selectable_options.js index 973247f56..292c0cf96 100644 --- a/tests/unit/selectable/selectable_options.js +++ b/tests/unit/selectable/selectable_options.js @@ -51,7 +51,6 @@ test("filter", function() { sel = $("*", el), selected = function() { actual += 1; }; - el = $("#selectable1").selectable({ filter: ".special", selected: selected }); el.simulate( "drag", { dx: 1000, diff --git a/tests/unit/selectmenu/selectmenu_core.js b/tests/unit/selectmenu/selectmenu_core.js index 3b5472996..0a2431861 100644 --- a/tests/unit/selectmenu/selectmenu_core.js +++ b/tests/unit/selectmenu/selectmenu_core.js @@ -52,7 +52,6 @@ asyncTest( "accessibility", function() { }); }); - test( "_renderButtonItem()", function() { expect( 2 ); diff --git a/tests/unit/selectmenu/selectmenu_options.js b/tests/unit/selectmenu/selectmenu_options.js index ce66e822b..954d572b5 100644 --- a/tests/unit/selectmenu/selectmenu_options.js +++ b/tests/unit/selectmenu/selectmenu_options.js @@ -67,7 +67,6 @@ test( "appendTo: ui-front", function() { $( "#selectmenu-wrap2" )[ 0 ], "empty jQuery object, inside .ui-front" ); }); - test( "CSS styles", function() { expect( 2 ); @@ -85,7 +84,6 @@ test( "CSS styles", function() { "menu styles dropdown" ); }); - test( "width", function() { expect( 5 ); diff --git a/tests/unit/sortable/sortable_events.js b/tests/unit/sortable/sortable_events.js index 1bf72cc6a..46e86c906 100644 --- a/tests/unit/sortable/sortable_events.js +++ b/tests/unit/sortable/sortable_events.js @@ -26,8 +26,6 @@ test("start", function() { // todo: see if these events should actually have sane values in them ok("position" in hash, "UI hash includes: position"); ok("offset" in hash, "UI hash includes: offset"); - - }); test("sort", function() { diff --git a/tests/unit/sortable/sortable_test_helpers.js b/tests/unit/sortable/sortable_test_helpers.js index 12e4829ea..3cda43b8f 100644 --- a/tests/unit/sortable/sortable_test_helpers.js +++ b/tests/unit/sortable/sortable_test_helpers.js @@ -6,4 +6,4 @@ TestHelpers.sortable = { }); equal( $( handle ).parent().children().index( handle ), index, msg ); } -}; \ No newline at end of file +}; diff --git a/tests/unit/spinner/spinner_test_helpers.js b/tests/unit/spinner/spinner_test_helpers.js index 2021e8f7a..9b11cf632 100644 --- a/tests/unit/spinner/spinner_test_helpers.js +++ b/tests/unit/spinner/spinner_test_helpers.js @@ -5,4 +5,3 @@ TestHelpers.spinner = { .simulate( "keyup", { keyCode: keyCode, shiftKey: shift || false } ); } }; - diff --git a/tests/unit/tabs/tabs_options.js b/tests/unit/tabs/tabs_options.js index a9632481e..7f5352bac 100644 --- a/tests/unit/tabs/tabs_options.js +++ b/tests/unit/tabs/tabs_options.js @@ -362,5 +362,4 @@ asyncTest( "hide and show - animation", function() { element.tabs( "option", "active", 1 ); }); - }( jQuery ) ); diff --git a/tests/unit/tabs/tabs_test_helpers.js b/tests/unit/tabs/tabs_test_helpers.js index 0c9309186..1be90f8fe 100644 --- a/tests/unit/tabs/tabs_test_helpers.js +++ b/tests/unit/tabs/tabs_test_helpers.js @@ -64,4 +64,3 @@ TestHelpers.tabs = { deepEqual( actual, expected ); } }; - diff --git a/tests/unit/testsuite.js b/tests/unit/testsuite.js index 186864114..00ff98677 100644 --- a/tests/unit/testsuite.js +++ b/tests/unit/testsuite.js @@ -27,7 +27,6 @@ QUnit.reset = function() { reset.apply( this, arguments ); }; - QUnit.config.requireExpects = true; /* -- 2.39.5