Add allow_leading_zeroes option to isInt()

src/lib/isInt.js

@@ -1,10 +1,23 @@
 import assertString from './util/assertString';
 
-const int = /^[-+]?[0-9]+$/;
+const int = /^(?:[-+]?(?:0|[1-9][0-9]*))$/;
+const intLeadingZeroes = /^[-+]?[0-9]+$/;
 
 export default function isInt(str, options) {
   assertString(str);
   options = options || {};
-  return int.test(str) && (!options.hasOwnProperty('min') ||
-    str >= options.min) && (!options.hasOwnProperty('max') || str <= options.max);
+
+  // Get the regex to use for testing, based on whether
+  // leading zeroes are allowed or not.
+  let regex = (
+    options.hasOwnProperty('allow_leading_zeroes') && options.allow_leading_zeroes ?
+    intLeadingZeroes :
+    int
+  );
+
+  // Check min/max
+  let minCheckPassed = (!options.hasOwnProperty('min') || str >= options.min);
+  let maxCheckPassed = (!options.hasOwnProperty('max') || str <= options.max);
+
+  return regex.test(str) && minCheckPassed && maxCheckPassed;
 }

test/validators.js

@@ -883,6 +883,27 @@ describe('Validators', function () {
   it('should validate integers', function () {
     test({
       validator: 'isInt',
+      valid: [
+        '13',
+        '123',
+        '0',
+        '123',
+        '-0',
+        '+1',
+      ],
+      invalid: [
+        '01',
+        '-01',
+        '000',
+        '100e10',
+        '123.123',
+        '   ',
+        '',
+      ],
+    });
+    test({
+      validator: 'isInt',
+      args: [{ allow_leading_zeroes: true }],
       valid: [
         '13',
         '123',
@@ -893,6 +914,8 @@ describe('Validators', function () {
         '01',
         '-01',
         '000',
+        '-000',
+        '+000',
       ],
       invalid: [
         '100e10',