search

regiov / voluntarios

Website voluntarios.com.br
GNU General Public License v3.0
14 stars 12 forks source link

Adicionando testes do modelo Usuario do app vol #38

Closed GBTelles1 closed 1 year ago

GBTelles1 commented 1 year ago

Adicionei alguns testes unitários em cima do modelo Usuario e modifiquei a estrutura de pastas dos testes do app vol

regiov commented 1 year ago

Legal, @GBTelles1! Só uma dúvida: o pull request está com mais de 400 arquivos. Será que você não incluiu coisa a mais por engano? Dá pra ver que tem centenas de arquivos estáticos de imagens, configuração de ambiente virtual e inclusive executáveis do python...

GBTelles1 commented 1 year ago

Verdade! Acabei colocando arquivos da minha venv no python sem nem ver. Vou fechar esse PR e abrir outro. Já coloquei mais testes tb. Pode ser?

regiov commented 1 year ago

OK, @GBTelles1, também não sei a melhor forma de lidar com essas situações. Suponho que seja melhor mesmo cancelar e gerar outro PR. Se você conseguir cancelar por aí, manda ver. Qualquer coisa me avise. O novo PR, pelo que entendi, deve conter apenas inclusões bem pontuais no arquivo tests.py. Se bem que teve outra coisa que você alterou no models que parecia ser um bug. Se quiser pode incluir essa alteração tb, caso contrário faço ela por aqui depois. Obrigado!

GBTelles1 commented 1 year ago

Perfeito! Vou fazer isso. E bem lembrado, sobre o models. Mudei de "termo" pra "self", pq "termo" não estava definido em nenhum lugar e, até pelo padrão das outras funções e pela definição da classe "TermoAdesao", achei que fosse um erro mesmo. De onde vem esse "termo"?

regiov commented 1 year ago

Boa pergunta. Tá com cara de algum erro de copy & paste :-)

GBTelles1 commented 1 year ago

Sim, imaginei que fosse algum errinho de digitação. Vou deixar com o self, pelo menos por enquanto, então :)