Comments (1)
Está siendo utilizado en la clase EquipamientoIncrementadoTest.
Hay un primer test que mockea al gladiador, hace que el test parezca poco útil pero es verdaderamente unitario. Los otros test utilizan al Gladiador como dependencia, perdiendo su carácter de unitario unitario. No los veo mal, no se si los dejaria ahí o las pasaría directamente al paquete otros_casos_de_uso. Lo dejo como discusión. (Entiendo que esto sucede en algunas pruebas mas, el enunciado del tp dice Entrega3: Finalización del modelo junto a todas las pruebas unitarias y de integración por lo cual son útiles la discusión pasa por el lugar donde deben ir)
Más allá de la discusión anterior todos estos test que utilizan el gladiador.GetEquipamiento()
tienen su parecido con los casos de uso 5 y 6 de la entrega 1, en donde encontramos otra estrategia para testear y evitar ese getter, con lo cual pueden ser refactorizado de ese modo y así eliminar el metodo getEquipamiento()
del gladiador
from fiuba-algo3-tp2.
Related Issues (20)
- Pasar todos los paquetes a modelo HOT 1
- Hay algunas excepciones sueltas fuera de los paquetes que la utilizan o cualquiera sea la forma en que vayamos a organizarlas
- Refactorizar seniorities test
- FieraSalvajeTest pruebas sin mock podria pasar a otro paquete
- SinConcecuencia es una clase anemica HOT 1
- Refactor AssertEquals de energia
- CasillerosFactory no implementa el patron Factory HOT 1
- El equals de la clase equipamiento esta raro.
- Hay numeros que podrian cambiar por constantes, varios en FieraSalveje, uno en comida HOT 1
- El atributo booleano de gladiador EstaHabilitadoParaMover podria ser una entidad, de esa forma pueden sacar el if del metodo avanzar.
- Si un jugador tiene una energía inferior a 0, ese turno se pasa y se le acreditan 5 puntos extra HOT 1
- Asiste a un Bacanal no debe ser instanciado con la cantidad de tragos
- Los test de la entrega 2 probablemente precisan un refactor HOT 1
- Refactor casos de uso entega 2 HOT 1
- Eliminar el metodo inicializarJuego, dejar la logica dentro del jugarTurno
- callJustOnce en vez de Mockito.times(1)
- Código repetido en clase Camino.java
- Unificar constantes
- Entrega Final
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from fiuba-algo3-tp2.