feat(isBase32): Add option for Crockford's base 32 alternative by BigOsvaap · Pull Request #1888 · validatorjs/validator.js
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would suggest you move this test out to its own it() statement so it is more clear precisely what is being tested. At least many other tests are split up like this. See 'should validate JSON' and 'should validate JSON with primitives'.
Examples to the contrary are found as well ('should validate ISSNs' has everything in one it() statement), so I suppose a clarification of the convention here would be nice, but that's a bigger topic.