From aadac0835933d8e5568f13a341763f56ebdf629f Mon Sep 17 00:00:00 2001 From: Fares Eidi Date: Tue, 28 Nov 2023 02:42:40 +0100 Subject: [PATCH] fix style rule issue (line breaks) - p/r review --- .../api/src/decorators/has-permission.decorator.ts | 3 ++- apps/api/src/guards/has-permissions.guard.spec.ts | 11 ++++------- apps/api/src/guards/has-permissions.guard.ts | 14 +++++++++++--- 3 files changed, 17 insertions(+), 11 deletions(-) diff --git a/apps/api/src/decorators/has-permission.decorator.ts b/apps/api/src/decorators/has-permission.decorator.ts index cd350edc6..b5dfdfe92 100644 --- a/apps/api/src/decorators/has-permission.decorator.ts +++ b/apps/api/src/decorators/has-permission.decorator.ts @@ -1,3 +1,4 @@ import { SetMetadata } from '@nestjs/common'; export const HAS_PERMISSION_KEY = 'has_permission'; -export const HasPermission = (permission: string) => SetMetadata(HAS_PERMISSION_KEY, permission); +export const HasPermission = (permission: string) => + SetMetadata(HAS_PERMISSION_KEY, permission); diff --git a/apps/api/src/guards/has-permissions.guard.spec.ts b/apps/api/src/guards/has-permissions.guard.spec.ts index 18c694f35..ab86d4956 100644 --- a/apps/api/src/guards/has-permissions.guard.spec.ts +++ b/apps/api/src/guards/has-permissions.guard.spec.ts @@ -10,10 +10,7 @@ describe('HasPermissionsGuard', () => { beforeEach(async () => { const module: TestingModule = await Test.createTestingModule({ - providers: [ - HasPermissionsGuard, - Reflector, - ], + providers: [HasPermissionsGuard, Reflector] }).compile(); guard = module.get(HasPermissionsGuard); @@ -28,9 +25,9 @@ describe('HasPermissionsGuard', () => { return new ExecutionContextHost([ { user: { - permissions, // Set user permissions based on the argument - }, - }, + permissions // Set user permissions based on the argument + } + } ]); } diff --git a/apps/api/src/guards/has-permissions.guard.ts b/apps/api/src/guards/has-permissions.guard.ts index 545bed0c7..eeb79eda1 100644 --- a/apps/api/src/guards/has-permissions.guard.ts +++ b/apps/api/src/guards/has-permissions.guard.ts @@ -1,4 +1,9 @@ -import { Injectable, CanActivate, ExecutionContext, HttpException } from '@nestjs/common'; +import { + Injectable, + CanActivate, + ExecutionContext, + HttpException +} from '@nestjs/common'; import { Reflector } from '@nestjs/core'; import { StatusCodes, getReasonPhrase } from 'http-status-codes'; import { HAS_PERMISSION_KEY } from '@ghostfolio/api/decorators/has-permission.decorator'; @@ -6,10 +11,13 @@ import { hasPermission } from '@ghostfolio/common/permissions'; @Injectable() export class HasPermissionsGuard implements CanActivate { - constructor(private reflector: Reflector) { } + constructor(private reflector: Reflector) {} canActivate(context: ExecutionContext): boolean { - const requiredPermission = this.reflector.get(HAS_PERMISSION_KEY, context.getHandler()); + const requiredPermission = this.reflector.get( + HAS_PERMISSION_KEY, + context.getHandler() + ); if (!requiredPermission) { return true; // No specific permissions required