Revert "Upgrade from vscode-test to @vscode/test-electron"

Author: aeisenberg

extensions/ql-vscode/package-lock.json

@@ -1297,7 +1297,6 @@
     "@types/xml2js": "~0.4.4",
     "@typescript-eslint/eslint-plugin": "^4.26.0",
     "@typescript-eslint/parser": "^4.26.0",
-    "@vscode/test-electron": "^2.1.5",
     "ansi-colors": "^4.1.1",
     "applicationinsights": "^2.3.5",
     "babel-loader": "^8.2.5",
@@ -1336,6 +1335,7 @@
     "typescript": "^4.5.5",
     "typescript-formatter": "^7.2.2",
     "vsce": "^2.7.0",
+    "vscode-test": "^1.4.0",
     "webpack": "^5.62.2",
     "webpack-cli": "^4.6.0"
   },

extensions/ql-vscode/package.json

@@ -1,4 +1,5 @@
 import 'source-map-support/register';
+import 'vscode-test';
 import { runTestsInDirectory } from '../index-template';
 import 'mocha';
 import * as sinonChai from 'sinon-chai';

extensions/ql-vscode/src/vscode-tests/cli-integration/index.ts

@@ -1,4 +1,5 @@
 import 'source-map-support/register';
+import 'vscode-test';
 import * as sinonChai from 'sinon-chai';
 import * as chai from 'chai';
 import 'chai/register-should';

extensions/ql-vscode/src/vscode-tests/minimal-workspace/index.ts

@@ -1,4 +1,5 @@
 import 'source-map-support/register';
+import 'vscode-test';
 import * as sinonChai from 'sinon-chai';
 import * as chai from 'chai';
 import * as chaiAsPromised from 'chai-as-promised';

extensions/ql-vscode/src/vscode-tests/no-workspace/index.ts

@@ -4,14 +4,14 @@ import * as cp from 'child_process';
 import {
   runTests,
   downloadAndUnzipVSCode,
-  resolveCliArgsFromVSCodeExecutablePath
-} from '@vscode/test-electron';
+  resolveCliPathFromVSCodeExecutablePath
+} from 'vscode-test';
 import { assertNever } from '../pure/helpers-pure';
 import * as tmp from 'tmp-promise';
 
 // For some reason, the following are not exported directly from `vscode-test`,
 // but we can be tricky and import directly from the out file.
-import { TestOptions } from '@vscode/test-electron/out/runTest';
+import { TestOptions } from 'vscode-test/out/runTest';
 
 
 // For CI purposes we want to leave this at 'stable' to catch any bugs
@@ -78,11 +78,10 @@ async function main() {
     const extensionTestsEnv: Record<string, string> = {};
     if (dirs.includes(TestDir.CliIntegration)) {
       console.log('Installing required extensions');
-      const [cli, ...args] = resolveCliArgsFromVSCodeExecutablePath(vscodeExecutablePath);
+      const cliPath = resolveCliPathFromVSCodeExecutablePath(vscodeExecutablePath);
       cp.spawnSync(
-        cli,
+        cliPath,
         [
-          ...args,
           '--install-extension',
           'hbenl.vscode-test-explorer',
           '--install-extension',