Browse Source

fix style rule issue (line breaks) - p/r review

pull/2693/head
Fares Eidi 2 years ago
parent
commit
aadac08359
  1. 3
      apps/api/src/decorators/has-permission.decorator.ts
  2. 11
      apps/api/src/guards/has-permissions.guard.spec.ts
  3. 12
      apps/api/src/guards/has-permissions.guard.ts

3
apps/api/src/decorators/has-permission.decorator.ts

@ -1,3 +1,4 @@
import { SetMetadata } from '@nestjs/common'; import { SetMetadata } from '@nestjs/common';
export const HAS_PERMISSION_KEY = 'has_permission'; 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);

11
apps/api/src/guards/has-permissions.guard.spec.ts

@ -10,10 +10,7 @@ describe('HasPermissionsGuard', () => {
beforeEach(async () => { beforeEach(async () => {
const module: TestingModule = await Test.createTestingModule({ const module: TestingModule = await Test.createTestingModule({
providers: [ providers: [HasPermissionsGuard, Reflector]
HasPermissionsGuard,
Reflector,
],
}).compile(); }).compile();
guard = module.get<HasPermissionsGuard>(HasPermissionsGuard); guard = module.get<HasPermissionsGuard>(HasPermissionsGuard);
@ -28,9 +25,9 @@ describe('HasPermissionsGuard', () => {
return new ExecutionContextHost([ return new ExecutionContextHost([
{ {
user: { user: {
permissions, // Set user permissions based on the argument permissions // Set user permissions based on the argument
}, }
}, }
]); ]);
} }

12
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 { Reflector } from '@nestjs/core';
import { StatusCodes, getReasonPhrase } from 'http-status-codes'; import { StatusCodes, getReasonPhrase } from 'http-status-codes';
import { HAS_PERMISSION_KEY } from '@ghostfolio/api/decorators/has-permission.decorator'; import { HAS_PERMISSION_KEY } from '@ghostfolio/api/decorators/has-permission.decorator';
@ -9,7 +14,10 @@ export class HasPermissionsGuard implements CanActivate {
constructor(private reflector: Reflector) {} constructor(private reflector: Reflector) {}
canActivate(context: ExecutionContext): boolean { canActivate(context: ExecutionContext): boolean {
const requiredPermission = this.reflector.get<string>(HAS_PERMISSION_KEY, context.getHandler()); const requiredPermission = this.reflector.get<string>(
HAS_PERMISSION_KEY,
context.getHandler()
);
if (!requiredPermission) { if (!requiredPermission) {
return true; // No specific permissions required return true; // No specific permissions required

Loading…
Cancel
Save