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

Configurar ambiente para testes automatizados #38

Open
rougeth opened this issue Sep 16, 2019 · 6 comments
Open

Configurar ambiente para testes automatizados #38

rougeth opened this issue Sep 16, 2019 · 6 comments
Labels

Comments

@rougeth
Copy link
Collaborator

rougeth commented Sep 16, 2019

Com testes unitários, criar novas features e refatorar as já existentes ficará muito mais seguro, com menos chances de quebrar o que já funciona. Escopo dessa issue é configurar o ambiente (instalar dependências, criar arquivos de configuração, etc) para que seja possível testar o Rastreiobot.

Seria ideal que o trabalho feito em relação a issue #37 já tivessem testes.

@rougeth rougeth changed the title Criar testes automatizados Configurar ambiente para testes automatizados Sep 26, 2019
@itepifanio
Copy link
Contributor

Posso trabalhar nessa issue. Eu penso em adicionar uma pasta tests na raiz do diretório e criar os unitários ali dentro utiizando a biblioteca unittest do python. Sei que seria interessante ter a issue #37 já pronta, mas acho que poucas adaptaçõe serão necessarias para incorporar uma a outra.

@rougeth
Copy link
Collaborator Author

rougeth commented Oct 1, 2019

@itepifanio seria ótimo ter essa issue feita antes da #37, porque daí a refatoração pode vir acompanhada de testes. O que eu tinha pensando em fazer pra essa issue era:

  • Adicionar pytest como dependência;
  • Criar o diretório tests/;
  • Criar um arquivo teste "dummy", algo como test_helloworld.py com apenas um teste dentro. A ideia é ter apenas o modelo para seguirmos em outros pull requests;
  • Configurar Travis ou CircleCI para rodar os testes em todos os PRs.

O que você acha?

@itepifanio
Copy link
Contributor

Tudo certo, mais tarde eu mando o PR. Se quiser posso adicionar a verificação do PEP8 no Travis também.

@rougeth
Copy link
Collaborator Author

rougeth commented Oct 2, 2019

@itepifanio acho que ainda não é o momento, vai quebrar muita coisa se fizermos isso agora. O ideal seria usar black, mas acho que eles não suportam checar apenas o diff do PR.

@rennerocha
Copy link

É um pouco traumático, mas acho que poderíamos aplicar o black na base de código atual o quanto antes (dado que não existe nenhum PR em aberto que tenha grandes modificações de código) e assim já incluir essa validação no travis.

@rennerocha
Copy link

Caso achem que já vale a pena aplicar: #49

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants