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

[WIP] Adiciona Github Actions para rodar os testes #75

Closed
wants to merge 6 commits into from

Conversation

filipedeschamps
Copy link
Owner

@filipedeschamps filipedeschamps commented Jul 20, 2021

Closes: #74 (PR anterior onde explico o ambiente de testes local)
Closes: #62 (Issue relacionada da Milestone)

Foi uma aventura perceber como o ambiente em que se roda o Github Actions é diferente do meu ambiente local aqui :)

  1. O módulo wait-on funciona localmente que é uma maravilha, mas quebra no ambiente do Github Actions. Eu tava ficando quase maluco tentando isolar o problema de todas as formas e finalmente esbarrei nessa issue. Removi ela das dependências.
  2. Até nosso amigo axios se demonstra bugado na última versão 0.21.1 conforme essa issue e tive que voltar para versão 0.19.0. Como essa versão tem falha de segurança, movi ela para as dependências de desenvolvimento até a próxima versão sair.
  3. Pra testar as Actions sem precisar ficar fazendo push toda hora aqui para o repositório, eu descobri uma ferramenta sensacional chamada act. Ela simula o container que o Github roda nas Actions. É muito legal e foi por ali que peguei o bug do wait-on, porque essa lib falava que tal url ainda não estava disponível, mas por dentro do container fazendo curl ela estava normalmente.
  4. Tem uma última característica que só acontece dentro do ambiente do Github Actions que é a forma de fechar processos, tanto que apesar de fechar eles no código, eu precisei rodar o jest com a flag --forceExit. Mais para frente vou investigar.

And transform it from a Model to a infrastructure
component. Also make every method responsible of
opening and closing the database connection, since
it's safer this way.
* Works with ES6 modules
* Works with Next.js Absolute Paths
@vercel
Copy link

vercel bot commented Jul 20, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/tabnews/tabnews/8cvAsof7EPNqK7pM7dWLriETQHqV
✅ Preview: https://tabnews-git-actions-tabnews.vercel.app

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.

Rodar testes local e CI
1 participant