brutils-python icon indicating copy to clipboard operation
brutils-python copied to clipboard

Release v<2.3.0>

Open Oleari19 opened this issue 10 months ago • 3 comments

Descrição

Utilitário is_valid_renavam https://github.com/brazilian-utils/brutils-python/issues/430

Implementação da issue #430

Mudanças Propostas

Criação da função de validar o renavam e seus rescpectivos testes, alterações também na documentação conforme pede o arquivo de contribuição

Checklist de Revisão

  • [ X] Eu li o Contributing.md
  • [ X] Os testes foram adicionados ou atualizados para refletir as mudanças (se aplicável).
  • [ X] Foi adicionada uma entrada no changelog / Meu PR não necessita de uma nova entrada no changelog.
  • [ X] A documentação em português foi atualizada ou criada, se necessário.
  • [ X] Se feita a documentação, a atualização do arquivo em inglês.
  • [ X] Eu documentei as minhas mudanças no código, adicionando docstrings e comentários. Instruções
  • [ X] O código segue as diretrizes de estilo e padrões de codificação do projeto.
  • [ X] Todos os testes passam. Instruções
  • [ X] O Pull Request foi testado localmente. Instruções
  • [ X] Não há conflitos de mesclagem.

Comentários Adicionais (opcional)

Ao rodar 'make test', foi preciso executar os comandos 'pip install holidays' e 'pip install num2words', seria important documentar.

Além disso, não consegui rodar apenas o meu arquivo de testes criados, devido a isso fiz usando pytest e ao final rodei o 'make test' e passou todos.

Issue Relacionada

Closes #430

Oleari19 avatar Feb 01 '25 14:02 Oleari19

@tiagornandrade não entendi muito o problema... peguei todas as alterações do https://github.com/brazilian-utils/brutils-python/pull/488 e adicionei na branch 430. Rodei todos os testes e passaram!! No aguardo de um feedback

Oleari19 avatar Feb 05 '25 16:02 Oleari19

Sugestão de Issue: fazer as devidas alterações para o 'test_date_utils.TestIsHoliday' funcionar. Segue o resultado final que aparece no terminal ao realizar o comando 'make test', todos passam menos ele

====================================================================== FAIL: test_data_passada_futura (test_date_utils.TestIsHoliday)

Traceback (most recent call last): File "/home/maria/Documentos/teste/Projeto/brutils-python/tests/test_date_utils.py", line 63, in test_data_passada_futura self.assertTrue(is_holiday(datetime(2150, 12, 25))) # Ano futuro AssertionError: False is not true


Ran 137 tests in 2.391s

FAILED (failures=1)

Oleari19 avatar Feb 05 '25 16:02 Oleari19

@Oleari19 não é um problema na resolução da issue, foi só um lembrete que tinha o seu PR e outro fazendo ajuste no mesmo arquivo. Mas o PR anterior já tratou o possível conflito

tiagornandrade avatar Feb 05 '25 17:02 tiagornandrade