feat:retorna codigo do ibge do nome do municipio e da uf #411
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #411 +/- ##
=======================================
Coverage 99.78% 99.79%
=======================================
Files 18 19 +1
Lines 472 495 +23
=======================================
+ Hits 471 494 +23
Misses 1 1 ☔ View full report in Codecov by Sentry. |
| value = file_cities.get(name_city) | ||
|
|
||
| if value == None: | ||
| return None |
There was a problem hiding this comment.
O aviso aqui diz que essa linha do código não está sendo coberta pelos testes. Ou seja, que rodando os testes, essa parte do código nunca é executada. Para isso, precisamos adicionar um teste que caia dentro desse if. Por exemplo:
self.assertIsNone(get_code_by_municipality_name("nome inválido", "rj"))
|
@camilamaia obrigado pela ajuda, em breve aplicarei as mudanças! |
|
|
||
| Retorna: | ||
| * str: O código IBGE do município. Retorna None se o nome não for válido ou não existir. | ||
| Exemplo: |
There was a problem hiding this comment.
pular linha aqui por conta da formatação
| * str: O código IBGE do município. Retorna None se o nome não for válido ou não existir. | ||
| Exemplo: | ||
|
|
||
| ``` |
| @@ -1110,6 +1112,41 @@ Exemplo: | |||
| 'RJ' | |||
| >>> convert_code_to_uf("99") | |||
| >>> | |||
| >>>>>>> main | |||
There was a problem hiding this comment.
essa modificação está correta?
There was a problem hiding this comment.
lembra os passos/comandos para gerar esse arquivo?
seria interessante ter essa referência para uma possível automação futura
| return code | ||
|
|
||
|
|
||
| def _transform_text(municipality_name: str): # type: (str) -> str |
There was a problem hiding this comment.
valeria escrever teste só para essa função?
Descrição
Implementa uma função que recebe o nome do municipio e a sigla da UF e retorna seu código do IBGE.
@mateusoliveira43, @pedromendesjr, @nettorobson
Mudanças Propostas
Checklist de Revisão
Comentários Adicionais (opcional)
Estamos na dúvida na utilização de um arquivo json para leitura dos códigos referentes ao municípios (caso implementado) ou utilização de uma API externa para obter tais códigos.
Uma sugestão de desenvolvimento para melhoria seria a criação de um processo de automação para atualização do json
Issue Relacionada
Closes #399