From 8483927823aeadac916a983f56bd388ecef3b851 Mon Sep 17 00:00:00 2001 From: Mathis HERRIOT <197931332+0x485254@users.noreply.github.com> Date: Tue, 20 Jan 2026 21:42:16 +0100 Subject: [PATCH] fix(tests): replace `any` with `Record` in repository tests - Updated type assertions in repository test files to use `Record` instead of `any`. --- backend/src/api-keys/repositories/api-keys.repository.spec.ts | 2 +- .../src/categories/repositories/categories.repository.spec.ts | 2 +- .../src/favorites/repositories/favorites.repository.spec.ts | 3 +-- backend/src/media/media.controller.spec.ts | 4 +--- backend/src/reports/repositories/reports.repository.spec.ts | 3 +-- 5 files changed, 5 insertions(+), 9 deletions(-) diff --git a/backend/src/api-keys/repositories/api-keys.repository.spec.ts b/backend/src/api-keys/repositories/api-keys.repository.spec.ts index 0cd9d04..c70b2ed 100644 --- a/backend/src/api-keys/repositories/api-keys.repository.spec.ts +++ b/backend/src/api-keys/repositories/api-keys.repository.spec.ts @@ -23,7 +23,7 @@ describe("ApiKeysRepository", () => { // biome-ignore lint/suspicious/noThenProperty: Necessary to mock Drizzle's awaitable query builder Object.defineProperty(obj, "then", { value: function (onFulfilled: (arg0: unknown) => void) { - const result = (this as any).execute(); + const result = (this as Record).execute(); return Promise.resolve(result).then(onFulfilled); }, configurable: true, diff --git a/backend/src/categories/repositories/categories.repository.spec.ts b/backend/src/categories/repositories/categories.repository.spec.ts index 9585965..fb1f9ca 100644 --- a/backend/src/categories/repositories/categories.repository.spec.ts +++ b/backend/src/categories/repositories/categories.repository.spec.ts @@ -24,7 +24,7 @@ describe("CategoriesRepository", () => { // biome-ignore lint/suspicious/noThenProperty: Necessary to mock Drizzle's awaitable query builder Object.defineProperty(obj, "then", { value: function (onFulfilled: (arg0: unknown) => void) { - const result = (this as any).execute(); + const result = (this as Record).execute(); return Promise.resolve(result).then(onFulfilled); }, configurable: true, diff --git a/backend/src/favorites/repositories/favorites.repository.spec.ts b/backend/src/favorites/repositories/favorites.repository.spec.ts index 0abb289..4f5f0dc 100644 --- a/backend/src/favorites/repositories/favorites.repository.spec.ts +++ b/backend/src/favorites/repositories/favorites.repository.spec.ts @@ -21,10 +21,9 @@ describe("FavoritesRepository", () => { const wrapWithThen = (obj: unknown) => { // biome-ignore lint/suspicious/noThenProperty: Necessary to mock Drizzle's awaitable query builder - // biome-ignore lint/suspicious/noExplicitAny: Necessary to mock Drizzle's awaitable query builder Object.defineProperty(obj, "then", { value: function (onFulfilled: (arg0: unknown) => void) { - const result = (this as any).execute(); + const result = (this as Record).execute(); return Promise.resolve(result).then(onFulfilled); }, configurable: true, diff --git a/backend/src/media/media.controller.spec.ts b/backend/src/media/media.controller.spec.ts index 84199b2..5a6a8c4 100644 --- a/backend/src/media/media.controller.spec.ts +++ b/backend/src/media/media.controller.spec.ts @@ -51,9 +51,7 @@ describe("MediaController", () => { it("should throw NotFoundException if path is missing", async () => { const res = {} as unknown as Response; - await expect(controller.getFile("", res)).rejects.toThrow( - NotFoundException, - ); + await expect(controller.getFile("", res)).rejects.toThrow(NotFoundException); }); it("should throw NotFoundException if file is not found", async () => { diff --git a/backend/src/reports/repositories/reports.repository.spec.ts b/backend/src/reports/repositories/reports.repository.spec.ts index 88e9683..bf55eba 100644 --- a/backend/src/reports/repositories/reports.repository.spec.ts +++ b/backend/src/reports/repositories/reports.repository.spec.ts @@ -23,10 +23,9 @@ describe("ReportsRepository", () => { const wrapWithThen = (obj: unknown) => { // biome-ignore lint/suspicious/noThenProperty: Necessary to mock Drizzle's awaitable query builder - // biome-ignore lint/suspicious/noExplicitAny: Necessary to mock Drizzle's awaitable query builder Object.defineProperty(obj, "then", { value: function (onFulfilled: (arg0: unknown) => void) { - const result = (this as any).execute(); + const result = (this as Record).execute(); return Promise.resolve(result).then(onFulfilled); }, configurable: true,