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

Validação de CNH errada #53

Closed
MrEko opened this issue Jan 10, 2020 · 2 comments
Closed

Validação de CNH errada #53

MrEko opened this issue Jan 10, 2020 · 2 comments

Comments

@MrEko
Copy link
Contributor

MrEko commented Jan 10, 2020

O validador implementado não cobre todos os casos corretamente.
Exemplo real que chegou pra mim hoje: 04463004100.

O validador do 4devs usado como referência do projeto indica falso.
O validador em:
https://www.treinaweb.com.br/ferramentas-para-desenvolvedores/validar/cnh
Aponta corretamento como verdadeiro.

Vou pegar o algoritmo correto com o detran. E depois faço um pull request.

Mas já fica o alerta, e é a 2ª vez que encontro erro nas implementação desse 4devs. Não são muito confiáveis não.

E acredito que deva ter mais exemplos nos casos de teste, com apenas 1 caso é muito fácil passar um algoritmo problemático.

@MrEko MrEko mentioned this issue Jan 10, 2020
@geekcom
Copy link
Owner

geekcom commented Jan 13, 2020

Boa @MrEko, obrigado pela ajuda que tem dado na lib, realmente os validadores da 4devs tem decepcionado. Vou escrever novos casos de testes para termos maior cobertura.

@geekcom
Copy link
Owner

geekcom commented Jan 13, 2020

#54

@geekcom geekcom closed this as completed Jan 13, 2020
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

No branches or pull requests

2 participants