tests - do not break codeAutomationExit API (refs microsoft/vscode-test-web#82) (microsoft#181682)

Author: bpasero

src/vs/platform/log/browser/log.ts

@@ -11,7 +11,7 @@ import { AdapterLogger, DEFAULT_LOG_LEVEL, ILogger, LogLevel } from 'vs/platform
 
 export interface IAutomatedWindow {
 	codeAutomationLog(type: string, args: any[]): void;
-	codeAutomationExit(logs: Array<ILogFile>, code: number): void;
+	codeAutomationExit(code: number, logs: Array<ILogFile>): void;
 }
 
 export interface ILogFile {

src/vs/workbench/services/extensions/browser/extensionService.ts

@@ -146,7 +146,7 @@ export class ExtensionService extends AbstractExtensionService implements IExten
 		// If we are running extension tests, forward logs and exit code
 		const automatedWindow = window as unknown as IAutomatedWindow;
 		if (typeof automatedWindow.codeAutomationExit === 'function') {
-			automatedWindow.codeAutomationExit(await getLogs(this._fileService, this._environmentService), code);
+			automatedWindow.codeAutomationExit(code, await getLogs(this._fileService, this._environmentService));
 		}
 	}
 }

test/integration/browser/src/index.ts

@@ -67,7 +67,7 @@ async function runTestsInBrowser(browserType: BrowserType, endpoint: url.UrlWith
 		console[type](...args);
 	});
 
-	await page.exposeFunction('codeAutomationExit', async (logs: Array<{ readonly relativePath: string; readonly contents: string }>, code: number) => {
+	await page.exposeFunction('codeAutomationExit', async (code: number, logs: Array<{ readonly relativePath: string; readonly contents: string }>) => {
 		try {
 			for (const log of logs) {
 				const absoluteLogsPath = path.join(logsPath, log.relativePath);