Fix ordering of params.

Jamie Rothfeder · Jamie Rothfeder · commit ff27bfeb4ab0 · 2025-12-16T14:13:34.000-05:00
diff --git a/src/apptesting/parseTestFiles.ts b/src/apptesting/parseTestFiles.ts
@@ -48,7 +48,7 @@ export async function parseTestFiles(
           }
           for (const rawTestDef of parsedFile.tests) {
             if (!nameFilterFn(rawTestDef.testName)) continue;
-            const testDef = toTestDef(rawTestDef, targetUri, defaultConfig);
+            const testDef = toTestDef(rawTestDef, defaultConfig, targetUri);
             results.push(testDef);
           }
         } catch (ex) {
@@ -65,7 +65,7 @@ export async function parseTestFiles(
   return parseTestFilesRecursive(dir);
 }
 
-function toTestDef(testDef: any, targetUri?: string, defaultConfig: any): TestCaseInvocation {
+function toTestDef(testDef: any, defaultConfig: any, targetUri?: string): TestCaseInvocation {
   const steps = testDef.steps ?? [];
   const route = testDef.testConfig?.route ?? defaultConfig?.route ?? "";
   const browsers: Browser[] = testDef.testConfig?.browsers ??

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ export async function parseTestFiles(`
`48`	`48`	`}`
`49`	`49`	`for (const rawTestDef of parsedFile.tests) {`
`50`	`50`	`if (!nameFilterFn(rawTestDef.testName)) continue;`
`51`		`- const testDef = toTestDef(rawTestDef, targetUri, defaultConfig);`
	`51`	`+ const testDef = toTestDef(rawTestDef, defaultConfig, targetUri);`
`52`	`52`	`results.push(testDef);`
`53`	`53`	`}`
`54`	`54`	`} catch (ex) {`
`@@ -65,7 +65,7 @@ export async function parseTestFiles(`
`65`	`65`	`return parseTestFilesRecursive(dir);`
`66`	`66`	`}`
`67`	`67`
`68`		`-function toTestDef(testDef: any, targetUri?: string, defaultConfig: any): TestCaseInvocation {`
	`68`	`+function toTestDef(testDef: any, defaultConfig: any, targetUri?: string): TestCaseInvocation {`
`69`	`69`	`const steps = testDef.steps ?? [];`
`70`	`70`	`const route = testDef.testConfig?.route ?? defaultConfig?.route ?? "";`
`71`	`71`	`const browsers: Browser[] = testDef.testConfig?.browsers ??`