Build: Configure eslint for test folder

- Fixed all style issues detected by eslint using jquery configuration
jquery · JuanMaRuiz · Jan 20, 2019 · Jan 20, 2019 · Jan 20, 2019 · Jan 20, 2019
commit 57efcdab169af5b323b633147c6fcc18553669fd
diff --git a/src/sizzle.js b/src/sizzle.js
@@ -770,6 +770,7 @@
 		rbuggyQSA = [];
 
 		if ( ( support.qsa = rnative.test( document.querySelectorAll ) ) ) {
+
 			// Build QSA regex
 			// Regex strategy adopted from Diego Perini
 			assert( function( el ) {
@@ -859,6 +860,7 @@
 			docElem.msMatchesSelector ) ) ) ) {
 
 			assert( function( el ) {
+
 				// Check to see if it's possible to do matchesSelector
 				// on a disconnected node (IE 9)
 				support.disconnectedMatch = matches.call( el, "*" );
@@ -1208,6 +1210,7 @@
 				match[ 1 ] = match[ 1 ].toLowerCase();
 
 				if ( match[ 1 ].slice( 0, 3 ) === "nth" ) {
+
 					// nth-* requires argument
 					if ( !match[ 3 ] ) {
 						Sizzle.error( match[ 0 ] );

diff --git a/tasks/commit.js b/tasks/commit.js
@@ -1,8 +1,7 @@
-"use strict";
-
 var exec = require( "child_process" ).exec;
 
 module.exports = function( grunt ) {
+	"use strict";
 	grunt.registerTask( "commit", "Add and commit changes", function( message ) {
 
 		// Always add dist directory

diff --git a/tasks/compile.js b/tasks/compile.js
@@ -1,6 +1,6 @@
-"use strict";
 
 module.exports = function( grunt ) {
+	"use strict";
 	grunt.registerMultiTask(
 		"compile",
 		"Compile sizzle.js to the dist directory. Embed date/version.",

diff --git a/tasks/dist.js b/tasks/dist.js
@@ -1,8 +1,8 @@
-"use strict";
 
 var fs = require( "fs" );
 
 module.exports = function( grunt ) {
+	"use strict";
 	grunt.registerTask( "dist", "Process files for distribution", function() {
 		var files = grunt.file.expand( { filter: "isFile" }, "dist/*" );
 

diff --git a/tasks/ensure_ascii.js b/tasks/ensure_ascii.js
@@ -1,8 +1,8 @@
-"use strict";
 
 var fs = require( "fs" );
 
 module.exports = function( grunt ) {
+	"use strict";
 	grunt.registerMultiTask(
 		"ensure_ascii",
 		"Verify that files contain no non-ASCII characters.",

diff --git a/tasks/release.js b/tasks/release.js
@@ -1,8 +1,7 @@
-"use strict";
-
 var exec = require( "child_process" ).exec;
 
 module.exports = function( grunt ) {
+	"use strict";
 	var rpreversion = /(\d\.\d+\.\d+)-pre/;
 
 	grunt.registerTask( "release",

diff --git a/tasks/tag.js b/tasks/tag.js
@@ -1,8 +1,7 @@
-"use strict";
-
 var exec = require( "child_process" ).exec;
 
 module.exports = function( grunt ) {
+	"use strict";
 	grunt.registerTask( "tag", "Tag the specified version", function( version ) {
 		exec( "git tag " + version, this.async() );
 	} );

diff --git a/tasks/test.js b/tasks/test.js
@@ -1,6 +1,5 @@
-"use strict";
-
 module.exports = function( grunt ) {
+	"use strict";
 	grunt.registerTask( "test", "Run lint and tests for the specific browsers", function() {
 		var browsers = grunt.option( "browsers" ),
 			options = grunt.config.data.karma.all;

diff --git a/tasks/version.js b/tasks/version.js
@@ -1,8 +1,7 @@
-"use strict";
-
 var exec = require( "child_process" ).exec;
 
 module.exports = function( grunt ) {
+	"use strict";
 	grunt.registerTask( "version", "Commit a new version", function( version ) {
 		if ( !/\d\.\d+\.\d+(?:-pre)?/.test( version ) ) {
 			grunt.fatal( "Version must follow semver release format: " + version );

diff --git a/test/.eslintrc b/test/.eslintrc
@@ -7,7 +7,8 @@
 
 		// In source the browser env is not enabled but unit tests rely on them
 		// too much and we don't run them in non-browser environments anyway.
-		"browser": true
+		"browser": true,
+		"node": true
 	},
 
 	"globals": {
@@ -17,7 +18,8 @@
 		"q": true,
 		"t": true,
 		"setup": true,
-		"testIframeWithCallback": true
+		"testIframeWithCallback": true,
+		"createWithFriesXML": true
 	},
 
 	"rules": {

diff --git a/test/karma/karma.conf.js b/test/karma/karma.conf.js
@@ -9,10 +9,10 @@ module.exports = function( config ) {
 			process.env.BROWSER_STACK_ACCESS_KEY ),
 		hostName = isBrowserStack ? "bs-local.com" : "localhost";
 
-	config.set({
+	config.set( {
 		browserStack: {
 			project: "sizzle",
-			build: "local run" + (dateString ? ", " + dateString : ""),
+			build: "local run" + ( dateString ? ", " + dateString : "" ),
 			timeout: 600, // 10 min
 			// BrowserStack has a limit of 120 requests per minute. The default
 			// "request per second" strategy doesn't scale to so many browsers.
@@ -91,7 +91,7 @@ module.exports = function( config ) {
 		browserNoActivityTimeout: 3e5,
 		browserDisconnectTimeout: 3e5,
 		browserDisconnectTolerance: 3
-	});
+	} );
 
 	// Deal with Travis environment
 	if ( isTravis ) {

diff --git a/test/unit/extending.js b/test/unit/extending.js
@@ -1,68 +1,68 @@
 QUnit.module( "extending", { beforeEach: setup } );
 
-QUnit.test("custom pseudos", function( assert ) {
+QUnit.test( "custom pseudos", function( assert ) {
 	assert.expect( 6 );
 
 	Sizzle.selectors.filters.foundation = Sizzle.selectors.filters.root;
-	assert.deepEqual( Sizzle(":foundation"), [ document.documentElement ], "Copy element filter with new name" );
+	assert.deepEqual( Sizzle( ":foundation" ), [ document.documentElement ], "Copy element filter with new name" );
 	delete Sizzle.selectors.filters.foundation;
 
 	Sizzle.selectors.setFilters.primary = Sizzle.selectors.setFilters.first;
 	t( "Copy set filter with new name", "div#qunit-fixture :primary", [ "firstp" ] );
 	delete Sizzle.selectors.setFilters.primary;
 
-	Sizzle.selectors.filters.aristotlean = Sizzle.selectors.createPseudo(function() {
+	Sizzle.selectors.filters.aristotlean = Sizzle.selectors.createPseudo( function() {
 		return function( elem ) {
 			return !!elem.id;
 		};
-	});
+	} );
 	t( "Custom element filter", "#foo :aristotlean", [ "sndp", "en", "yahoo", "sap", "anchor2", "simon" ] );
 	delete Sizzle.selectors.filters.aristotlean;
 
-	Sizzle.selectors.filters.endswith = Sizzle.selectors.createPseudo(function( text ) {
+	Sizzle.selectors.filters.endswith = Sizzle.selectors.createPseudo( function( text ) {
 		return function( elem ) {
 			return Sizzle.getText( elem ).slice( -text.length ) === text;
 		};
-	});
-	t( "Custom element filter with argument", "a:endswith(ogle)", ["google"] );
+	} );
+	t( "Custom element filter with argument", "a:endswith(ogle)", [ "google" ] );
 	delete Sizzle.selectors.filters.endswith;
 
-	Sizzle.selectors.setFilters.second = Sizzle.selectors.createPseudo(function() {
-		return Sizzle.selectors.createPseudo(function( seed, matches ) {
-			if ( seed[1] ) {
-				matches[1] = seed[1];
-				seed[1] = false;
+	Sizzle.selectors.setFilters.second = Sizzle.selectors.createPseudo( function() {
+		return Sizzle.selectors.createPseudo( function( seed, matches ) {
+			if ( seed[ 1 ] ) {
+				matches[ 1 ] = seed[ 1 ];
+				seed[ 1 ] = false;
 			}
-		});
-	});
-	t( "Custom set filter", "#qunit-fixture p:second", ["ap"] );
+		} );
+	} );
+	t( "Custom set filter", "#qunit-fixture p:second", [ "ap" ] );
 	delete Sizzle.selectors.filters.second;
 
-	Sizzle.selectors.setFilters.slice = Sizzle.selectors.createPseudo(function( argument ) {
-		var bounds = argument.split(":");
-		return Sizzle.selectors.createPseudo(function( seed, matches ) {
-			var i = bounds[1];
+	Sizzle.selectors.setFilters.slice = Sizzle.selectors.createPseudo( function( argument ) {
+		var bounds = argument.split( ":" );
+		return Sizzle.selectors.createPseudo( function( seed, matches ) {
+			var i = bounds[ 1 ];
 
 			// Match elements found at the specified indexes
-			while ( --i >= bounds[0] ) {
-				if ( seed[i] ) {
-					matches[i] = seed[i];
-					seed[i] = false;
+			while ( --i >= bounds[ 0 ] ) {
+				if ( seed[ i ] ) {
+					matches[ i ] = seed[ i ];
+					seed[ i ] = false;
 				}
 			}
-		});
-	});
+		} );
+	} );
 	t( "Custom set filter with argument", "#qunit-fixture p:slice(1:3)", [ "ap", "sndp" ] );
 	delete Sizzle.selectors.filters.slice;
-});
+} );
 
-QUnit.test("backwards-compatible custom pseudos", function( assert ) {
+QUnit.test( "backwards-compatible custom pseudos", function( assert ) {
 	assert.expect( 3 );
 
 	Sizzle.selectors.filters.icontains = function( elem, i, match ) {
-		return Sizzle.getText( elem ).toLowerCase().indexOf( (match[3] || "").toLowerCase() ) > -1;
+		return Sizzle.getText( elem ).toLowerCase().indexOf( ( match[ 3 ] || "" ).toLowerCase() ) > -1;
 	};
-	t( "Custom element filter with argument", "a:icontains(THIS BLOG ENTRY)", ["simon1"] );
+	t( "Custom element filter with argument", "a:icontains(THIS BLOG ENTRY)", [ "simon1" ] );
 	delete Sizzle.selectors.filters.icontains;
 
 	Sizzle.selectors.setFilters.podium = function( elements, argument ) {
@@ -71,32 +71,32 @@ QUnit.test("backwards-compatible custom pseudos", function( assert ) {
 	};
 	// Using TAG as the first token here forces this setMatcher into a fail state
 	// Where the descendent combinator was lost
-	t( "Custom setFilter", "form#form :PODIUM", ["label-for", "text1", "text2"] );
-	t( "Custom setFilter with argument", "#form input:Podium(1)", ["text1"] );
+	t( "Custom setFilter", "form#form :PODIUM", [ "label-for", "text1", "text2" ] );
+	t( "Custom setFilter with argument", "#form input:Podium(1)", [ "text1" ] );
 	delete Sizzle.selectors.setFilters.podium;
-});
+} );
 
-QUnit.test("custom attribute getters", function( assert ) {
+QUnit.test( "custom attribute getters", function( assert ) {
 	assert.expect( 2 );
 
 	var original = Sizzle.selectors.attrHandle.hreflang,
 		selector = "a:contains('mark')[hreflang='http://diveintomark.org/en']";
 
 	Sizzle.selectors.attrHandle.hreflang = function( elem, name ) {
-		var href = elem.getAttribute("href"),
+		var href = elem.getAttribute( "href" ),
 			lang = elem.getAttribute( name );
 		return lang && ( href + lang );
 	};
 
-	assert.deepEqual( Sizzle(selector, createWithFriesXML()), [], "Custom attrHandle (preferred document)" );
-	t( "Custom attrHandle (preferred document)", selector, ["mark"] );
+	assert.deepEqual( Sizzle( selector, createWithFriesXML() ), [], "Custom attrHandle (preferred document)" );
+	t( "Custom attrHandle (preferred document)", selector, [ "mark" ] );
 
 	Sizzle.selectors.attrHandle.hreflang = original;
-});
+} );
 
-QUnit.test("Ensure no 'undefined' handler is added", function( assert ) {
+QUnit.test( "Ensure no 'undefined' handler is added", function( assert ) {
 	assert.expect( 1 );
 
 	assert.ok( !Sizzle.selectors.attrHandle.hasOwnProperty( "undefined" ),
 		"Extra attr handlers are not added to Expr.attrHandle (gh-353)" );
-});
+} );