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

Agregar programa para testear que los initialize en describe y clases ya no triggerean error #164

Conversation

juanbono
Copy link
Member

Pendientes:

  • Hay alguna variante de @Expect que me permita testear que justamente no se triggerea la validation?
  • Que deberia hacer con los objects? Tiene sentido un initialize en esos casos?

Copy link
Contributor

@PalumboN PalumboN left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Piolixxx!!! 🔥 🚀

Y yo agregaría estos tests en el archivo shouldUseOverrideKeyword que ya existía.
Así es fácil linkear validación con todos sus tests.

Contesto las preguntas:

  • Sí, podés dejar un @NotExpect(code="shouldUseOverrideKeyword") y será ignorado por el test del validador como comenté en el otro PR.
  • Sí, creo que tiene sentido agregar a los objects (y mixines) en el test.

@juanbono
Copy link
Member Author

Buenisimo, muchas gracias @PalumboN . Mañana hago los cambios 👍 !

@fdodino
Copy link
Contributor

fdodino commented Oct 29, 2023

Buenisimo, muchas gracias @PalumboN . Mañana hago los cambios 👍 !

Copado @juanbono !!
Cuando puedas meté los cambios y lo mergeamos en el próximo release!! Abrazo!

@fdodino
Copy link
Contributor

fdodino commented Nov 30, 2023

Buenisimo, muchas gracias @PalumboN . Mañana hago los cambios 👍 !

Copado @juanbono !! Cuando puedas meté los cambios y lo mergeamos en el próximo release!! Abrazo!

@juanbono , ahora que lo veo el archivo tiene que ser un wtest, eso está bien, y tendría que llamarse shouldUseOverrideKeyword.wtest (igual que el validador). El resto sería agregarle la anotación. Fijate si lo podés hacer en estos días, si no lo hacemos nosotros pero estaría copado que te sumes. Abrazo!

@PalumboN PalumboN merged commit 472e9cd into uqbar-project:master Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants