Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Código MyBagManager.getUnitsIterator() + test MyBagManagerTest.testGetUnitsIterator() #95

Open
wants to merge 8 commits into
base: BagManager
Choose a base branch
from

Conversation

elemoncitos
Copy link

CORRECCIÓN: nombre del test y comprobaciones.
Se añaden los códigos de getUnitsIterator() en la clase MyBagManager para ordenar la cesta según el número de unidades de productos y el test que comprueba la funcionalidad del mismo.
Corrección: se modifica MyBagManager.getBag() cambiando el tipo retorno a una Collection debido a que no se puede hacer la conversión de lista a Collection

Elena Mosquera and others added 7 commits December 27, 2020 20:50
Se implemeta un metodo que devuelve un iterador con la cesta ordenaa por
productos
Añadimos los tests por pares de mi compañera y los tests que tengo que
probar yo
Corrección de MyBagManager.java
 - Error en el cast de devolución del metodo getBag(). cesta.values()
devuelve una Collection y el cast a List<Product> no es posible, por lo
que se redefine y se deja como tipo collection.
Correccion de UnitsComparatorTest
@elemoncitos elemoncitos added mejora modificaciones a un código anterior para enriquecerlo test Implementación de test requerida new code Nueva implementación labels Jan 23, 2021
Copy link

@eyksamech1 eyksamech1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved @elemoncitos

@MIT-FS MIT-FS removed their assignment Mar 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mejora modificaciones a un código anterior para enriquecerlo new code Nueva implementación test Implementación de test requerida
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants