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

Problemas de TP6, Ej 3, Spootify #1

Open
emanuelbas opened this issue Jun 6, 2018 · 0 comments
Open

Problemas de TP6, Ej 3, Spootify #1

emanuelbas opened this issue Jun 6, 2018 · 0 comments

Comments

@emanuelbas
Copy link
Owner

  1. Sobre el diagrama UML:

i. faltan los tipos de los atributos de las clases Autor, Album y TemaSpootify
ii. falta la cardinalidad en las asociaciones.
iii. la navegabilidad de las asociaciones es correcta pero no así su notación. Se debe utilizar la punta de flecha no rellena.
iv. faltan los nombres de los parámetros en las responsabilidades (están sus tipos, pero no sus nombres).

  1. Sobre la implementación:

i. No se implementó la búsqueda case insensitive, tal como pedía el enunciado.

ii. Hay métodos, por ejemplo: #buscarPorTitulo: de la clase AutorSpootify, que deberían haber usado mayor nivel de abstracción como por ejemplo #flatCollect: en lugar de agregar a mano en una colección de colecciones.

  1. Sobre los tests:
    i. Los escenarios de prueba son pobres. Debería haber un test para cada escenario a probar.

ii. No hay ningún test que controle que las 3 búsquedas sean case insensitive.

iii. No hay ningún test que controle que la búsqueda funcione para cuando el resultado es vacío.

iv. No hay tests para probar el agregado temas repetidos a la música favorita de un usuario.

v. No hay tests para probar la eliminación de una canción inexistente en la música favorita de un usuario.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant