Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IMP] l10n_br_cnpj_search: add valid cnpj in tests. #3451

Closed
wants to merge 1 commit into from

Conversation

Tiago370
Copy link

No teste test_serpro_empresa, está sendo criado um partner do tipo "Company" para cada sócio. Porém até então os dados de mock do campo "cpf" não são CNPJs válidos, em alguns testes automáticos isso tem sido um problema, pois a constrain _check_cnpj_inscr_est vai retornar um erro.

Esse PR altera os "cpf" dos sócios do mock específico para esse teste, trocando por CNPJs válidos, e também apaga alguns dados do mock que não são relevantes para o teste.

@antoniospneto
Copy link
Contributor

@Tiago370 mas faz sentido ser um CNPJ? O sócio tem quer ser um CPF válido, não um CNPJ

@Tiago370
Copy link
Author

@antoniospneto Você tem razão, olhei melhor o código aqui, não faz sentido.
Alguns nomes de variáveis e o fato da validação que retornava um erro na _check_cnpj_inscr_est ser uma validação de CNPJ, acreditei se tratar CNPJs. Mas olhando o contexto maior dessa validação eu entendi.
Vou fechar o PR.

@Tiago370 Tiago370 closed this Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants