fix: amend typescript errors

src/auth/controllers/auth.controller.spec.ts

@@ -3,7 +3,7 @@ import { Test } from '@nestjs/testing';
 import { createMock } from 'ts-auto-mock';
 
 import { AuthController } from '@/auth/controllers/auth.controller';
-import { LoginUser } from '@/auth/dto/login-user.dto';
+import type { LoginUser } from '@/auth/dto/login-user.dto';
 import type { RegisterUser } from '@/auth/dto/register-user.dto';
 import type { UpdateUser } from '@/auth/dto/update-user.dto';
 import { User } from '@/auth/entities/user.entity';
@@ -47,6 +47,8 @@ describe('AuthController', () => {
               ),
           });
         }
+
+        return;
       })
       .compile();
 

src/auth/dto/login-user.dto.spec.ts

@@ -26,6 +26,8 @@ describe('Login user validations', () => {
             ),
           });
         }
+
+        return;
       })
       .compile();
 

src/auth/dto/register-user.dto.spec.ts

@@ -23,6 +23,8 @@ describe('Register user validations', () => {
         if (token === getRepositoryToken(User)) {
           return createMock<Repository<User>>();
         }
+
+        return;
       })
       .compile();
 

src/auth/dto/update-user.dto.spec.ts

@@ -26,6 +26,8 @@ describe('Update user validations', () => {
         if (token === getRepositoryToken(User)) {
           return createMock<Repository<User>>();
         }
+
+        return;
       })
       .compile();
 

src/auth/interceptors/current-user.interceptor.spec.ts

@@ -1,4 +1,4 @@
-import { CallHandler } from '@nestjs/common';
+import type { CallHandler } from '@nestjs/common';
 import { ExecutionContextHost } from '@nestjs/core/helpers/execution-context-host';
 import { createMocks } from 'node-mocks-http';
 import { lastValueFrom, of } from 'rxjs';
@@ -42,8 +42,8 @@ describe('UserInterceptor', () => {
   it("should not inject the user's id when there is no request's body", async () => {
     const { req, res } = createMocks({
       user,
-      body: undefined,
     });
+    delete req.body;
     const context = new ExecutionContextHost([req, res]);
     const next: CallHandler = {
       handle: () => of({}),

src/auth/interceptors/token.interceptor.spec.ts

@@ -6,7 +6,7 @@ import { createMocks } from 'node-mocks-http';
 import { lastValueFrom, of } from 'rxjs';
 import { createMock } from 'ts-auto-mock';
 
-import { User } from '@/auth/entities/user.entity';
+import type { User } from '@/auth/entities/user.entity';
 import { TokenInterceptor } from '@/auth/interceptors/token.interceptor';
 
 describe('TokenInterceptor', () => {
@@ -21,6 +21,8 @@ describe('TokenInterceptor', () => {
         if (token === JwtService) {
           return createMock<JwtService>();
         }
+
+        return;
       })
       .compile();
 

src/auth/pipes/strip-id.pipe.spec.ts

@@ -1,4 +1,4 @@
-import { ArgumentMetadata } from '@nestjs/common';
+import type { ArgumentMetadata } from '@nestjs/common';
 
 import { StripIdPipe } from '@/auth/pipes/strip-id.pipe';
 import { credentials } from '@/common/test-helpers';

src/auth/pipes/swap-password.pipe.spec.ts

@@ -1,4 +1,4 @@
-import { ArgumentMetadata } from '@nestjs/common';
+import type { ArgumentMetadata } from '@nestjs/common';
 
 import { SwapPasswordPipe } from '@/auth/pipes/swap-password.pipe';
 import { credentials } from '@/common/test-helpers';

src/auth/pipes/swap-password.pipe.ts

@@ -1,4 +1,5 @@
 import { ArgumentMetadata, Injectable, PipeTransform } from '@nestjs/common';
+import invariant from 'tiny-invariant';
 
 import type { UpdateUser } from '@/auth/dto/update-user.dto';
 
@@ -13,10 +14,11 @@ export class SwapPasswordPipe implements PipeTransform {
     if (metadata.type !== 'body') return value;
 
     if (this.#want2ChangePassword(value)) {
+      invariant(value.newPassword);
       value.password = value.newPassword;
       delete value.newPassword;
     } else {
-      delete value.password;
+      delete value['password'];
     }
 
     return value;
@@ -25,6 +27,6 @@ export class SwapPasswordPipe implements PipeTransform {
   #want2ChangePassword(
     value: Record<string, unknown>,
   ): value is Writeable<UpdateUser> {
-    return typeof value.newPassword === 'string';
+    return typeof value['newPassword'] === 'string';
   }
 }

src/auth/services/authentication.service.spec.ts

@@ -44,6 +44,8 @@ describe('AuthenticationService', () => {
             merge: jest.fn().mockImplementation(Object.assign),
           });
         }
+
+        return;
       })
       .compile();
 

src/auth/services/authentication.service.ts

@@ -2,9 +2,9 @@ import { Injectable } from '@nestjs/common';
 import { InjectRepository } from '@nestjs/typeorm';
 import type { Repository } from 'typeorm';
 
-import { LoginUser } from '@/auth/dto/login-user.dto';
-import { RegisterUser } from '@/auth/dto/register-user.dto';
-import { UpdateUser } from '@/auth/dto/update-user.dto';
+import type { LoginUser } from '@/auth/dto/login-user.dto';
+import type { RegisterUser } from '@/auth/dto/register-user.dto';
+import type { UpdateUser } from '@/auth/dto/update-user.dto';
 import { User } from '@/auth/entities/user.entity';
 import type { JwtPayload } from '@/auth/interfaces/jwt-payload.interface';
 

src/auth/strategies/jwt.strategy.ts

@@ -2,8 +2,8 @@ import { Injectable } from '@nestjs/common';
 import { PassportStrategy } from '@nestjs/passport';
 import { ExtractJwt, JwtFromRequestFunction, Strategy } from 'passport-jwt';
 
-import { User } from '@/auth/entities/user.entity';
-import { JwtPayload } from '@/auth/interfaces/jwt-payload.interface';
+import type { User } from '@/auth/entities/user.entity';
+import type { JwtPayload } from '@/auth/interfaces/jwt-payload.interface';
 import { AuthenticationService } from '@/auth/services/authentication.service';
 
 declare module 'express' {
@@ -16,7 +16,7 @@ declare module 'express' {
 export const JWT = 'jwt' as const;
 
 const extractJwtFromCookie: JwtFromRequestFunction = (request) => {
-  return request.signedCookies.token;
+  return request.signedCookies['token']!;
 };
 
 @Injectable()

src/auth/validators/is-already-register.validator.spec.ts

@@ -39,6 +39,8 @@ describe('IsAlreadyRegister', () => {
         if (token === getRepositoryToken(User)) {
           return createMock<Repository<User>>();
         }
+
+        return;
       })
       .compile();
 

src/auth/validators/is-already-register.validator.ts

@@ -47,7 +47,7 @@ export class IsAlreadyRegisterConstraint
   }
 }
 
-export function IsAlreadyRegister(options?: ValidationOptions) {
+export function IsAlreadyRegister(options: ValidationOptions = {}) {
   return function (object: object, propertyName: 'username' | 'email') {
     registerDecorator({
       target: object.constructor,

src/auth/validators/is-not-the-same.spec.ts

@@ -29,7 +29,7 @@ describe('IsNotTheSame', () => {
     const errors = validateSync(dto);
 
     expect(errors).toHaveLength(1);
-    expect(errors[0].constraints).toMatchObject({
+    expect(errors[0]!.constraints).toMatchObject({
       isNotTheSame: 'password must be different than username',
     });
   });

src/auth/validators/is-not-the-same.ts

@@ -25,7 +25,7 @@ export class IsNotTheSameConstraint implements ValidatorConstraintInterface {
 
 export function IsNotTheSame<Type>(
   property: keyof Type,
-  validationOptions?: ValidationOptions,
+  validationOptions: ValidationOptions = {},
 ) {
   return (object: object, propertyName: string) => {
     registerDecorator({

src/auth/validators/is-not-vulnerable.validator.ts

@@ -35,7 +35,7 @@ class IsNotVulnerableConstraint implements ValidatorConstraintInterface {
   }
 }
 
-export function IsNotVulnerable(options?: ValidationOptions) {
+export function IsNotVulnerable(options: ValidationOptions = {}) {
   return function (object: object, propertyName: string) {
     registerDecorator({
       target: object.constructor,

src/auth/validators/validate-credential.validator.spec.ts

@@ -55,6 +55,8 @@ describe('ValidateCredential', () => {
         if (token === getRepositoryToken(User)) {
           return createMock<Repository<User>>();
         }
+
+        return;
       })
       .compile();
 

src/auth/validators/validate-credential.validator.ts

@@ -57,7 +57,7 @@ export class ValidateCredentialConstraint
   }
 }
 
-export function ValidateCredential(options?: ValidationOptions) {
+export function ValidateCredential(options: ValidationOptions = {}) {
   return function (object: object, propertyName: 'username' | 'password') {
     registerDecorator({
       target: object.constructor,

src/blog/controllers/article.controller.spec.ts

@@ -3,8 +3,8 @@ import { createMock } from 'ts-auto-mock';
 
 import { User } from '@/auth/entities/user.entity';
 import { ArticleController } from '@/blog/controllers/article.controller';
-import { CreateArticle } from '@/blog/dto/create-article.dto';
-import { UpdateArticle } from '@/blog/dto/update-article.dto';
+import type { CreateArticle } from '@/blog/dto/create-article.dto';
+import type { UpdateArticle } from '@/blog/dto/update-article.dto';
 import { Article } from '@/blog/entities/article.entity';
 import { ArticleService } from '@/blog/services/article.service';
 
@@ -20,6 +20,8 @@ describe('ArticleController', () => {
         if (token === ArticleService) {
           return createMock<ArticleService>();
         }
+
+        return;
       })
       .compile();
 

src/blog/controllers/comment.controller.spec.ts

@@ -3,7 +3,7 @@ import { createMock } from 'ts-auto-mock';
 
 import { User } from '@/auth/entities/user.entity';
 import { CommentController } from '@/blog/controllers/comment.controller';
-import { CreateComment } from '@/blog/dto/create-comment';
+import type { CreateComment } from '@/blog/dto/create-comment';
 import { Article } from '@/blog/entities/article.entity';
 import { Comment } from '@/blog/entities/comment.entity';
 import { ArticleService } from '@/blog/services/article.service';
@@ -25,6 +25,8 @@ describe('CommentController', () => {
         if (token === ArticleService) {
           return createMock<ArticleService>();
         }
+
+        return;
       })
       .compile();
 

src/blog/controllers/comment.controller.ts

@@ -18,7 +18,7 @@ import { JWTAuthGuard } from '@/auth/guards/jwt-auth.guard';
 import { IsComment } from '@/blog/decorators/is-entity.decorator';
 import { CreateComment } from '@/blog/dto/create-comment';
 import type { Article } from '@/blog/entities/article.entity';
-import { Comment } from '@/blog/entities/comment.entity';
+import type { Comment } from '@/blog/entities/comment.entity';
 import { IsAuthorGuard } from '@/blog/guards/is-author.guard';
 import { SetArticleInterceptor } from '@/blog/interceptors/set-article.interceptor';
 import { SetAuthorInterceptor } from '@/blog/interceptors/set-author.interceptor';

src/blog/dto/create-comment.spec.ts

@@ -23,6 +23,8 @@ describe('CreateComment DTO', () => {
               ),
           });
         }
+
+        return;
       })
       .compile();
 

src/blog/fixtures/article.fixture.ts

@@ -2,7 +2,7 @@ import { plainToInstance } from 'class-transformer';
 import { fluse } from 'fluse';
 
 import { CreateArticle } from '@/blog/dto/create-article.dto';
-import { UpdateArticle } from '@/blog/dto/update-article.dto';
+import type { UpdateArticle } from '@/blog/dto/update-article.dto';
 import { fakerPlugin } from '@/common/fluse-plugin-faker';
 import { txtgenPlugin } from '@/common/fluse-plugin-txtgen';
 

src/blog/guards/is-author.guard.spec.ts

@@ -56,6 +56,8 @@ describe('IsAuthorGuard', () => {
                   ),
                 });
               }
+
+              return;
             }),
           }),
         },

src/blog/guards/is-author.guard.ts

@@ -21,7 +21,9 @@ export class IsAuthorGuard implements CanActivate {
   ) {}
 
   async canActivate(context: ExecutionContext): Promise<boolean> {
-    const request = context.switchToHttp().getRequest<Request>();
+    const request = context
+      .switchToHttp()
+      .getRequest<Request<{ id: string }>>();
     const kindOf = this.reflector.get<Entities>(
       ENTITY_METADATA_KEY,
       context.getHandler(),

src/blog/interceptors/set-article.interceptor.spec.ts

@@ -1,4 +1,4 @@
-import { CallHandler } from '@nestjs/common';
+import type { CallHandler } from '@nestjs/common';
 import { ExecutionContextHost } from '@nestjs/core/helpers/execution-context-host';
 import { createMocks } from 'node-mocks-http';
 import { lastValueFrom, of } from 'rxjs';

src/blog/interceptors/set-author.interceptor.spec.ts

@@ -1,4 +1,4 @@
-import { CallHandler } from '@nestjs/common';
+import type { CallHandler } from '@nestjs/common';
 import { ExecutionContextHost } from '@nestjs/core/helpers/execution-context-host';
 import { createMocks } from 'node-mocks-http';
 import { lastValueFrom, of } from 'rxjs';

src/blog/interceptors/set-author.interceptor.ts

@@ -5,7 +5,7 @@ import {
   NestInterceptor,
 } from '@nestjs/common';
 import type { Request } from 'express';
-import { Observable } from 'rxjs';
+import type { Observable } from 'rxjs';
 import invariant from 'tiny-invariant';
 
 @Injectable()

src/blog/pipes/article.pipe.spec.ts

@@ -26,6 +26,8 @@ describe('ArticlePipe', () => {
             ),
           });
         }
+
+        return;
       })
       .compile();
 

src/blog/seeders/article.seeder.ts

@@ -21,7 +21,7 @@ export class ArticleSeeder implements Seeder {
 However, cats have begun to rent blueberries over the past few months, specifically for sharks associated with their puppies.
 Some assert that however, ducks have begun to rent octopus over the past few months, specifically for sharks associated with their snakes.
 Though we assume the latter, however, birds have begun to rent kumquats over the past few months, specifically for chickens associated with their watermelons.`,
-      author: users.find((user) => user.username === 'john-doe'),
+      author: users.find((user) => user.username === 'john-doe')!,
     });
     await articleFactory.save({
       id: '31a10506-c334-4841-97a6-144a55bf4ebb',
@@ -32,7 +32,7 @@ Though we assume the latter, however, birds have begun to rent kumquats over the
 Of course, however, plums have begun to rent raspberries over the past few months, specifically for giraffes associated with their lions.
 This is not to discredit the idea that however, alligators have begun to rent sharks over the past few months, specifically for sheeps associated with their monkeys.
 However, pomegranates have begun to rent sheeps over the past few months, specifically for apricots associated with their grapes.`,
-      author: users.find((user) => user.username === 'jane-doe'),
+      author: users.find((user) => user.username === 'jane-doe')!,
     });
 
     for (const author of users) {

src/blog/seeders/comment.seeder.ts

@@ -30,21 +30,21 @@ export class CommentSeeder implements Seeder {
     await Promise.allSettled([
       commentFactory.save({
         id: '9395e782-367b-4487-a048-242e37169109',
-        article: article1,
-        author: jane,
+        article: article1!,
+        author: jane!,
       }),
       commentFactory.saveMany(14, {
-        article: article1,
-        author: jane,
+        article: article1!,
+        author: jane!,
       }),
       commentFactory.save({
         id: '2cce7079-b434-42fb-85e3-8d1aadd7bb8a',
-        article: article2,
-        author: john,
+        article: article2!,
+        author: john!,
       }),
       commentFactory.saveMany(14, {
-        article: article2,
-        author: john,
+        article: article2!,
+        author: john!,
       }),
     ]);
   }