10197260214

Committed 01 Aug 2024 11:15AM UTC coverage: 47.16% (-0.1%) from 47.286%

Build Type

push

github

Committed by

Commit Message

Allow optional signature when registering notifications

Run Details

454 of 2822 branches covered (16.09%)

Branch coverage included in aggregate %.

6 of 19 new or added lines in 5 files covered. (31.58%)

7 existing lines in 4 files now uncovered.

4461 of 7600 relevant lines covered (58.7%)

12.68 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

33.33

/src/routes/notifications/guards/notifications.guard.ts

1	import { AuthController } from '@/routes/auth/auth.controller';	16✔
2	import { AuthGuard } from '@/routes/auth/guards/auth.guard';	16✔
3	import { CanActivate, ExecutionContext, Injectable } from '@nestjs/common';	16✔
4	import { Request } from 'express';
5
6	@Injectable()
7	export class NotificationsGuard implements CanActivate {	16✔
NEW 8	constructor(private readonly authGuard: AuthGuard) {}	×
9	canActivate(context: ExecutionContext): boolean {
NEW 10	const request: Request = context.switchToHttp().getRequest();	×
11
12	const accessToken: string \| undefined =
NEW 13	request.cookies[AuthController.ACCESS_TOKEN_COOKIE_NAME];	×
14
15	/**
16	* If there is no access token, we allow the request to proceed as
17	* users can register for "public" notifications having not signed.
18	*/
NEW 19	if (!accessToken) {	×
NEW 20	return true;	×
21	}
22
NEW 23	return this.authGuard.canActivate(context);	×
24	}
25	}