Merge pull request #140 from Discookie/ericsson/fix-ci

Update dependencies and fix Yarn 2 enable bug

.github/workflows/package.yml

@@ -24,6 +24,9 @@ jobs:
     steps:
       - uses: actions/checkout@v3
 
+      - name: Enable Corepack
+        run: corepack enable
+
       - name: Enable Yarn 2
         run: yarn set version stable
 

.github/workflows/test.yml

@@ -10,6 +10,9 @@ jobs:
     steps:
       - uses: actions/checkout@v3
 
+      - name: Enable Corepack
+        run: corepack enable
+
       - name: Enable Yarn 2
         run: yarn set version stable
 

package.json

@@ -340,26 +340,27 @@
     "test": "yarn run pretest && node ./out/test/runTest.js"
   },
   "devDependencies": {
-    "@types/glob": "^7.1.3",
-    "@types/mocha": "^8.0.4",
-    "@types/node": "^12.11.7",
-    "@types/shell-quote": "^1.7.1",
+    "@types/glob": "^8.1.0",
+    "@types/mocha": "^10.0.6",
+    "@types/node": "^16.11.7",
+    "@types/shell-quote": "^1.7.5",
     "@types/vscode": "1.53.0",
-    "@typescript-eslint/eslint-plugin": "^5.55.0",
-    "@typescript-eslint/parser": "^5.55.0",
-    "@vscode/test-electron": "^1.5.0",
-    "@vscode/vsce": "^2.18.0",
-    "eslint": "^7.19.0",
-    "glob": "^7.1.6",
-    "mocha": "^8.2.1",
-    "sinon": "^13.0.1",
-    "ts-loader": "^8.0.14",
-    "typescript": "^4.1.3",
-    "webpack": "^5.19.0",
-    "webpack-cli": "^4.4.0"
+    "@typescript-eslint/eslint-plugin": "^7.1.0",
+    "@typescript-eslint/parser": "^7.1.0",
+    "@vscode/test-electron": "^2.3.9",
+    "@vscode/vsce": "^2.24.0",
+    "eslint": "^8.57.0",
+    "glob": "^8.1.0",
+    "mocha": "^10.3.0",
+    "sinon": "^17.0.1",
+    "ts-loader": "^9.5.1",
+    "typescript": "^5.3.3",
+    "webpack": "^5.90.3",
+    "webpack-cli": "^5.1.4"
   },
   "dependencies": {
-    "shell-quote": "^1.7.3"
+    "shell-quote": "^1.8.1",
+    "tslib": "^2.6.2"
   },
-  "packageManager": "yarn@3.4.1"
+  "packageManager": "[email protected].0"
 }

src/backend/executor/process.ts

@@ -1,4 +1,4 @@
-import * as child_process from 'child_process';
+import * as childProcess from 'child_process';
 import * as os from 'os';
 import { quote } from 'shell-quote';
 import { Disposable, Event, EventEmitter, ExtensionContext, workspace } from 'vscode';
@@ -56,7 +56,7 @@ export class ScheduledProcess implements Disposable {
     public readonly executable: string;
     public readonly commandArgs: string[];
 
-    private activeProcess?: child_process.ChildProcess;
+    private activeProcess?: childProcess.ChildProcess;
 
     /** Contains parameters for the executor. All members are defined. */
     public readonly processParameters: ProcessParameters;
@@ -135,7 +135,7 @@ export class ScheduledProcess implements Disposable {
 
         this._processStderr.fire(`>>> Starting process '${commonName}'\n`);
         this._processStderr.fire(`> ${this.commandLine}\n`);
-        this.activeProcess = child_process.spawn(
+        this.activeProcess = childProcess.spawn(
             this.executable,
             this.commandArgs,
             { cwd: workspace.workspaceFolders[0].uri.fsPath }

src/test/parser/metadata.unit.test.ts

@@ -3,7 +3,7 @@ import * as fs from 'fs';
 import * as path from 'path';
 import { promisify } from 'util';
 
-import { parseMetadata } from '../../backend/parser';
+import { MetadataParseError, parseMetadata } from '../../backend/parser';
 import { STATIC_FILE_PATH } from '../utils/constants';
 
 suite('Unit Test: Metadata Parser', () => {
@@ -42,7 +42,7 @@ suite('Unit Test: Metadata Parser', () => {
         const testAndExpectCode = async (filename: string, code: string) => {
             await assert.rejects(
                 () => parseMetadata(path.join(staticPath, filename)),
-                (err) => {
+                (err: MetadataParseError) => {
                     assert.strictEqual(err.code, code, 'wrong error code');
                     return true;
                 },
@@ -52,7 +52,7 @@ suite('Unit Test: Metadata Parser', () => {
         const testAndExpectSyntaxError = async (filename: string) => {
             await assert.rejects(
                 () => parseMetadata(path.join(staticPath, filename)),
-                (err) => {
+                (err: MetadataParseError) => {
                     assert.ok(err instanceof SyntaxError, 'wrong error type');
                     return true;
                 },

src/test/suite/index.ts

@@ -12,7 +12,7 @@ export function run(): Promise<void> {
     const testsRoot = path.resolve(__dirname, '..');
 
     return new Promise((c, e) => {
-        glob('**/**.test.js', { cwd: testsRoot }, (err, files) => {
+        glob.glob('**/**.test.js', { cwd: testsRoot }, (err: any, files: string[]) => {
             if (err) {
                 return e(err);
             }