@lenne.tech/nest-server 8.6.1 → 8.6.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lenne.tech/nest-server",
-  "version": "8.6.1",
+  "version": "8.6.4",
   "description": "Modern, fast, powerful Node.js web framework in TypeScript based on Nest with a GraphQL API and a connection to MongoDB (or other databases).",
   "keywords": [
     "node",
@@ -32,9 +32,9 @@
     "test": "NODE_ENV=local jest",
     "test:cov": "NODE_ENV=local jest --coverage",
     "test:debug": "NODE_ENV=local node --inspect-brk -r tsconfig-paths/register -r ts-node/register node_modules/.bin/jest --runInBand",
-    "test:e2e": "NODE_ENV=local jest --config jest-e2e.json",
-    "test:e2e-cov": "NODE_ENV=local jest --config jest-e2e.json --coverage",
-    "test:ci": "NODE_ENV=local jest --config jest-e2e.json --ci",
+    "test:e2e": "NODE_ENV=local jest --config jest-e2e.json --forceExit --detectOpenHandles",
+    "test:e2e-cov": "NODE_ENV=local jest --config jest-e2e.json --coverage --forceExit --detectOpenHandles",
+    "test:ci": "NODE_ENV=local jest --config jest-e2e.json --ci --forceExit --detectOpenHandles",
     "test:watch": "NODE_ENV=local jest --watch",
     "prepack": "npm run prestart:prod",
     "prepare": "husky install",

package/src/core/common/decorators/restricted.decorator.ts

@@ -134,7 +134,7 @@ export const checkRestricted = (
           return (
             typeof item === 'object' &&
             // Check if object is valid
-            item.memberOf.length &&
+            item.memberOf?.length &&
             // Check if processType is specified and is valid for current process
             (config.processType && item.processType ? config.processType === item.processType : true)
           );

package/src/core/common/models/core-model.model.ts

@@ -34,14 +34,9 @@ export abstract class CoreModel {
       mapId?: boolean;
     } = {}
   ): T {
-    const config = {
-      init: true,
-      ...options,
-    };
-
     const item = options.item || new this();
     delete options.item;
-    return item.map(data, config);
+    return item.map(data, options);
   }
 
   /**
@@ -64,14 +59,9 @@ export abstract class CoreModel {
       mapId?: boolean;
     } = {}
   ): T {
-    const config = {
-      init: true,
-      ...options,
-    };
-
     const item = options.item || new this();
     delete options.item;
-    return item.mapDeep(data, config);
+    return item.mapDeep(data, options);
   }
 
   /**

package/src/core/common/services/module.service.ts

@@ -88,7 +88,7 @@ export abstract class ModuleService<T extends CoreModel = any> {
       if (!opts.targetModel && config.inputType) {
         opts.targetModel = config.inputType;
       }
-      await this.prepareInput(config.input, opts);
+      config.input = await this.prepareInput(config.input, opts);
     }
 
     // Get DB object

package/src/core/modules/user/inputs/core-user.input.ts

@@ -1,6 +1,7 @@
-import { IsEmail, IsOptional } from 'class-validator';
 import { Field, InputType } from '@nestjs/graphql';
+import { IsEmail, IsOptional } from 'class-validator';
 import { Restricted } from '../../../common/decorators/restricted.decorator';
+import { ProcessType } from '../../../common/enums/process-type.enum';
 import { RoleEnum } from '../../../common/enums/role.enum';
 import { CoreInput } from '../../../common/inputs/core-input.input';
 
@@ -38,7 +39,7 @@ export abstract class CoreUserInput extends CoreInput {
   /**
    * Roles of the user
    */
-  @Restricted(RoleEnum.ADMIN)
+  @Restricted({ roles: RoleEnum.ADMIN, processType: ProcessType.INPUT })
   @Field((type) => [String], { description: 'Roles of the user', nullable: true })
   @IsOptional()
   roles?: string[] = undefined;

package/src/server/modules/user/user.service.ts

@@ -50,7 +50,7 @@ export class UserService extends CoreUserService<User, UserInput, UserCreateInpu
     // and could not exist as currentUser before
     if (!user.createdBy) {
       await this.mainDbModel.findByIdAndUpdate(user.id, { createdBy: user.id });
-      user = await this.get(user.id, { ...serviceOptions, currentUser: serviceOptions.currentUser || user });
+      user = await this.get(user.id, { ...serviceOptions, currentUser: serviceOptions?.currentUser || user });
     }
 
     // Publish action