Skip to content

jcdd1/devOpsAppQuiz

 
 

Folders and files

NameName
Last commit message
Last commit date

Latest commit

 

History

17 Commits
 
 
 
 
 
 
 
 
 
 
 
 
 
 

Repository files navigation

devOpsAppQuiz

[5%] Realizar un fork del proyecto, y agregar miembros del team aca en el encabezado

[15%] Hacer un code Review encontrado las razones por las cuales el codigo en el repo es un APIRest o no y justificar, y escribir el resultado del code review dentro de este readme, utilizar las syntaxis propuesta en:

  • https://docs.github.com/en/get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax
  • https://github.com/othneildrew/Best-README-Template
  • Hallazgos a partir del code review:
    • RestBarriosController no retorna un objeto HTTP, los métodos HTTP de una solicitud HTTP de la API de RESTful son importantes ya que contienen información de identificación importante con respecto a los metadatos, la autorización, el identificador uniforme de recursos (URI), el almacenamiento en caché, las cookies y otros elementos de la solicitud.
    • Falta añadir comportamiento de @RestController, La clase RestBarriosController no posee la datanotation necesaria para ser usada como controller.
    • Falta la lógica del negocio para cada uno de los métodos createPerson y updatePerson en el controlador RestBarriosController. Adicionalmente ninguno de los anteriores métodos retornaba los datos necesarios
    • Falta el manejo de persistencia de los objetos barrio, tanto como un Service asociado.
    • Utilizar bien la responsabilidad de la clase, es decir, los nombres de los métodos no corresponden a lo que realmente hacen en la clase RestBarriosController.
    • Las clases EmpleoDto y PaisDto no tienen una definición de métodos o propiedades; aun que esto en sí no es un problema para que sea un API REST.
    • Hacen falta las pruebas unitarios para los servicios y otros metodos importantes; aun que esto en sí no es un problema para que sea un API REST.
    • La interfaz ManejoUsuarioServiceInt no contiene los metodos saveUsuarios, editUsuario, deleteUsuario, estos metodos son parte del modelo CRUD y deberian estar para poder ser usados en el controlador.

[50%] Completar la implementacion usando: hacer commit en el master branch

[30%] Implementar GitFlow Startegy en el proyecto

About

devOpsAppQuiz_JCDD_GJGM

Resources

Stars

Watchers

Forks

Packages

No packages published

Languages

  • Java 100.0%