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

Permitir filtrar testes em qualquer modo de execução #1644

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

aprendendofelipe
Copy link
Collaborator

Atualmente nossos scripts não permitem filtrar os testes que queremos executar a não ser que estejamos executando os testes em modo watch.

Mudanças realizadas

  • Adiciona a possibilidade de filtrar os testes passando argumentos para o Jest usando passthrough-arguments do concurrently (-P e {@}).
  • Sempre executar services:stop ao encerrar o processo de testes: Passando -s first para o concurrently, de forma que ele seja encerrado com código 0 dependendo apenas do encerramento do processo do Next.js e não dos testes.
  • Organiza o script concurrently, separando o preconcurrently e o postconcurrently, além de trazer o comando do Jest para o script principal.
  • Adequa o README às novas possibilidades de filtrar não apenas em modo watch.

Tipo de mudança

  • Nova funcionalidade
  • Atualização de documentação

Checklist:

  • As modificações não geram novos logs de erro ou aviso (warning).
  • Os antigos testes estão passando localmente.

Copy link

vercel bot commented Mar 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
tabnews ✅ Ready (Inspect) Visit Preview Mar 7, 2024 11:38pm

Copy link
Collaborator

@Rafatcb Rafatcb left a comment

Choose a reason for hiding this comment

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

Parece bom para mim 👍

Testar as alterações me fez lembrar que o npm run test nunca funcionou para mim quando outro terminal estava com o npm run dev. Ele mata o serviço, mas não prossegue para os testes. É só aqui que acontece isso?

@aprendendofelipe
Copy link
Collaborator Author

Testar as alterações me fez lembrar que o npm run test nunca funcionou para mim quando outro terminal estava com o npm run dev. Ele mata o serviço, mas não prossegue para os testes. É só aqui que acontece isso?

O kill-port mata o serviço do Next.js, não o script dev, então services:up e services:stop rodam simultaneamente, cada um por um dos scripts, então a única vantagem que vejo para usar o kill-port é impedir a criação de instâncias extras do Next.js em diferentes portas quando a 3000 já estiver em uso.

Mas pra poder rodar um script sem precisar matar o outro primeiro, o kill-port realmente não ajuda.

Uma solução simples seria parar de chamar automaticamente o services:stop. A única desvantagem que vejo nisso seria um esquecimento do contêiner rodando em segundo plano, mas como são leves, isso não deveria ser uma grande preocupação.

@aprendendofelipe aprendendofelipe merged commit 29fb0b0 into main Mar 8, 2024
7 checks passed
@aprendendofelipe aprendendofelipe deleted the filter-tests branch March 8, 2024 00:49
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.

2 participants