RastreioBot icon indicating copy to clipboard operation
RastreioBot copied to clipboard

Configurar ambiente para testes automatizados

Open rougeth opened this issue 4 years ago • 6 comments

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 https://github.com/GabrielRF/RastreioBot/issues/37 já tivessem testes.

rougeth avatar Sep 16 '19 17:09 rougeth

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.

itepifanio avatar Oct 01 '19 14:10 itepifanio

@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?

rougeth avatar Oct 01 '19 17:10 rougeth

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

itepifanio avatar Oct 01 '19 20:10 itepifanio

@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.

rougeth avatar Oct 02 '19 17:10 rougeth

É 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 avatar Oct 03 '19 18:10 rennerocha

Caso achem que já vale a pena aplicar: https://github.com/GabrielRF/RastreioBot/pull/49

rennerocha avatar Oct 03 '19 18:10 rennerocha