Tests: Move QUnit.jQuerySelectors{,Pos} definitions up

mgol · mgol · commit 73f57c13c864 · 2022-10-07T16:51:41.000+02:00
Previously, they were loaded after test files which meant top level usage
was not evaluated correctly.
diff --git a/test/data/testinit.js b/test/data/testinit.js
@@ -410,6 +410,18 @@ this.loadTests = function() {
 
 	// Get testSubproject from testrunner first
 	require( [ parentUrl + "test/data/testrunner.js" ], function() {
+
+		// Says whether jQuery positional selector extensions are supported.
+		// A full selector engine is required to support them as they need to
+		// be evaluated left-to-right. Remove that property when support for
+		// positional selectors is dropped.
+		QUnit.jQuerySelectorsPos = includesModule( "selector-full" );
+
+		// Says whether jQuery selector extensions are supported. Change that
+		// to `false` if your custom jQuery versions relies more on native qSA.
+		// This doesn't include support for positional selectors (see above).
+		QUnit.jQuerySelectors = includesModule( "selector-full" );
+
 		var i = 0,
 			tests = [
 				// A special module with basic tests, meant for
@@ -469,17 +481,6 @@ this.loadTests = function() {
 				 */
 				jQuery.noConflict();
 
-				// Says whether jQuery positional selector extensions are supported.
-				// A full selector engine is required to support them as they need to
-				// be evaluated left-to-right. Remove that property when support for
-				// positional selectors is dropped.
-				QUnit.jQuerySelectorsPos = includesModule( "selector-full" );
-
-				// Says whether jQuery selector extensions are supported. Change that
-				// to `false` if your custom jQuery versions relies more on native qSA.
-				// This doesn't include support for positional selectors (see above).
-				QUnit.jQuerySelectors = includesModule( "selector-full" );
-
 				// Load the TestSwarm listener if swarmURL is in the address.
 				if ( QUnit.isSwarm ) {
 					require( [ "https://swarm.jquery.org/js/inject.js?" + ( new Date() ).getTime() ],
