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

Adição de alguns elementos 'data-testid' no front para automação E2E + Pytest Playwright #334

Draft
wants to merge 22 commits into
base: main
Choose a base branch
from

Conversation

williamvitu
Copy link

@williamvitu williamvitu commented Feb 1, 2025

Português (BR) | English (US)

Comunidade

Tipo de alteração

  • 🐞 Correção de problema
  • ✨ Melhoria ou nova funcionalidade
  • 📰 Nova postagem no blog

Issues relacionadas

Issues que são relacionadas a esta Pull Request.

Validação

  • Validei a alteração no link gerado pelo bot da Netlify (Deploy Preview/Preview on mobile)
  • Validei o Layout responsivo (desktop/mobile) após a implementação
  • Verifiquei o registro do deploy (Latest deploy log) e nenhum novo alerta ou erro foi adicionado

Evidências

Anexe evidências do antes e do depois da alteração (quando necessário).

Documentação

  • A documentação deste repositório foi atualizada (quando necessário).
  • Esta alteração requer que a documentação externa seja atualizada.

Copy link

netlify bot commented Feb 1, 2025

Deploy Preview for querido-diario-plataforma failed.

Name Link
🔨 Latest commit a6fc975
🔍 Latest deploy log https://app.netlify.com/sites/querido-diario-plataforma/deploys/67b7bbf41b94790008d83225

@lguima
Copy link
Collaborator

lguima commented Feb 1, 2025

fala @williamvitu, excelente iniciativa!

teve alguma razão pra optarmos pelo Playwright com Python em vez de Node.js?

@williamvitu
Copy link
Author

williamvitu commented Feb 1, 2025

fala @williamvitu, excelente iniciativa!

teve alguma razão pra optarmos pelo Playwright com Python em vez de Node.js?

@lguima blza?

Eu optei pelo python justamente por que é a linguagem em que tenho mais conhecimento, mas se acharem necessário, podemos mudar para nodejs sem problemas.

Já criei a grande maioria dos testes que irei automatizar no front, então agora só estou iniciando mesmo

@lguima
Copy link
Collaborator

lguima commented Feb 1, 2025

@williamvitu bom demais! saquei, será que como o frontend está no contexto de JavaScript a opção por Node.js seria mais próxima pra juntar a mesma comunidade pra suportar?

cc: @MoisesMsa @ogecece

@williamvitu
Copy link
Author

@williamvitu bom demais! saquei, será que como o frontend está no contexto de JavaScript a opção por Node.js seria mais próxima pra juntar a mesma comunidade pra suportar?

cc: @MoisesMsa @ogecece

Se não for um problema continuar no python, eu agradeço, ja adiantei algumas coisas. Depois posso fazer a migração pra js.

O código de automação é bem simples

@williamvitu williamvitu changed the title Adição de alguns elementos 'data-testid' no front para iniciarmos automação E2E Adição de alguns elementos 'data-testid' no front para automação E2E + Pytest Playwright Feb 5, 2025
@williamvitu williamvitu marked this pull request as draft February 5, 2025 22:43
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