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

Some tests working #5

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

Sheuka
Copy link

@Sheuka Sheuka commented Oct 1, 2024

Imposible simular el drag and drop debido a la librería usada para hacerlo :(

@SoyJorgePilo
Copy link

Hola @Sheuka, ¿podrías contarme un poco más sobre tu problema para poder ayudarte mejor? :D

@felipecao
Copy link

@Sheuka a parte la pregunta de Jorge, me gustaría agradecerte por el esfuerzo, hacer con que el drag n drop funcione es realmente complicado.

De todas formas, me gustaría compartir un par de ideas sobre tu trabajo:

  • interesante lo que hiciste entre Cypress y BE, sacando un endpoint para resetear los tests
  • los tests son bastante asertivos, es decir, efectivamente se aseguran de cubrir lo que piden los requisitos
  • quizás algo que se pudiera haber hecha de manera distinta es usar mas el concepto de "fixtures" de Cypress para reducir un poco el nivel de hardcodeo. Igualmente, para variables, se podría haber usado el fichero cypress.config.js de manera más profundo con el mismo proposito

@Sheuka
Copy link
Author

Sheuka commented Oct 3, 2024

Hola @felipecao y @SoyJorgePilo. Entiendo lo que me dices @felipecao pero, por un lado, mi tiempo para estudiar la teoría y realizar las prácticas es muy limitado. Y por otro, no tengo demasiada experiencia en las tecnologías que estamos utilizando para el ejercicio. Por ello, no puedo entrar tanto en el detalle de Cypress como para entender que es mejor usar los fixtures o que en el config se pueden introducir variables. Supongo que con algo más de tiempo hubiera podido realizar algo más notable.
Aún así os agredezco las respuestas e intentaré (cuando tenga algo más de tiempo) modificar mi ejercicio usando las buenas prácticas que me indicas. También he de decir que aunque no haya trabajado antes con Cypress he aprendido mucho sobre el.

Saludos

@felipecao
Copy link

@Sheuka te entiendo completamente! Muchas gracias por el feedback en cuanto a la cuestión del tiempo, estamos atentos y trabajando para ajustar el tiempo que se invierte en este ejercicio.

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