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

Test tareaXX #674

Open
maogaz opened this issue Jul 5, 2019 · 1 comment
Open

Test tareaXX #674

maogaz opened this issue Jul 5, 2019 · 1 comment
Assignees
Labels
IMPORTANTE tareaxx Issues de la tarea: TareaXX

Comments

@maogaz
Copy link

maogaz commented Jul 5, 2019

Hola, una duda con el test final de la tareaXX: en el excel me sale que mis árboles son inválidos y quiero probar en que me equivoqué pero no logro correr el archivo de los tests.
Revisando el código en sí tengo el siguiente caso que pueda ser que lo detecte inválido:
Mi clase AlgarroboTree usa el método balance usando un nodo de donde empezar a hacer el balance y en mi clase insertar se utiliza el método para ordenar el árbol automáticamente cada vez que agrego un valor.
En los test la función balance no necesita ningún atributo, por lo que puede ser la fuente del error.
Test:
image
Código:
image

Es posible que se pueda hacer algo al respecto?, mis árboles se instancian y la funciones me tiran las respuestas correctas.

@Hernan4444
Copy link
Contributor

Hola

Se decidió volver a ejecutar todos los test, pero ahora con la línea tree.balance() comentada porque algunos estaban llamando a balance dentro del insert y ese supuesto era totalmente válido, por lo tanto llamar de nuevo a balance generaba error. Aparte que en el test se ingresaron números de menor a mayor para no necesitar balance, así que se comentó para probar dicha opción.

Se actualizó la planilla de notas con las nuevas décimas porque hubo varias tareas que funcionaron ahora.

Saludos ✌️

@Hernan4444 Hernan4444 self-assigned this Jul 5, 2019
@Hernan4444 Hernan4444 added IMPORTANTE tareaxx Issues de la tarea: TareaXX labels Jul 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IMPORTANTE tareaxx Issues de la tarea: TareaXX
Projects
None yet
Development

No branches or pull requests

2 participants