fix: Vehicle vin is always 17 characters long

Author: demipel8

src/vehicle.ts

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

test/vehicle.spec.ts

@@ -106,6 +106,24 @@ describe('vehicle', () => {
           );
         });
       });
+      
+      describe('vin()', () => {
+        it('returns valid vin number', () => {
+          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})$/
+          );
+        });
+
+        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('model()', () => {
         it('should return random vehicle model', () => {