search

uqbar-project / wollok

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

Proyecto roto causa ERROR:null al correr los tests #1841

Closed fdodino closed 5 years ago

fdodino commented 5 years ago

Hoy corrigiendo me saltó un proyecto que hacía romper todo el entorno Wollok: los tests, apareció el Dirty State del Editor, era ingobernable. Refactorizándolo quedó pipí cucú, pero no pude ver qué tenía. Subo para que lo analicemos.

git@github.com:wollok/issue-1841.git https://github.com/wollok/issue-1841.git

fdodino commented 5 years ago

El problema está en el fixture, que está reventando, y eso produce que los tests no se inicialicen correctamente. En el fixture estas líneas:

pepe.aprobo(tc1)
matias.aprobo(tc1)
matias.aprobo(tc2)

tiran un error de Wollok, por validaciones del negocio erróneamente definidas!

npasserini commented 5 years ago

Che, pero si el fixture revienta el IDE debería informarlo razonablemente, no?

El lun., 21 de oct. de 2019 a la(s) 07:55, Fernando Dodino ( notifications@github.com) escribió:

Closed #1841 https://github.com/uqbar-project/wollok/issues/1841 via 882222f https://github.com/uqbar-project/wollok/commit/882222fbba8cfae2a747330f33147cf2d3abf09c .

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/uqbar-project/wollok/issues/1841?email_source=notifications&email_token=ABDLKOLHZOCKQBQQARRUEP3QPWDBRA5CNFSM4I7HGNUKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUKWHJ4Y#event-2729211123, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABDLKOLZT3MUYNTYFQL4W2TQPWDBRANCNFSM4I7HGNUA .

fdodino commented 5 years ago

Sí, eso es lo que arreglé

npasserini commented 5 years ago

👍😎

El lun., 21 de oct. de 2019 a la(s) 10:46, Fernando Dodino ( notifications@github.com) escribió:

Sí, eso es lo que arreglé

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/uqbar-project/wollok/issues/1841?email_source=notifications&email_token=ABDLKOKQA7HDCGMI5Q2LD6LQPWXCZA5CNFSM4I7HGNUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEB2L7XQ#issuecomment-544522206, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABDLKON7XSGQIKXYB3RYTJTQPWXCZANCNFSM4I7HGNUA .