-
Notifications
You must be signed in to change notification settings - Fork 7
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 options exact match #141
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #141 +/- ##
==========================================
+ Coverage 84.05% 84.35% +0.30%
==========================================
Files 9 9
Lines 671 684 +13
Branches 154 162 +8
==========================================
+ Hits 564 577 +13
Misses 107 107 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Muy piola el laburo!
Yo te dejé una sugerencia para tratar de no repetir el mismo filtrado de tests en la ejecución.
Hay que probarlo bien en Windows porque recuerdo en su momento que se rompía por el tema caracteres en el endemoniado Powershell / CMD.
@ivojawer , ahí lo probé en Windows, funciona de 10: con CMDcon Powershellcon Git Bash |
También lo probé con el cambio que te sugerí y funciona ok. Solo encontré que hay un test que se rompe. |
Co-authored-by: Fernando Dodino <[email protected]>
Tremendo laburo dodain! Graciasss 😇 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pioliiixxxxx 🚀
Dejé un comentario sobre cómo se hace la búsqueda.
Relacionado a uqbar-project/wollok-lsp-ide#156 cambia como se usan las opciones file/describe/test para que en vez de convertirlo en un fqn recorra el AST para encontrar un test exacto
Usado por uqbar-project/wollok-lsp-ide#158