feat(pt-BR): tax id, passport and license plates #1613

mschunke · 2021-02-21T12:53:04Z

Included Tax ID, passport, and license plate validators for pt-BR (Brazil) locale. Tests written and passed. Locale added to the documentation.

Checklist

[ X ] PR contains only changes related; no stray files, etc.
[ X ] README updated (where applicable)
[ X ] Tests written (where applicable)

renanmontebelo · 2021-02-28T04:15:33Z

There's some overlap with these already opened PRs:

We might consider to join forces if you wish.

mschunke · 2021-03-01T11:08:54Z

Sure, for what I see @renanmontebelo license plate approach is more complete as "mercosur" plates are available too. My approach for tax Id validation uses the current .isTaxId function and supports also id (CNPJ) validation for entities and well as CPF (persons) validations.

I would suggest to keep his for license plates and mine for tax Ids.

tux-tn

Thank you for your contribution @mschunke. Can you please resolve merge conflicts (delete validator.js and validator.min.js files and fix conflict in README.md ) and that would be great if you join forces and decide what to keep in each PR.
MB I approved the PR instead of requesting changes 😅

codecov · 2021-03-05T11:40:29Z

Codecov Report

Merging #1613 (410ee54) into master (deb1d1e) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master     #1613   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          100       100           
  Lines         1807      1849   +42     
=========================================
+ Hits          1807      1849   +42

Impacted Files	Coverage Δ
src/lib/isPassportNumber.js	`100.00% <ø> (ø)`
src/lib/isTaxID.js	`100.00% <100.00%> (ø)`

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update deb1d1e...a511310. Read the comment docs.

mschunke · 2021-03-05T11:51:04Z

Hey @tux-tn, all conflicts are solved and build checks passed. Anything else I should do?

tux-tn · 2021-03-05T13:22:16Z

src/lib/isTaxID.js

+      tin === '88888888888' ||
+      tin === '99999999999' ||
+      tin === '00000000000'
+    ) return false;


This line is not covered by tests, you probably just need to add a testcase with an invalid CPF

tux-tn · 2021-03-05T13:22:55Z

src/lib/isTaxID.js

+    tin === '66666666666666' ||
+    tin === '77777777777777' ||
+    tin === '88888888888888' ||
+    tin === '99999999999999') { return false; }


Same here, this condition is not covered, adding a test with an invalid CNPJ should fix it

tux-tn

Thank you @mschunke ! I added two comments concerning untested conditions/code branches. Can you check it?

tux-tn · 2021-03-05T13:25:57Z

@renanmontebelo @mschunke what have you decided concerning what approach to keep?
Should this one add only tax id and passport and the other one add license plate validation?

mschunke · 2021-03-05T13:31:13Z

@renanmontebelo @mschunke what have you decided concerning what approach to keep?
Should this one add only tax id and passport and the other one add license plate validation?

Hey, @tux-tn,
Got no reply from @renanmontebelo, but I'm fine with this PR just adding tax id and passport and using the other PR for license plates.

mschunke · 2021-03-05T13:32:33Z

Thank you @mschunke ! I added two comments concerning untested conditions/code branches. Can you check it?

@tux-tn
Was wondering where those untested lines were... thank you! Code updated with 100% coverage.

renanmontebelo · 2021-03-06T13:47:06Z

@mschunke sorry for the delay, I can only work on this in the weekends. Selecting your PR for tax id and my PR for license plates sounds good. I'll close #1589 in favor of yours.

My tax id implementation also accepts an optional parameter to require or ignore formatting characters like . and -. Please feel free to copy into yours if you think it's a good addition. Thank you.

mschunke · 2021-03-06T19:49:19Z

@renanmontebelo np. We all work on these on our spare time. I've removed my license plate validation for we can use yours, and kept my tax ID validator as agreed. Thanks for the cooperation!

btw, my tax id validator accepts both formatted and unformatted input.

renanmontebelo · 2021-03-06T21:28:02Z

Sounds good, thank you for the cooperation as well. I missed that yours accept both formatted/unformatted CPFs, that's awesome.

tux-tn

Thank you @mschunke 🎉 ! Can you just remove the isLicensePlate PT-BR entry in README.md? i think it's the only necessary change

mschunke · 2021-03-08T12:28:52Z

@tux-tn sorry about that, forget about the readme file! File updated. Please let me know if you need anything else 😁

tux-tn

@mschunke that was fast 😅
LGTM 🎉 Thank you again for the fast response and for your collaboration with @renanmontebelo !

profnandaa

Thanks @mschunke for the PR and welcome to the project! 🎉

murilopulley · 2021-03-12T10:28:41Z

Thanks @profnandaa! Happy to help :)

feat(pt-BR): tax id, passport and license plates

effd4c9

tux-tn approved these changes Mar 1, 2021 •

edited

Loading

View reviewed changes

tux-tn requested changes Mar 1, 2021

View reviewed changes

tux-tn added 🍿 discussion 🧹 needs-update For PRs that need to be updated before landing labels Mar 1, 2021

mschunke added 2 commits March 5, 2021 08:31

delete compiled files

fc51d4d

Merge branch 'master' of https://github.com/validatorjs/validator.js

5da17cd

tux-tn reviewed Mar 5, 2021

View reviewed changes

tux-tn requested changes Mar 5, 2021

View reviewed changes

fix(tests): tests now covering 100% of code

6967456

renanmontebelo mentioned this pull request Mar 6, 2021

feat(isCPF): adding validator for CPF (brazilian SSN) #1589

Closed

3 tasks

fix(licenseplate): removed license plate pt-br

23ae508

mschunke requested a review from tux-tn March 6, 2021 19:49

tux-tn requested changes Mar 8, 2021

View reviewed changes

fix(readme): removed license plate changes

a511310

tux-tn approved these changes Mar 8, 2021

View reviewed changes

tux-tn removed 🍿 discussion 🧹 needs-update For PRs that need to be updated before landing labels Mar 8, 2021

tux-tn added needs-more-review 🎉 first-pr labels Mar 8, 2021

profnandaa approved these changes Mar 12, 2021

View reviewed changes

profnandaa merged commit 3c771e8 into validatorjs:master Mar 12, 2021

profnandaa mentioned this pull request Apr 19, 2021

[release staging] April 2021 Release (v13.6.0) #1650

Closed

tux-tn mentioned this pull request Oct 4, 2021

feat(isCpf): Add Brazilian CPF validation #1755

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(pt-BR): tax id, passport and license plates #1613

feat(pt-BR): tax id, passport and license plates #1613

mschunke commented Feb 21, 2021

renanmontebelo commented Feb 28, 2021

mschunke commented Mar 1, 2021

tux-tn left a comment

codecov bot commented Mar 5, 2021 •

edited

Loading

mschunke commented Mar 5, 2021

tux-tn Mar 5, 2021

tux-tn Mar 5, 2021

tux-tn left a comment

tux-tn commented Mar 5, 2021

mschunke commented Mar 5, 2021 •

edited

Loading

mschunke commented Mar 5, 2021 •

edited

Loading

renanmontebelo commented Mar 6, 2021

mschunke commented Mar 6, 2021

renanmontebelo commented Mar 6, 2021

tux-tn left a comment

mschunke commented Mar 8, 2021

tux-tn left a comment

profnandaa left a comment

murilopulley commented Mar 12, 2021

feat(pt-BR): tax id, passport and license plates #1613

feat(pt-BR): tax id, passport and license plates #1613

Conversation

mschunke commented Feb 21, 2021

Checklist

renanmontebelo commented Feb 28, 2021

mschunke commented Mar 1, 2021

tux-tn left a comment

Choose a reason for hiding this comment

codecov bot commented Mar 5, 2021 • edited Loading

Codecov Report

mschunke commented Mar 5, 2021

tux-tn Mar 5, 2021

Choose a reason for hiding this comment

tux-tn Mar 5, 2021

Choose a reason for hiding this comment

tux-tn left a comment

Choose a reason for hiding this comment

tux-tn commented Mar 5, 2021

mschunke commented Mar 5, 2021 • edited Loading

mschunke commented Mar 5, 2021 • edited Loading

renanmontebelo commented Mar 6, 2021

mschunke commented Mar 6, 2021

renanmontebelo commented Mar 6, 2021

tux-tn left a comment

Choose a reason for hiding this comment

mschunke commented Mar 8, 2021

tux-tn left a comment

Choose a reason for hiding this comment

profnandaa left a comment

Choose a reason for hiding this comment

murilopulley commented Mar 12, 2021

codecov bot commented Mar 5, 2021 •

edited

Loading

mschunke commented Mar 5, 2021 •

edited

Loading

mschunke commented Mar 5, 2021 •

edited

Loading