Tests: more style corrections

Provocated by jscs-dev/node-jscs@2de68c0

Ref c161eecce09fc66ce252d4adf64b565f726bb6d2

diff --git a/test/unit/ajax.js b/test/unit/ajax.js
index edeed18fc4424c37f19f2e6f1499984ed896b7c9..4d9b1bc49c9ec662dc781ca0e503f5fbe191e444 100644 (file)
--- a/test/unit/ajax.js
+++ b/test/unit/ajax.js
@@ -792,7 +792,7 @@ QUnit.module( "ajax", {
                                                );
                                        }
 
-                                       jQuery.ajax({
+                                       jQuery.ajax( {
                                                url: "data/jsonp.php",
                                                dataType: "jsonp",
                                                crossDomain: crossDomain,
@@ -800,7 +800,7 @@ QUnit.module( "ajax", {
                                                        assert.strictEqual( this.jsonpCallback, previous.callback, "JSONP callback name is re-used" );
                                                        return false;
                                                }
-                                       });
+                                       } );
                                }
                        };
                } );
@@ -1549,7 +1549,7 @@ QUnit.module( "ajax", {
                                        assert.ok( false, "error" );
                                }
                        };
-               });
+               } );
        }
 
        ajaxTest( "#14683 - jQuery.ajax() - Exceptions thrown synchronously by xhr.send should be caught", 4, function( assert ) {
@@ -1582,8 +1582,6 @@ QUnit.module( "ajax", {
                }
        );
 
-
-
 // //----------- jQuery.ajaxPrefilter()
 
        ajaxTest( "jQuery.ajaxPrefilter() - abort", 1, function( assert ) {

diff --git a/test/unit/core.js b/test/unit/core.js
index de0fc0bcd46d6792f0370c04c3add02a9e9bd91b..e466dad8d22149aaebe2cf752e0490708e5f646f 100644 (file)
--- a/test/unit/core.js
+++ b/test/unit/core.js
@@ -1373,7 +1373,7 @@ QUnit.test( "jQuery.parseHTML", function( assert ) {
 
 if ( jQuery.support.createHTMLDocument ) {
        QUnit.asyncTest( "jQuery.parseHTML", function( assert ) {
-               assert.expect ( 1 );
+               assert.expect( 1 );
 
                Globals.register( "parseHTMLError" );
 

diff --git a/test/unit/data.js b/test/unit/data.js
index 5a232c401d7cfb3770dae12adfd230853ea16048..20d7c27d022deaa7186137c327172eb5781fd3aa 100644 (file)
--- a/test/unit/data.js
+++ b/test/unit/data.js
@@ -535,8 +535,8 @@ QUnit.test( "data-* attributes", function( assert ) {
                child.data( "notjson" ), " {}",
                "JSON object with leading non-JSON read from attribute as string" );
        assert.strictEqual(
-               child.data("notjson2"), "[] ",
-               "JSON array with trailing non-JSON read from attribute as string");
+               child.data( "notjson2" ), "[] ",
+               "JSON array with trailing non-JSON read from attribute as string" );
        assert.strictEqual(
                child.data( "empty" ), "", "Empty string read from attribute"
        );
@@ -1128,8 +1128,9 @@ QUnit.test( ".data(prop) does not create expando", function( assert ) {
        var key,
                div = jQuery( "<div/>" );
 
-       div.data("foo");
-       assert.equal( false, jQuery.hasData( div[0] ) );
+       div.data( "foo" );
+       assert.equal( false, jQuery.hasData( div[ 0 ] ) );
+
        // Make sure no expando has been added
        for ( key in div[ 0 ] ) {
                if ( /^jQuery/.test( key ) ) {

diff --git a/test/unit/effects.js b/test/unit/effects.js
index f290eec63750c9819d032cbaea366cb2861febf7..e3dc0e462cef58139725f1896b2bff23a810662a 100644 (file)
--- a/test/unit/effects.js
+++ b/test/unit/effects.js
@@ -1849,7 +1849,7 @@ QUnit.test( "non-px animation handles non-numeric start (#11971)", function( ass
        this.clock.tick( 10 );
 } );
 
-QUnit.test("Animation callbacks (#11797)", function( assert ) {
+QUnit.test( "Animation callbacks (#11797)", function( assert ) {
        assert.expect( 16 );
 
        var prog = 0,
@@ -1970,7 +1970,7 @@ QUnit.test( "Animation callbacks in order (#2292)", function( assert ) {
                always: function() {
                        assert.step( 5 );
                }
-       }).finish();
+       } ).finish();
 
        this.clock.tick( dur + 10 );
 } );

diff --git a/test/unit/event.js b/test/unit/event.js
index cb4fb629b5eb4463108746c1352f85ab54457004..0f500e6cdaa67df932ce8a4eef99dd0f06c1d34c 100644 (file)
--- a/test/unit/event.js
+++ b/test/unit/event.js
@@ -2515,7 +2515,7 @@ testIframeWithCallback(
                var input = jQuery( frameDoc ).find( "#frame-input" );
 
                // Create a focusin handler on the parent; shouldn't affect the iframe's fate
-               jQuery ( "body" ).on( "focusin.iframeTest", function() {
+               jQuery( "body" ).on( "focusin.iframeTest", function() {
                        assert.ok( false, "fired a focusin event in the parent document" );
                } );
 

diff --git a/test/unit/manipulation.js b/test/unit/manipulation.js
index dad61a4769d5c686beb9bab46c0af237f75889d6..a3da639c66b7a178317d415c5bbaad279579fd7f 100644 (file)
--- a/test/unit/manipulation.js
+++ b/test/unit/manipulation.js
@@ -99,11 +99,11 @@ function testText( valueObj, assert ) {
 }
 
 QUnit.test( "text(String)", function( assert ) {
-       testText(manipulationBareObj,  assert );
+       testText( manipulationBareObj,  assert );
 } );
 
 QUnit.test( "text(Function)", function( assert ) {
-       testText(manipulationFunctionReturningObj,  assert );
+       testText( manipulationFunctionReturningObj,  assert );
 } );
 
 QUnit.test( "text(Function) with incoming value", function( assert ) {
@@ -198,8 +198,8 @@ function testAppend( valueObj, assert ) {
 
        assert.expect( 78 );
 
-       testAppendForObject( valueObj,false, assert  );
-       testAppendForObject( valueObj,true, assert  );
+       testAppendForObject( valueObj, false, assert  );
+       testAppendForObject( valueObj, true, assert  );
 
        var defaultText, result, message, iframe, iframeDoc, j, d,
                $input, $radioChecked, $radioUnchecked, $radioParent, $map, $table;
@@ -306,11 +306,11 @@ function testAppend( valueObj, assert ) {
 }
 
 QUnit.test( "append(String|Element|Array<Element>|jQuery)", function( assert ) {
-       testAppend(manipulationBareObj, assert  );
+       testAppend( manipulationBareObj, assert  );
 } );
 
 QUnit.test( "append(Function)", function( assert ) {
-       testAppend(manipulationFunctionReturningObj, assert  );
+       testAppend( manipulationFunctionReturningObj, assert  );
 } );
 
 QUnit.test( "append(param) to object, see #11280", function( assert ) {