upgrade dev dependencies to the latest and fix lint errors (#1089)

package.json

@@ -670,26 +670,26 @@
     "@types/istanbul-lib-coverage": "^2.0.4",
     "@types/istanbul-lib-source-maps": "^4.0.1",
     "@types/jest": "^29.5.6",
-    "@types/node": "^18.11.18",
-    "@typescript-eslint/eslint-plugin": "^5.48.1",
-    "@typescript-eslint/parser": "^5.48.1",
+    "@types/node": "^20.8.10",
+    "@typescript-eslint/eslint-plugin": "^6.9.1",
+    "@typescript-eslint/parser": "^6.9.1",
     "@vscode/test-electron": "^2.2.2",
     "eslint": "^8.31.0",
-    "eslint-config-prettier": "^8.6.0",
+    "eslint-config-prettier": "^9.0.0",
     "eslint-plugin-jest": "^27.2.1",
-    "eslint-plugin-jsdoc": "^39.6.4",
+    "eslint-plugin-jsdoc": "^46.8.2",
     "eslint-plugin-prefer-arrow": "^1.2.3",
-    "eslint-plugin-prettier": "^4.2.1",
+    "eslint-plugin-prettier": "^5.0.1",
     "fs-extra": "^11.1.1",
     "jest": "^29.7",
     "jest-snapshot": "^27.2.0",
-    "prettier": "^2.8.2",
+    "prettier": "^3.0.3",
     "raw-loader": "^4.0.1",
-    "rimraf": "^3.0.2",
+    "rimraf": "^5.0.5",
     "ts-jest": "^29.0.3",
     "ts-loader": "^9.4.2",
     "ts-node": "^10.9.1",
-    "typescript": "^4.9.4",
+    "typescript": "^5.2.2",
     "webpack": "^5.75.0",
     "webpack-cli": "^5.0.1"
   },

src/JestExt/output-terminal.ts

@@ -64,7 +64,10 @@ export class ExtOutputTerminal implements JestExtOutput {
       this._terminal = undefined;
     },
   };
-  constructor(private name: string, enabled?: boolean) {
+  constructor(
+    private name: string,
+    enabled?: boolean
+  ) {
     this.ptyIsOpen = false;
     this.pendingMessages = new PendingOutput();
     this.enabled = enabled ?? false;

src/JestExt/process-listeners.ts

@@ -178,7 +178,11 @@ export const DEFAULT_LONG_RUN_THRESHOLD = 60000;
 export class LongRunMonitor {
   private timer: NodeJS.Timeout | undefined;
   public readonly thresholdMs: number;
-  constructor(private callback: () => void, private logging: Logging, option?: MonitorLongRun) {
+  constructor(
+    private callback: () => void,
+    private logging: Logging,
+    option?: MonitorLongRun
+  ) {
     if (option == null) {
       this.thresholdMs = DEFAULT_LONG_RUN_THRESHOLD;
     } else if (typeof option === 'number' && option > 0) {

src/JestProcessManagement/JestProcess.ts

@@ -89,7 +89,6 @@ export class JestProcess implements JestProcessInfo {
   }
 
   private getReporterPath() {
-    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
     const extensionPath = vscode.extensions.getExtension(extensionId)!.extensionPath;
     return join(extensionPath, 'out', 'reporter.js');
   }
@@ -112,7 +111,6 @@ export class JestProcess implements JestProcessInfo {
       args: { args: ['--colors'] },
     };
 
-    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
     const args = options.args!.args;
 
     switch (this.request.type) {
@@ -186,7 +184,7 @@ export class JestProcess implements JestProcessInfo {
     const promise = new Promise<void>((resolve, reject) => {
       taskInfo = { runner, resolve, reject };
     });
-    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+
     this.task = { ...taskInfo!, promise };
 
     runner.start(this.watchMode !== WatchMode.None, this.watchMode === WatchMode.WatchAll);

src/JestProcessManagement/JestProcessManager.ts

@@ -28,7 +28,6 @@ export class JestProcessManager implements TaskArrayFunctions<JestProcess> {
   }
 
   private getQueue(type: QueueType): TaskQueue<JestProcess> {
-    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
     return this.queues.get(type)!;
   }
 

src/StatusBar.ts

@@ -35,7 +35,10 @@ export interface StatusBarUpdateRequest {
 class TypedStatusBarItem {
   public status: ExtensionStatus = {};
   public isVisible = false;
-  constructor(public readonly type: StatusType, protected readonly actual: vscode.StatusBarItem) {
+  constructor(
+    public readonly type: StatusType,
+    protected readonly actual: vscode.StatusBarItem
+  ) {
     this.actual.hide();
   }
   hide() {

src/TestResults/TestResultProvider.ts

@@ -62,7 +62,11 @@ export class TestSuiteRecord implements TestSuiteUpdatable {
   private _testBlocks?: TestBlocks | 'failed';
   private _assertionContainer?: ContainerNode<TestAssertionStatus>;
 
-  constructor(public testFile: string, private reconciler: TestReconciler, private parser: Parser) {
+  constructor(
+    public testFile: string,
+    private reconciler: TestReconciler,
+    private parser: Parser
+  ) {
     this._status = TestReconciliationState.Unknown;
     this._message = '';
   }

src/TestResults/match-by-context.ts

@@ -169,7 +169,7 @@ const createMessaging =
 type MatchResultType<C extends ContextType> = [
   ChildNodeType<ItBlock, C>,
   ChildNodeType<TestAssertionStatus, C>[],
-  MatchEvent
+  MatchEvent,
 ];
 interface ContextMatchAlgorithm {
   match: (

src/messaging.ts

@@ -16,7 +16,7 @@ function _extractActionTitles(actions?: MessageAction[]): string[] {
   return actions ? actions.map((a) => a.title) : [];
 }
 // expose the internal function so we can unit testing it
-// eslint-disable-next-line @typescript-eslint/no-empty-function
+
 const doNothing = () => {};
 export function _handleMessageActions(actions?: MessageAction[]): (action?: string) => void {
   if (!actions || actions.length <= 0) {

src/noop-fs-provider.ts

@@ -1,5 +1,5 @@
 /* istanbul ignore file */
-/* eslint-disable @typescript-eslint/no-empty-function */
+
 import * as vscode from 'vscode';
 
 /**

src/test-provider/test-item-data.ts

@@ -34,11 +34,13 @@ abstract class TestItemDataBase implements TestItemData, JestRunnable, WithUri {
   item!: vscode.TestItem;
   log: Logging;
 
-  constructor(public context: JestTestProviderContext, name: string) {
+  constructor(
+    public context: JestTestProviderContext,
+    name: string
+  ) {
     this.log = context.ext.loggingFactory.create(name);
   }
   get uri(): vscode.Uri {
-    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
     return this.item.uri!;
   }
 
@@ -466,7 +468,6 @@ export class WorkspaceRoot extends TestItemDataBase {
 
 export class FolderData extends TestItemDataBase {
   static makeUri = (parent: vscode.TestItem, folderName: string): vscode.Uri => {
-    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
     return vscode.Uri.joinPath(parent.uri!, folderName);
   };
   constructor(
@@ -522,7 +523,10 @@ const isEmpty = (node?: ItemNodeType): boolean => {
 
 // type AssertNode = NodeType<TestAssertionStatus>;
 abstract class TestResultData extends TestItemDataBase {
-  constructor(readonly context: JestTestProviderContext, name: string) {
+  constructor(
+    readonly context: JestTestProviderContext,
+    name: string
+  ) {
     super(context, name);
   }
 

tests/TestResults/TestResultProvider.test.ts

@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-non-null-assertion */
 jest.unmock('../../src/TestResults/TestResultProvider');
 jest.unmock('../../src/TestResults/test-result-events');
 jest.unmock('../../src/TestResults/match-node');

tests/test-helper.ts

@@ -91,7 +91,7 @@ export const makeAssertion = (
     status,
     location: location ? makeLocation(location) : EmptyLocation,
     ...(override || {}),
-  } as TestAssertionStatus);
+  }) as TestAssertionStatus;
 
 export const makeTestResult = (
   title: string,