auth module all tests ok

Author: Me-Baran

Diff for: .env.development

@@ -7,7 +7,7 @@ DB_TYPE=postgres
 DB_HOST=localhost
 DB_PORT=5432
 DB_USERNAME=postgres
-DB_PASSWORD=postgres
+DB_PASSWORD=example
 DB_NAME=movie_management
 DB_SYNCHRONIZE=true
 DB_LOGGING=true

Diff for: :memory

@@ -87,6 +87,9 @@
       "**/*.(t|j)s"
     ],
     "coverageDirectory": "../coverage",
-    "testEnvironment": "node"
+    "testEnvironment": "node",
+    "moduleNameMapper": {
+      "^src/(.*)$": "<rootDir>/$1"
+    }
   }
 }

Diff for: drafts.ts

@@ -21,17 +21,21 @@ import { TimeoutInterceptor } from './common/filters/timeout.interceptor';
     TypeOrmModule.forRootAsync({
       imports: [ConfigModule],
       inject: [ConfigService],
-      useFactory: (configService: ConfigService) => ({
-        type: configService.get('DB_TYPE', 'postgres') as any,
-        host: configService.get('DB_HOST', 'localhost'),
-        port: configService.get('DB_PORT', 5432),
-        username: configService.get('DB_USERNAME', 'postgres'),
-        password: configService.get('DB_PASSWORD', 'postgres'),
-        databse: configService.get('DB_NAME', 'movie_management'),
-        entities: [__dirname + '/**/*.typeorm-entity{.ts}'],
-        synchronize: configService.get<boolean>('DB_SYNCHRONIZE', false),
-        logging: configService.get<boolean>('DB_LOGGING', false)
-      })
+      useFactory: (configService: ConfigService) => {
+        const dbConfig = {
+          type: configService.get<string>('DB_TYPE', 'postgres') as 'postgres',
+          host: configService.get<string>('DB_HOST', 'localhost'),
+          port: configService.get<number>('DB_PORT', 5432),
+          username: configService.get<string>('DB_USERNAME', 'postgres'),
+          password: configService.get<string>('DB_PASSWORD', 'example'),
+          database: configService.get<string>('DB_NAME', 'movie_management'),
+          entities: [__dirname + '/**/*.typeorm-entity.{ts,js}'],
+          synchronize: configService.get<boolean>('DB_SYNCHRONIZE', false),
+          logging: configService.get<boolean>('DB_LOGGING', false),
+        };
+
+        return dbConfig;
+      },
     }),
 
     // Domain modules
@@ -59,7 +63,7 @@ import { TimeoutInterceptor } from './common/filters/timeout.interceptor';
     }
   ],
 })
-export class AppModule {}
+export class AppModule { }
 
 
 /**

Diff for: package.json

@@ -12,7 +12,7 @@ import { Roles } from "../../secondary/security/utils/decorators/roles.decorator
 export class UserController {
     constructor(private readonly userService: Userservice) {}
 
-    @Get('id')
+    @Get(':id')
     @Roles('manager') // This attaches metadata('roles': ['manager']) to the method 
     @ApiOperation({summary: 'Get user by ID (managers only)'})
     @ApiResponse({
@@ -31,12 +31,12 @@ export class UserController {
     @ApiResponse({status: 401, description: 'Unauthorized'})
     @ApiResponse({status: 403, description: 'Forbidden - Requires manager role'})
     @ApiResponse({status: 404, description: 'User not found'})
-    async getUserById(@Param('id') id: 'string') {
+    async getUserById(@Param('id') id: string) {
         const user = await this.userService.getUserById(id);
         return {
             id: user.getId(),
             username: user.getUsername(),
-            role: user.getRole(),
+            role: user.getRole().getValue(),
             age: user.getAge()
         };
     }

Diff for: src/app.module.ts

@@ -27,6 +27,7 @@ export class JwtStrategy extends PassportStrategy(Strategy) {
     async validate(payload: any) {
         try {
             const user = await this.authService.validateUser(payload.sub);
+            // The returned user object is attached to the request as req.user
             return {
                 id: user.getId(),
                 username: user.getUsername(),

Diff for: src/modules/auth/adapters/primary/rest/user.controller.ts

@@ -1,4 +1,4 @@
-import { CanActivate, ExecutionContext } from "@nestjs/common";
+import { CanActivate, ExecutionContext, Injectable } from "@nestjs/common";
 import { Reflector } from "@nestjs/core";
 import { ROLES_KEY } from "../decorators/roles.decorator";
 
@@ -9,6 +9,7 @@ import { ROLES_KEY } from "../decorators/roles.decorator";
  * @see [1] Reflection metadata system
  * @see [2] Execution context types
  */
+@Injectable()
 export class RolesGuard implements CanActivate {
     constructor(private reflector: Reflector) { }
 

Diff for: src/modules/auth/adapters/secondary/security/jwt-adapter/jwt.strategy.ts

@@ -1,8 +1,8 @@
-import { NotFoundException } from "@nestjs/common";
+import { Injectable, NotFoundException } from "@nestjs/common";
 import { IUserRepository } from "../ports/user-repository.interface";
 import { User } from "../../domain/models/user.entity";
-import { use } from "passport";
 
+@Injectable()
 export class Userservice {
     constructor(private readonly userRepository: IUserRepository) {}
 

Diff for: src/modules/auth/adapters/secondary/security/utils/guards/roles.guard.ts

@@ -22,6 +22,7 @@ import { UserMapper } from "./adapters/secondary/persistence/user.mapper";
 import { UserCreatedHandler } from "./application/event-handlers/user-created.handler";
 import { LoginFailedHandler } from "./application/event-handlers/login-failed.handler";
 import { JwtStrategy } from "./adapters/secondary/security/jwt-adapter/jwt.strategy";
+import { RolesGuard } from "./adapters/secondary/security/utils/guards/roles.guard";
 
 const handlers = [UserCreatedHandler, LoginFailedHandler];
 
@@ -66,7 +67,10 @@ const handlers = [UserCreatedHandler, LoginFailedHandler];
         JwtStrategy,
 
         // Event handlers
-        ...handlers
+        ...handlers,
+
+        // Guard
+       //RolesGuard
     ],
     exports: [AuthService, Userservice]
 })