Skip to content

Commit d8b7e7b

Browse files
markelogmarkelog
committed
Tests: more style corrections
Provocated by jscs-dev/node-jscs@2de68c0 Ref c161eec
1 parent 2f0cedc commit d8b7e7b

File tree

6 files changed

+18
-19
lines changed

6 files changed

+18
-19
lines changed

‎test/unit/ajax.js‎

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -792,15 +792,15 @@ QUnit.module( "ajax", {
792792
);
793793
}
794794

795-
jQuery.ajax({
795+
jQuery.ajax( {
796796
url: "data/jsonp.php",
797797
dataType: "jsonp",
798798
crossDomain: crossDomain,
799799
beforeSend: function() {
800800
assert.strictEqual( this.jsonpCallback, previous.callback, "JSONP callback name is re-used" );
801801
return false;
802802
}
803-
});
803+
} );
804804
}
805805
};
806806
} );
@@ -1549,7 +1549,7 @@ QUnit.module( "ajax", {
15491549
assert.ok( false, "error" );
15501550
}
15511551
};
1552-
});
1552+
} );
15531553
}
15541554

15551555
ajaxTest( "#14683 - jQuery.ajax() - Exceptions thrown synchronously by xhr.send should be caught", 4, function( assert ) {
@@ -1582,8 +1582,6 @@ QUnit.module( "ajax", {
15821582
}
15831583
);
15841584

1585-
1586-
15871585
// //----------- jQuery.ajaxPrefilter()
15881586

15891587
ajaxTest( "jQuery.ajaxPrefilter() - abort", 1, function( assert ) {

‎test/unit/core.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1373,7 +1373,7 @@ QUnit.test( "jQuery.parseHTML", function( assert ) {
13731373

13741374
if ( jQuery.support.createHTMLDocument ) {
13751375
QUnit.asyncTest( "jQuery.parseHTML", function( assert ) {
1376-
assert.expect ( 1 );
1376+
assert.expect( 1 );
13771377

13781378
Globals.register( "parseHTMLError" );
13791379

‎test/unit/data.js‎

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -535,8 +535,8 @@ QUnit.test( "data-* attributes", function( assert ) {
535535
child.data( "notjson" ), " {}",
536536
"JSON object with leading non-JSON read from attribute as string" );
537537
assert.strictEqual(
538-
child.data("notjson2"), "[] ",
539-
"JSON array with trailing non-JSON read from attribute as string");
538+
child.data( "notjson2" ), "[] ",
539+
"JSON array with trailing non-JSON read from attribute as string" );
540540
assert.strictEqual(
541541
child.data( "empty" ), "", "Empty string read from attribute"
542542
);
@@ -1128,8 +1128,9 @@ QUnit.test( ".data(prop) does not create expando", function( assert ) {
11281128
var key,
11291129
div = jQuery( "<div/>" );
11301130

1131-
div.data("foo");
1132-
assert.equal( false, jQuery.hasData( div[0] ) );
1131+
div.data( "foo" );
1132+
assert.equal( false, jQuery.hasData( div[ 0 ] ) );
1133+
11331134
// Make sure no expando has been added
11341135
for ( key in div[ 0 ] ) {
11351136
if ( /^jQuery/.test( key ) ) {

‎test/unit/effects.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1849,7 +1849,7 @@ QUnit.test( "non-px animation handles non-numeric start (#11971)", function( ass
18491849
this.clock.tick( 10 );
18501850
} );
18511851

1852-
QUnit.test("Animation callbacks (#11797)", function( assert ) {
1852+
QUnit.test( "Animation callbacks (#11797)", function( assert ) {
18531853
assert.expect( 16 );
18541854

18551855
var prog = 0,
@@ -1970,7 +1970,7 @@ QUnit.test( "Animation callbacks in order (#2292)", function( assert ) {
19701970
always: function() {
19711971
assert.step( 5 );
19721972
}
1973-
}).finish();
1973+
} ).finish();
19741974

19751975
this.clock.tick( dur + 10 );
19761976
} );

‎test/unit/event.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2515,7 +2515,7 @@ testIframeWithCallback(
25152515
var input = jQuery( frameDoc ).find( "#frame-input" );
25162516

25172517
// Create a focusin handler on the parent; shouldn't affect the iframe's fate
2518-
jQuery ( "body" ).on( "focusin.iframeTest", function() {
2518+
jQuery( "body" ).on( "focusin.iframeTest", function() {
25192519
assert.ok( false, "fired a focusin event in the parent document" );
25202520
} );
25212521

‎test/unit/manipulation.js‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -99,11 +99,11 @@ function testText( valueObj, assert ) {
9999
}
100100

101101
QUnit.test( "text(String)", function( assert ) {
102-
testText(manipulationBareObj, assert );
102+
testText( manipulationBareObj, assert );
103103
} );
104104

105105
QUnit.test( "text(Function)", function( assert ) {
106-
testText(manipulationFunctionReturningObj, assert );
106+
testText( manipulationFunctionReturningObj, assert );
107107
} );
108108

109109
QUnit.test( "text(Function) with incoming value", function( assert ) {
@@ -198,8 +198,8 @@ function testAppend( valueObj, assert ) {
198198

199199
assert.expect( 78 );
200200

201-
testAppendForObject( valueObj,false, assert );
202-
testAppendForObject( valueObj,true, assert );
201+
testAppendForObject( valueObj, false, assert );
202+
testAppendForObject( valueObj, true, assert );
203203

204204
var defaultText, result, message, iframe, iframeDoc, j, d,
205205
$input, $radioChecked, $radioUnchecked, $radioParent, $map, $table;
@@ -306,11 +306,11 @@ function testAppend( valueObj, assert ) {
306306
}
307307

308308
QUnit.test( "append(String|Element|Array<Element>|jQuery)", function( assert ) {
309-
testAppend(manipulationBareObj, assert );
309+
testAppend( manipulationBareObj, assert );
310310
} );
311311

312312
QUnit.test( "append(Function)", function( assert ) {
313-
testAppend(manipulationFunctionReturningObj, assert );
313+
testAppend( manipulationFunctionReturningObj, assert );
314314
} );
315315

316316
QUnit.test( "append(param) to object, see #11280", function( assert ) {

0 commit comments

Comments
 (0)