radar-parlamentar / radar

Estamos de mudança para o GitLab: https://gitlab.com/radar-parlamentar/radar.
http://radarparlamentar.polignu.org
84 stars 44 forks source link

#390: Refatorando importadores CONV #474

Open hpe95 opened 6 years ago

hpe95 commented 6 years ago

Resolve #390

coveralls commented 6 years ago

Pull Request Test Coverage Report for Build 983


Files with Coverage Reduction New Missed Lines %
modelagem/models.py 2 96.03%
importadores/conv.py 4 95.76%
analises/tests/tests_analise.py 38 75.32%
modelagem/tests/tests_utils.py 39 80.6%
analises/analise.py 62 69.94%
<!-- Total: 145 -->
Totals Coverage Status
Change from base Build 978: -3.3%
Covered Lines: 3696
Relevant Lines: 4696

💛 - Coveralls
diraol commented 6 years ago

Aliás, pensando melhor aqui..... esses dados do "_votacoes_param" poderiam ficar num arquivo externo (csv ou json) que é lido nesse código que vocês fizeram, ao invés de ficar "hard-coded" dentro do módulo. (vejam, o codeclimate reclamou das duplicações, e faz sentido, "dados" não devem ficar no source dessa forma. ;)