uqbar-project / wollok

Wollok Programming Language
GNU General Public License v3.0
61 stars 16 forks source link

Dynamic diagram for single test #1897

Closed fdodino closed 4 years ago

fdodino commented 4 years ago

En realidad este es un PR bastante integral de todo el proceso de testing (todavía falta la parte de Mock):

demoTestDD

coveralls commented 4 years ago

Coverage Status

Coverage remained the same at 86.151% when pulling 472bf0222fe634d6477a4db0530772268be0a1fb on dynamic-diagram-for-single-test into f7f3f84acca98cbbdd941ec3eaad9fcbee44905b on dev.

fdodino commented 4 years ago

@PalumboN @isaiaslafon , acá están los íconos retocados: agregué un signo de admiración para el warning, y una cruz para el error. El test, describe y archivo de test ok queda sin nada.

image

image

isaiaslafon commented 4 years ago

genial! iba a hacer eso mas tarde hoy, te hubiese avisado así no gastabas tus habilidades en eso. disculpa.

El lun., 20 de jul. de 2020 09:55, Fernando Dodino notifications@github.com escribió:

@PalumboN https://github.com/PalumboN @isaiaslafon https://github.com/isaiaslafon , acá están los íconos retocados: agregué un signo de admiración para el warning, y una cruz para el error. El test, describe y archivo de test ok queda sin nada.

[image: image] https://user-images.githubusercontent.com/4549002/87938947-acf1df80-ca6d-11ea-8f86-a9a10ad30ea1.png

[image: image] https://user-images.githubusercontent.com/4549002/87939840-09093380-ca6f-11ea-9633-5514314f8489.png

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/uqbar-project/wollok/pull/1897#issuecomment-661021455, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALZLDNKC2NKAM7X7XXISP2LR4Q5CRANCNFSM4OCHYPIQ .

fdodino commented 4 years ago

Buenísimo todo. Te dejé un par de dudas más que nada para no estar tan perdido yo.

Aparecen cambios de wollok-language habría que tirar un PR allá también o cómo se maneja eso?

Sí, acabo de hacer un PR de wollok-language, y hay que hacer otro de cli.