Fix #1801 test runner cli several files

[fdodino]

Arregla estos issues

• El test runner desde cli debería correr todos los tests y no parar en el primer error. #1801 El test runner ya no para al primer error sino que chequea todos los tests
• Saber cuánto tardó cada test en ejecutar #1798 Se mide la performance de cada test
• Permite definir describes y tests sin descripción #1813 Tests y describes sin descripción los deja vacíos
• Cambiar los [ ] al fallar un assert #1797 Cambiar los [] al fallar un assert
• Objetos sin atributos se imprimen con [] #1772 Objetos sin atributos se imprimen con []

Este PR permite que este PR de wollok-cli se termine de implementar y puedas correr todos los tests de un conjunto de archivos con un buen rendimiento (ej: hoy tenemos 525 sanity tests corriendo en aproximadamente 10 segundos, donde un solo test se está llevando 2 segundos, si no sería mucho más rápido aun).

También se emprolijó bastante la forma de notificar a los reporters de los tests que ejecutan, agrupándolos hasta 3 niveles, midiendo la performance general de: 1 test, 1 describe, toda la ejecución.

Ejemplos

Correr los sanity tests + un ejemplo de un alumno que sabemos que falla, con múltiples archivos:

Corriendo en la UI varios archivos, se muestran los tiempos de ejecución de cada test:

Output de la consola REPL para los objetos que tienen y no tienen atributos:

Como tarea pendiente, habría que pasar estas mejoras al reporter que usa Wollok Server (WollokJSONTEstReporter).

[coveralls]

Coverage remained the same at 89.789% when pulling 788cf88 on fix-#1801-test-runner-cli-several-files into 0bcee4b on dev.

[npasserini]

Está muy bien, dejé un par de comentarios estéticos y otros detalles, lo único que me preocupa es si conviene cambiar el toString, me gustaría que no rompamos los sanities.

@faloi Te parece que este cambio en toStrings podría impactar en Mumuki?

Faltaría:

• actualizar el backlog
• si modificamos el toString corregir sanities (o al menos crear un ticket para eso).

[fdodino]

Listo con los cambios del PR @npasserini . Una vez mergeado a dev este PR y el que sigue ya estamos para liberar la versión 1.9.0

[npasserini]

Bah, a mí me pareció raro el elvis en esa situación... pero igual era un comentario mínimo, no les des bola. El sáb., 5 de oct. de 2019 a la(s) 19:37, Fernando Dodino ( notifications@github.com) escribió:

…

***@***.**** commented on this pull request. ------------------------------ In org.uqbar.project.wollok/src/org/uqbar/project/wollok/validation/WollokDslValidator.xtend <#1821 (comment)>: > @@ -1135,6 +1135,14 @@ class WollokDslValidator extends AbstractConfigurableDslValidator { report(WollokDslValidator_DONT_DUPLICATE_TEST_DESCRIPTION, wtest, WTEST__NAME) } + @check + @DefaultSeverity(WARN) + @CheckGroup(WollokCheckGroup.POTENTIAL_DESIGN_PROBLEM) + def testWithEmptyDescription(WTest it) { + if ((name ?: "").equals("")) Meh, no termino de comprar que nullOr me convenza mucho más que el Elvis operator, lo cambio pero no estoy familiarizado y me cuesta más interpretarlo. — You are receiving this because your review was requested. Reply to this email directly, view it on GitHub <#1821?email_source=notifications&email_token=ABDLKOIVB4YL7GIOLPBG3FTQNEJLBA5CNFSM4I3UP7P2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCHAD2WY#discussion_r331764043>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABDLKONRC2Q7GQVPZPYISE3QNEJLBANCNFSM4I3UP7PQ> .