Entrega 1 Updated README.md #23
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Agregamos diagramas de clases completo y secuencia para el caso donde se pide una mano de poker, que se evalúe y se le asigne puntaje al jugador (no hecho lo de asignar) Co-Authored-By: Lucas Nicolas Pagani <[email protected]> Co-Authored-By: STEVEN GUERRERO <[email protected]> Co-Authored-By: Ignacio Zanoni Mutti <[email protected]>
Arreglamos diagramas de secuencia y clases y los ordenamos. Además separamos clases en otros diagramas. Co-Authored-By: Lucas Nicolas Pagani <[email protected]>
Creamos los primeros 3 test obligatorios de la entrega1 y los "hicimos pasar" violando todo tipo de pilar del POO, cumpliendo TDD Co-Authored-By: facukresta <[email protected]> Co-Authored-By: Ignacio Zanoni Mutti <[email protected]> Co-Authored-By: STEVEN GUERRERO <[email protected]>
Agregamos Puntaje; Palo y sus derivadas; y agregamos pruebas de TDD para Carta, Puntaje, Palo, y un poco de Jugador. Co-Authored-By: Lucas Nicolas Pagani <[email protected]>
TDD en Jugador, Carta, Puntaje, Mano y Mazo. Sigue implementar Juego y sus puntajes. Pasa hasta el caso de uso 4 Co-Authored-By: Lucas Nicolas Pagani <[email protected]>
Agregamos cómo reconoce un Jugador su Juego con la clase juego y cómo puntúa su respectivo juego. Faltan implementaciones para cada juego (solo hay Color) para la prueba 5 y el tarot, para las pruebas 6 y 7. Co-Authored-By: Lucas Nicolas Pagani <[email protected]>
TDD de todas las clases Juego, actualmente anda hasta el test 5, falta implementar Tarot para los test 6 y 7. Co-Authored-By: Lucas Nicolas Pagani <[email protected]>
Hice un refactor de Mano, que especialmente la mano ahora recibe la cantidad de cartas, teniendo manos de distintas manos; y agregué la clase ManoDe5 que reemplaza el ArrayList de Jugador. Faltaría ver bien las excepciones, creo que no están bien quien las lanza y sus nombres.
Agregamos la Interfaz Tarto junto con 3 clases: SinTarot (para cuando una carta o juego no fue modificada por un Tarot), CambiadorPuntos y CambiadorMultiplicador (ambos bastante descriptivos). Además están las pruebas para estas clases. Para esto también se modificaron Juego y Carta para poder albergar estos cambios. Ya pasan todos los tests de CasosDeUsoEntrega1Test. Co-Authored-By: Ignacio Zanoni Mutti <[email protected]>
Agregado mi nombre y apellido y mi dirección a mi perfil de Github. Integrante 4
Actualizo diagramas de entrega-1 y pequeño cambio en comportamiento de ManoDe5 (visto a ser modificado en el futuro). Las excepciones finalmente están lanzadas por los objetos correspondientes.
Arreglos de pruebas y diagrama. Faltaría hacer las pruebas 100% unitarias con "Mockito" y el reporte. Co-Authored-By: Lucas Nicolas Pagani <[email protected]>
Las únicas que no sacamos son las dependencias internas ya que de hacerlo perdería valor el test, porque no comprobaría bien la interacción entre objetos internos del test. Observación: para correr "Mockito", cambiamos el JDK de la versión 17 a la 11. Co-Authored-By: Lucas Nicolas Pagani <[email protected]>
Modificamos el caso de la entrega0 con todos los cambios realizados hasta el momento Co-Authored-By: facukresta <[email protected]>
Agregamos el diagrama de paquete y le sacamos la skinn rosa al de secuencia Co-Authored-By: facukresta <[email protected]>
Finalizamos la entrega1 terminando su reporte Co-Authored-By: STEVEN GUERRERO <[email protected]> Co-Authored-By: facukresta <[email protected]> Co-Authored-By: Ignacio Zanoni Mutti <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Agregado cuarto integrante y el enlace a su perfil de Github