Skip to content

Remove lock emoji from project title in README#12

Merged
DannielEU merged 1 commit intodevelopfrom
DannielEU-patch-2
Nov 23, 2025
Merged

Remove lock emoji from project title in README#12
DannielEU merged 1 commit intodevelopfrom
DannielEU-patch-2

Conversation

@DannielEU
Copy link
Collaborator

No description provided.

@DannielEU DannielEU requested a review from Nella1414 November 23, 2025 00:45
@sonarqubecloud
Copy link

@DannielEU DannielEU merged commit c996b8b into develop Nov 23, 2025
1 check passed
@DannielEU DannielEU deleted the DannielEU-patch-2 branch November 23, 2025 00:50
DannielEU added a commit that referenced this pull request Nov 23, 2025
* docs: agregar documentación técnica completa con Swagger y convenciones de commits

- README.md con secciones de instalación, ejecución y testing
- Swagger UI configurado en /api/docs con decoradores en endpoints y DTOs
- COMMITS.md con guía de Conventional Commits y ejemplos prácticos
- Documentación adicional en carpeta docs/

* feat: add comprehensive unit tests for authentication module

Add 140 unit tests covering all critical authentication components with 68% overall code coverage.

Tests added:
- AuthService: Google OAuth validation, JWT generation, user CRUD operations
- AuthController: Google authentication endpoints and callbacks
- Guards: JwtAuthGuard, RolesGuard, GoogleAuthGuard
- Strategies: JwtStrategy, GoogleStrategy
- Decorators: @public, @roles, @getuser
- DTOs: GoogleUserDto, AuthResponseDto validation
- Configuration: Environment variable validation
- PrismaService: Database connection lifecycle

Test Infrastructure:
- Configure Jest with moduleNameMapper for path resolution
- Add reflect-metadata setup for decorators
- Create mock PrismaClient for testing without database
- Add test coverage reporting

Coverage:
- 100% coverage on AuthService, AuthController, all Guards, all Strategies, DTOs, and PrismaService
- 68.16% overall statement coverage
- 84.61% branch coverage
- 11 test suites, 140 tests passing

All tests pass successfully.

* refactor: add code structure for improved readability and maintainability

* refactor(google-auth): streamline handleRequest logging and error handling

* Add comment for PrismaService unit tests

Added a comment for unit tests related to database connection handling.

* Add unit tests for environment variable validation

Added unit tests to ensure all required environment variables exist, preventing production errors.

* fix - Ignacio Castillo: Pequeño cambio en el main

* feat: add @nestjs/swagger and @microsoft/tsdoc dependencies; update auth.controller imports

* fix: format ApiOperation and ApiResponse decorators in AuthController for consistency

* feat: implement Users module with controller and service for user management

* Feat/gestion usuarios (#9)

* feat(gestionUsuario): administración de usuarios

* feat(biografia):como usuario quiero editar mi bigrafia

* fix: update Prisma schema and user service to use new role and status structure (#10)

* fix: update Prisma schema and user service to use new role and status structure

* fix: update AuthController to use gatewayUrl instead of frontendUrl for redirects

* fix: update JWT_SECRET in .env.template for consistency

* Remove lock emoji from project title in README (#12)

---------

Co-authored-by: SebasDuqueC <sebastian.duque-c@mail.escuelaing.edu.co>
Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Nella1414 <marianella.polo-p@mail.escuelaing.edu.co>
Co-authored-by: IgnacioCastillo05 <ignacio14.castillor@gmail.com>
Co-authored-by: ignacio14.castillor@gmail.com <ignacio.castillo-r@mail.escuelaing.edu.co>
Co-authored-by: Nella <107373290+Nella1414@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants