lipe14-ops / brasilapy

Brasil API client
MIT License
94 stars 22 forks source link

V 1.0.0 proposta =) #3

Closed joepreludian closed 2 years ago

joepreludian commented 2 years ago

E aí @lipe14-ops; Fiz uma pequena revolução no seu projeto. Tentei usar o máximo de boas práticas possíveis pra tentar deixar a API o mais IDE friendly possível.

Também adicionei suporte a testes e está com cobertura de 100%:

$ pytest --cov

Agora acho que, para os próximos passos, já enxergo adicionando uma pipeline para testarmos e avaliarmos os pull requests de maneira automatizada e eventualmente fazer o publishing dentro do PyPI.

Por favor, sinta-se à vontade para escrutinizar meu código. =)

---------- coverage: platform darwin, python 3.10.6-final-0 ----------
Name                                    Stmts   Miss  Cover
-----------------------------------------------------------
brasilapy/__init__.py                       1      0   100%
brasilapy/client.py                        76      0   100%
brasilapy/constants.py                     15      0   100%
brasilapy/exceptions.py                     7      0   100%
brasilapy/models/__init__.py                0      0   100%
brasilapy/models/cnpj.py                   69      0   100%
brasilapy/models/general.py                75      0   100%
brasilapy/processor.py                     18      0   100%
brasilapy/utils.py                          3      0   100%
tests/banks/__init__.py                     0      0   100%
tests/banks/conftest.py                     4      0   100%
tests/banks/test_banks.py                  21      0   100%
tests/cep/__init__.py                       0      0   100%
tests/cep/conftest.py                       7      0   100%
tests/cep/test_cep.py                      34      0   100%
tests/cnpj/__init__.py                      0      0   100%
tests/cnpj/conftest.py                      4      0   100%
tests/cnpj/test_cnpj.py                    24      0   100%
tests/conftest.py                           5      0   100%
tests/ddd/__init__.py                       0      0   100%
tests/ddd/conftest.py                       4      0   100%
tests/ddd/test_ddd.py                      19      0   100%
tests/feriados/__init__.py                  0      0   100%
tests/feriados/conftest.py                  8      0   100%
tests/feriados/test_feriados.py            17      0   100%
tests/fipe/__init__.py                      0      0   100%
tests/fipe/conftest.py                     10      0   100%
tests/fipe/test_fipe.py                    23      0   100%
tests/ibge/__init__.py                      0      0   100%
tests/ibge/conftest.py                      7      0   100%
tests/ibge/test_ibge.py                    31      0   100%
tests/registro_br/__init__.py               0      0   100%
tests/registro_br/conftest.py               4      0   100%
tests/registro_br/test_registro_br.py      14      0   100%
tests/taxas/__init__.py                     0      0   100%
tests/taxas/conftest.py                     4      0   100%
tests/taxas/test_taxas.py                  15      0   100%
tests/test_processor.py                    49      0   100%
tests/test_utils.py                        14      0   100%
-----------------------------------------------------------
TOTAL                                     582      0   100%

========================== 31 passed in 0.13s ===========================
lipe14-ops commented 2 years ago

Bom quase que perfeito!!!

mas estou com problemas na hora de rodar os testes... Fiz algo de errado? image

também há alguns __init__.py perdidos por ai sem nenhum conteúdo... ( deletei rsrs )

joepreludian commented 2 years ago

Então. Vc precisa rodar via poetry.

$ poetry install
$ poetry run pytest --cov .

Ou instalar todas as dependências. Me diz se deu certo. :)

On Sat, Oct 1, 2022, 9:29 AM Filipe Soares @.***> wrote:

Bom quase que perfeito!!!

mas estou com problemas na hora de rodar os testes... Fiz algo de errado? [image: image] https://user-images.githubusercontent.com/78698099/193409579-70125e4e-1474-4463-a3a3-aa1327332fcb.png

— Reply to this email directly, view it on GitHub https://github.com/lipe14-ops/brasilapy/pull/3#issuecomment-1264350705, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAURDN3YEWNX7E3MLN3BUQDWBAVC7ANCNFSM6AAAAAAQZN2WUE . You are receiving this because you authored the thread.Message ID: @.***>

joepreludian commented 2 years ago

Faz assim, meu caro.

$ poetry install $ poetry shell

Ele vai ativar o venv pra você.

$ pytest --cov .

E caso queira rodar todos os lint e reformatacao do código basta rodar...

$ pre-commit run --all-files

Não sei se você conhece o pre-commit. Ele pode rodar uma série de comandos depois que você faz os commits pra validar se tudo está certinho.

Pra instalar nos seus git hooks, basta rodar

$ pre-commit install

Daí toda vez que vc fizer um commit ele vai ser executado. :)

Me diz se deu certo, ok? Abraços.

lipe14-ops commented 2 years ago

Deu tudo certo ok??? obrigadão!!!