fix: Vehicle vin is always 17 characters long

Author: demipel8

src/vehicle.ts

@@ -84,7 +84,7 @@ export class Vehicle {
         bannedChars: bannedChars,
       }) +
       this.faker.random.alphaNumeric(1, { bannedChars: bannedChars }) +
-      this.faker.datatype.number({ min: 10000, max: 100000 })
+      this.faker.datatype.number({ min: 10000, max: 99999 })
     ) // return five digit #
       .toUpperCase();
   }

test/vehicle.spec.ts

@@ -58,6 +58,15 @@ describe('vehicle', () => {
         /^([A-HJ-NPR-Z0-9]{10}[A-HJ-NPR-Z0-9]{1}[A-HJ-NPR-Z0-9]{1}\d{5})$/
       );
     });
+
+    it('is 17 characters long', () => {
+      for (let step = 0; step < 300000; step++) {
+        const vin = faker.vehicle.vin();
+        expect(vin).match(
+          /^([A-HJ-NPR-Z0-9]{10}[A-HJ-NPR-Z0-9]{1}[A-HJ-NPR-Z0-9]{1}\d{5})$/
+        );
+      }
+    });
   });
 
   describe('color()', () => {