refactor: implement reset method in CliPlatformTest to clear static collections and prevent memory leaks

Author: Romakita

packages/cli-testing/src/CliPlatformTest.ts

@@ -28,6 +28,16 @@ import {FakeCliFs} from "./FakeCliFs.js";
 import {FakeCliHttpClient} from "./FakeCliHttpClient.js";
 
 export class CliPlatformTest extends DITest {
+  static async reset() {
+    // Explicitly clear FakeCliFs static collections to prevent memory leaks
+    FakeCliFs.files.clear();
+    FakeCliFs.directories.clear();
+    FakeCliExeca.entries.clear();
+    FakeCliHttpClient.entries.clear();
+    // Call parent reset method
+    return super.reset();
+  }
+
   static async bootstrap(options: Partial<TsED.Configuration> = {}) {
     options = resolveConfiguration({
       name: "tsed",

packages/cli-testing/src/FakeCliFs.ts

@@ -297,11 +297,6 @@ export class FakeCliFs implements FileSystemHost {
     return matchGlobs(allFilePaths, patterns, this.getCurrentDirectory());
   }
 
-  $onDestroy() {
-    FakeCliFs.files.clear();
-    FakeCliFs.directories.clear();
-  }
-
   private normalizePath(path: any): string {
     return (normalizePath(path) as string).replace(/^\//, "") as string;
   }

packages/cli-testing/src/FakeCliHttpClient.ts

@@ -1,7 +1,6 @@
 import {CliHttpClient, type CliHttpClientOptions} from "@tsed/cli-core";
-import type {OnDestroy} from "@tsed/di";
 
-export class FakeCliHttpClient extends CliHttpClient implements OnDestroy {
+export class FakeCliHttpClient extends CliHttpClient {
   static entries = new Map<string, (endpoint: string, options: CliHttpClientOptions) => any>();
 
   get(endpoint: string, options: CliHttpClientOptions = {}): Promise<any> {
@@ -21,8 +20,4 @@ export class FakeCliHttpClient extends CliHttpClient implements OnDestroy {
 
     return FakeCliHttpClient.entries.get(key)?.(endpoint, options);
   }
-
-  $onDestroy() {
-    FakeCliHttpClient.entries.clear();
-  }
 }