No-Country-simulation / h1-06-java-react

Justina.io
https://h1-06-java-react.vercel.app
2 stars 0 forks source link

Back - JUST-001 - Register doctor #13

Open DMRamirezZarta opened 4 months ago

DMRamirezZarta commented 4 months ago

url : http://localhost:7082/api/v1/doctor/register action : post jwtt : no

body example: { "name" : "Paloma", "surname" : "Zamora", "dni" : 35350341, "email" : "palomazamora@gmail.com", "password" : 12345678, "dateOfBirth" : "1990-04-05", "gender": "FEMENINO", "specialty" : "CARDIOLOGIA", "medicalLicense" : "128875" }

Consideraciones:

DMRamirezZarta commented 4 months ago

Feedback` sobre el Testeo del Caso de Prueba CP1: Registro Exitoso de Médico

Descripción del Caso de Prueba: Verificar que un médico pueda registrarse correctamente en Justina.io.

Pasos Realizados:

Resultado Obtenido:

Paso 1: Llenar el formulario de registro con datos válidos. Datos utilizados: json Copiar código { "name" : "Raul", "surname" : "Reina", "dni" : 82134567, "email" : "raulli@gmail.com", "password" : 12345889, "dateOfBirth" : "1999-05-23", "gender": "OTRO", "specialty" : "ENDOCRINOLOGIA", "medicalLicense" : "LM40913238" } Verificación del Género: Se permiten los valores "Masculino", "Femenino" y "Otro" correctamente. Formato de Fecha de Nacimiento: Al ingresar un formato incorrecto, el sistema arroja un error, lo cual es correcto y esperado. Contraseña: La contraseña debe tener un mínimo de 8 dígitos. Intentos con menos de 8 dígitos son rechazados correctamente. Formato de Correo Electrónico: Al ingresar un formato incorrecto de correo electrónico, el sistema no permite el registro, lo cual es correcto. Número de DNI: Intentos de registro con un número de DNI ya existente son rechazados correctamente. Número de Licencia Médica: Se permite el registro con un número de licencia médica ya existente, lo cual es incorrecto. Este número debería ser único por profesional de salud. Nombres y Apellidos: El sistema permite el registro utilizando números o una combinación de números y letras en los campos de nombres y apellidos, lo cual es incorrecto. No se debería permitir el uso de caracteres no alfabéticos en estos campos. Conclusión: El sistema cumple en gran medida con las expectativas en cuanto a validaciones de datos. Sin embargo, se encontraron los siguientes puntos a mejorar:

Acciones Sugeridas:

Implementar la validación de unicidad para el número de licencia médica. Ajustar las validaciones para nombres y apellidos, permitiendo solo caracteres alfabéticos. Realizar pruebas adicionales para asegurar que estas validaciones funcionan correctamente tras su implementación.

Herramientas Utilizadas:

Postman: Para ejecutar las pruebas de registro. IntelliJ IDEA, MySQL Workbench: Para levantar el servidor y ejecutar el proyecto. excel: para dejar registro

Comentarios Adicionales: Agradecemos el esfuerzo del equipo y esperamos que las sugerencias ayuden a mejorar aún más la calidad del sistema.

Observaciones La validación de género, fecha de nacimiento, contraseña y correo electrónico funciona correctamente, y el sistema rechaza los registros con un DNI ya existente. Sin embargo, los campos de nombres y apellidos deben ajustarse para permitir solo caracteres alfabéticos, y se debe implementar la validación de unicidad para el número de licencia médica. La confirmación por email de registro exitoso será probada en las pruebas adicionales de la etapa final.

[Anexos]

CP1.xlsx

Image Image Image Image Image Image Image Image Image Image Image

@GuillermoDivan

GuillermoDivan commented 4 months ago

Número de licencias médicas: Dado que en Argentina puede haber iguales números de licencia médica porque cada provincia lleva su registro individual, se ha contemplado lo siguiente.

Observaciones. Para un registro 100% real, esto no sea suficiente por lo siguiente:

Sólo letras mayus y minus en nombres y apellidos Hecho! Adicionalmente, sólo números y letras mayúsculas en DNI. No debe permitir ni puntos. En todo caso incorrecto debería devolver 400 bad request con un mensaje acorde.

ADICIONALES

GuillermoDivan commented 4 months ago

ENUM DE UBICACIÓN MATRÍCULA. POSIBLES OPCIONES

MATRICULA_NACIONAL, MATRICULA_DE_BUENOS_AIRES, MATRICULA_DE_CATAMARCA, MATRICULA_DE_CHACO, MATRICULA_DE_CHUBUT, MATRICULA_DE_CORDOBA, MATRICULA_DE_CORRIENTES, MATRICULA_DE_ENTRE_RIOS, MATRICULA_DE_FORMOSA, MATRICULA_DE_JUJUY, MATRICULA_DE_LA_PAMPA, MATRICULA_DE_LA_RIOJA, MATRICULA_DE_MENDOZA, MATRICULA_DE_MISIONES, MATRICULA_DE_NEUQUEN, MATRICULA_DE_RIO_NEGRO, MATRICULA_DE_SALTA, MATRICULA_DE_SAN_JUAN, MATRICULA_DE_SAN_LUIS, MATRICULA_DE_SANTA_CRUZ, MATRICULA_DE_SANTA_FE, MATRICULA_DE_SANTIAGO_DEL_ESTERO, MATRICULA_DE_TIERRA_DEL_FUEGO, MATRICULA_DE_TUCUMAN, MATRICULA_DE_CIUDAD_AUTONOMA_DE_BUENOS_AIRES

DMRamirezZarta commented 4 months ago

Feedback Re-test de Registro de Médico

En esta nueva ronda de pruebas, me enfoque en verificar la funcionalidad del registro de médicos, y me complace informar que hemos logrado avances significativos:

Validación de Campos de Nombre y Apellido Anterior: En la prueba anterior, se permitía el registro utilizando números en los campos de nombre y apellido, lo cual no era correcto. Actual: En este nuevo test, hemos comprobado que este problema ha sido solucionado. Ahora, el sistema valida correctamente estos campos para asegurar que solo se ingresen caracteres alfabéticos.

Verificación de la Licencia Médica Anterior: El sistema permitía el registro usando un número de licencia médica que ya estaba registrado, lo cual podía generar duplicidades y confusión. Actual: Este inconveniente también ha sido solucionado. En la nueva prueba, comprobamos que el sistema ya no permite el registro con un número de licencia médica previamente registrado, mejorando así la integridad de la base de datos.

Comprobación por Email Actual: La verificación por email se dejará para la prueba final. Hasta el momento, todos los demás aspectos relacionados con este punto están funcionando correctamente.

Conclusión Gracias al esfuerzo y dedicación del equipo, hemos logrado solucionar problemas críticos en el registro de médicos. Continuaremos trabajando en la verificación por email y otros aspectos menores para asegurar que el sistema funcione perfectamente para el lanzamiento final. RD1 RD2 RD3

CP1-V2.xlsx

@GuillermoDivan