test(validators): remove duplicated test block from isMobilePhone by pano9000 · Pull Request #2137 · validatorjs/validator.js
Conversation
not sure what happened there, when this was committed with 2e6cc37 The test here is just a "merged" test of the individual en-ZA and be-BY tests. The individual tests themselves for en-ZA and be-BY are also still there and valid.
Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage 👍
Coverage data is based on head (
8cbd2d2) compared to base (531dc7f).
Patch has no changes to coverable lines.
Additional details and impacted files
@@ Coverage Diff @@ ## master #2137 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 104 104 Lines 2308 2308 Branches 578 578 ========================================= Hits 2308 2308
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch!
This was referenced
Jun 3, 2024This was referenced
Jun 12, 2024This was referenced
Jun 20, 2024This was referenced
Jul 10, 2024This was referenced
Jul 19, 2024This was referenced
Aug 30, 2024This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters