Skip to content

Refactor: Limpieza de Carrera, Mappers, Seguridad y 100% de cobertura en Unit Tests

Agustín Cantero requested to merge feature/carrera-refactor into master

Qué se hizo

  • Seguridad Dinámica: Se movieron las anotaciones @PreAuthorize("hasAuthority(...)") desde el CarreraController hacia el CarreraService, cumpliendo la regla de autorizaciones en la capa de negocio.
  • Endpoints Completados y Limpieza: Se agregó el endpoint faltante GET /api/carreras para listar todas las carreras (que ahora trae sus ITRs activos anidados). Además, el CarreraController quedó puramente delegativo y se ajustaron rutas (como PATCH /{id}/reactivar sin body).
  • MapStruct Personalizado: Se implementó CarreraMapper.java con mapeo manual estricto para filtrar únicamente las asociaciones activas de ITR.
  • Validaciones DTO: Se agregaron mensajes de error descriptivos en CarreraRequestDTO garantizando validaciones idénticas a los constraints de la base de datos (Ej: VARCHAR 80 y NOT NULL).
  • Cobertura de Tests (100%): Se creó CarreraServiceTest usando JUnit 5 y Mockito para cubrir todos los escenarios felices y de error en Carrera.
  • Estandarización de Pruebas: Se implementó @DisplayName en todas las clases de prueba (Auditoria, Rol, Permisos, Categorias) mejorando drásticamente la legibilidad de la consola.

Merge request reports