Validaciones y pruebas unitarias microservicios de gestión de usuarios y roles - Parte 2

edwargl7 commented 5 days ago

Se requiere realizar la revisión de pruebas unitarias a los endpoints del microservicio y realizar ajustes a las pruebas unitarias agregadas en la Issue #65 teniendo presente los asserts.

package main

import (
    "testing"

    "github.com/stretchr/testify/assert"
)

funcTestAddition(t *testing.T) {
    x := 2
    y := 2

    assert.Equal(t, x, y, "x and y should be the same")
}

Material de apoyo

Sub Tareas

[x] Agregar pruebas unitarias al microservicio CRUD.
[x] Ajustar pruebas unitarias agregando asserts.
[x] Agregar el resultado de coverage de las pruebas unitarias.

Criterios de aceptación

[x] Pruebas unitarias complementadas y ajustadas del microservicio CRUD.
[x] Coverage de las pruebas unitarias.

Requerimientos

No aplica

Definition of Ready - DoR

[x] Está refinada y estimada en puntos de historia por el equipo.
[x] Incluye la descripción y criterios de aceptación, con el detalle funcional y especificaciones técnicas, de forma entendible por cualquier miembro del equipo.
[x] No tiene bloqueos que impidan su ejecución.
[x] Las dependencias están identificadas y resueltas.
[x] Puede ser probada dentro del Sprint.

Definition of Done - DoD - Desarrollo

[x] Desarrollo en local.
[x] Push en Feature.
[x] Pruebas locales (funcionales).
[ ] PR a Develop.
[x] Criterios de aceptación cumplidos.
[x] Documentación de issue realizada.
[x] Aprobada por SM/Líder técnico.

jucastro94 commented 4 days ago

se crean metodos segun casos de validacion para validacion tipo fecha

jucastro94 commented 3 days ago

se ajustan los mensajes de respuesta a español en todos los controladores

jucastro94 commented 3 days ago

se ajustan validaciones para trabajar con assert

jucastro94 commented 3 days ago

se crea capa helpers con archivo utilidades.go para metodo de control de excepciones

se aplica defer en cada uno de los controladores

jucastro94 commented 2 days ago

se optimiza metodo

jucastro94 commented 2 days ago

se crean validaciones para controlador usuario

jucastro94 commented 2 days ago

Se debe agregar validación para el numero de documento

jucastro94 commented 2 days ago

generacion de coverage

edwargl7 commented 2 days ago

Pendiente remover los archivos comentados en el PR 2 y solicitar de nuevo el Pull Request ya sin dichos archivos.

jucastro94 commented 2 days ago

se remueven los archivos mencionados y se solicita PR

edwargl7 commented 20 hours ago

Se observa un buen trabajo, cuenta con los ajustes solicitados en las pruebas unitarias y cuenta con una buena organización del código.

:wrench: se debe revisar el PR ya que no se observa en este el commit en el que se removieron los archivos, en la rama de desarrollo si cuenta con la eliminación de estos.

Se procede a finalizar la tarea y se revisara con el desarrollador el PR.

udistrital / sisifo_documentacion

Validaciones y pruebas unitarias microservicios de gestión de usuarios y roles - Parte 2 #75