Remove any

w3bdesign · Dec 9, 2024 · 05a5f67 · 05a5f67
1 parent a0447ad
commit 05a5f67
Showing 1 changed file with 27 additions and 9 deletions.
diff --git a/backend/src/bookings/bookings.module.spec.ts b/backend/src/bookings/bookings.module.spec.ts
@@ -1,31 +1,44 @@
 import { BookingsModule } from "./bookings.module";
 import { TypeOrmModule } from "@nestjs/typeorm";
 import { BookingsService } from "./bookings.service";
+import { DynamicModule, ForwardReference, Type } from "@nestjs/common";
+
+type ModuleImport =
+  | Type<unknown>
+  | DynamicModule
+  | Promise<DynamicModule>
+  | ForwardReference;
 
 describe("BookingsModule", () => {
   it("should have correct module metadata", () => {
     // Get module metadata directly from the decorator
-    const moduleDecorator = Reflect.getMetadata("imports", BookingsModule);
+    const moduleDecorator = Reflect.getMetadata(
+      "imports",
+      BookingsModule
+    ) as ModuleImport[];
 
     // Check TypeOrmModule.forFeature
     const typeOrmFeature = moduleDecorator.find(
-      (item: any) =>
-        item && typeof item === "object" && item.module === TypeOrmModule
+      (item): item is DynamicModule =>
+        item !== null &&
+        typeof item === "object" &&
+        "module" in item &&
+        item.module === TypeOrmModule
     );
     expect(typeOrmFeature).toBeDefined();
 
     // Get all module names
     const moduleNames = moduleDecorator
-      .map((item: any) => {
-        if (item && typeof item === "function") {
+      .map((item): string | null => {
+        if (typeof item === "function") {
           return item.name;
         }
         if (item && typeof item === "object" && "name" in item) {
           return String(item.name);
         }
         return null;
       })
-      .filter(Boolean);
+      .filter((name): name is string => Boolean(name));
 
     // Check for expected modules
     const expectedModules = [
@@ -40,13 +53,18 @@ describe("BookingsModule", () => {
   });
 
   it("should export BookingsService and TypeOrmModule", () => {
-    const exports = Reflect.getMetadata("exports", BookingsModule);
+    const exports = Reflect.getMetadata("exports", BookingsModule) as Array<
+      Type<unknown> | DynamicModule
+    >;
     expect(exports).toContain(BookingsService);
 
     const hasTypeOrmExport = exports.some(
-      (exp: any) =>
+      (exp): boolean =>
         exp === TypeOrmModule ||
-        (exp && typeof exp === "object" && exp.module === TypeOrmModule)
+        (exp !== null &&
+          typeof exp === "object" &&
+          "module" in exp &&
+          exp.module === TypeOrmModule)
     );
     expect(hasTypeOrmExport).toBe(true);
   });