fix: settings typecheck errors

api/src/seeder.ts

@@ -23,6 +23,9 @@ import { NlpValueSeeder } from './nlp/seeds/nlp-value.seed';
 import { nlpValueModels } from './nlp/seeds/nlp-value.seed-model';
 import { SettingSeeder } from './setting/seeds/setting.seed';
 import { DEFAULT_SETTINGS } from './setting/seeds/setting.seed-model';
+import { PermissionCreateDto } from './user/dto/permission.dto';
+import { Model } from './user/schemas/model.schema';
+import { Role } from './user/schemas/role.schema';
 import { ModelSeeder } from './user/seeds/model.seed';
 import { modelModels } from './user/seeds/model.seed-model';
 import { PermissionSeeder } from './user/seeds/permission.seed';
@@ -69,19 +72,19 @@ export async function seedDatabase(app: INestApplicationContext) {
     throw e;
   }
 
-  const models = await modelSeeder.findAll();
-  const roles = await roleSeeder.findAll();
-  const adminRole = roles.find(({ name }) => name === 'admin');
-  const managerRole = roles.find(({ name }) => name === 'manager');
+  const models = (await modelSeeder.findAll()) as Model[];
+  const roles = (await roleSeeder.findAll()) as Role[];
+  const adminRole = roles.find(({ name }) => name === 'admin') as Role;
+  const managerRole = roles.find(({ name }) => name === 'manager') as Role;
   const managerModels = models.filter(
     (model) => !['Role', 'User', 'Permission'].includes(model.name),
   );
-  const roleModelsCombinations: [string, string][] = [
+  const roleModelsCombinations = [
     ...models.map((model) => [model.id, adminRole.id]),
     ...managerModels.map((model) => [model.id, managerRole.id]),
   ] as [string, string][];
 
-  const permissionSeeds = roleModelsCombinations.reduce(
+  const permissionSeeds = roleModelsCombinations.reduce<PermissionCreateDto[]>(
     (acc, [modelId, roleId]) => {
       return acc.concat(permissionModels(modelId, roleId));
     },

api/src/setting/controllers/setting.controller.spec.ts

@@ -23,7 +23,7 @@ import {
 } from '@/utils/test/test';
 
 import { SettingRepository } from '../repositories/setting.repository';
-import { SettingModel } from '../schemas/setting.schema';
+import { Setting, SettingModel } from '../schemas/setting.schema';
 import { SettingSeeder } from '../seeds/setting.seed';
 import { SettingService } from '../services/setting.service';
 
@@ -100,8 +100,9 @@ describe('SettingController', () => {
       const payload = {
         value: 'updated setting value',
       };
-      const id = (await settingService.findOne({ value: 'admin@example.com' }))
-        .id;
+      const { id } = (await settingService.findOne({
+        value: 'admin@example.com',
+      })) as Setting;
       const result = await settingController.updateOne(id, payload);
 
       expect(settingService.updateOne).toHaveBeenCalledWith(id, payload);

api/src/setting/repositories/setting.repository.ts

@@ -49,10 +49,15 @@ export class SettingRepository extends BaseRepository<Setting> {
   ): Promise<void> {
     if (!Array.isArray(updates)) {
       const payload = updates.$set;
-      if (typeof payload.value !== 'undefined') {
-        const { type } =
-          'type' in payload ? payload : await this.findOne(criteria);
-        this.validateSettingValue(type, payload.value);
+      if (payload && 'value' in payload) {
+        const hasType = 'type' in payload;
+        if (hasType) {
+          this.validateSettingValue(payload.type, payload.value);
+        } else {
+          const setting = await this.findOne(criteria);
+          if (setting && 'type' in setting)
+            this.validateSettingValue(setting.type, payload.value);
+        }
       }
     }
   }

api/src/setting/schemas/types.ts

@@ -77,7 +77,7 @@ export interface NumberSetting extends Setting {
   type: SettingType.number;
   value: number;
   options: never;
-  config?: {
+  config: {
     min: number;
     max: number;
     step: number;

api/src/setting/services/setting.service.spec.ts

@@ -35,8 +35,8 @@ describe('SettingService', () => {
   const commonAttributes = {
     type: SettingType.text,
     id: '',
-    createdAt: undefined,
-    updatedAt: undefined,
+    createdAt: new Date(),
+    updatedAt: new Date(),
   };
 
   beforeAll(async () => {

api/src/utils/generics/base-repository.ts

@@ -414,10 +414,24 @@ export abstract class BaseRepository<
     filters: TFilterQuery<T>,
     { skip, limit, sort }: PageQueryDto<T>,
   ): Query<T[], T, object, T, 'find', object> {
-    return this.findQuery(filters)
-      .skip(skip)
-      .limit(limit)
-      .sort([sort] as [string, SortOrder][]);
+    if (skip && limit) {
+      return this.findQuery(filters)
+        .skip(skip)
+        .limit(limit)
+        .sort([sort] as [string, SortOrder][]);
+    }
+    if (skip) {
+      return this.findQuery(filters)
+        .skip(skip)
+        .sort([sort] as [string, SortOrder][]);
+    }
+
+    if (limit) {
+      return this.findQuery(filters)
+        .limit(limit)
+        .sort([sort] as [string, SortOrder][]);
+    }
+    return this.findQuery(filters).sort([sort] as [string, SortOrder][]);
   }
 
   /**

api/src/utils/pagination/pagination-query.dto.ts

@@ -18,5 +18,5 @@ export type QuerySortDto<T> = [
 export type PageQueryDto<T> = {
   skip: number | undefined;
   limit: number | undefined;
-  sort: QuerySortDto<T>;
+  sort?: QuerySortDto<T>;
 };