Version 1.0.0

1. Fix bug of `isChineseIdCard`
2. Add more test case.

Author: SFantasy

example/example.js

@@ -23,4 +23,4 @@ console.log(validator.isAllEnglish('test'));
 console.log(validator.isAllDigit('12345'));
 
 // isChineseIdCard
-console.log(validator.isChineseIdCard('330682199110273013'));
+console.log(validator.isChineseIdCard('15210319861215033x'));

lib/index.js

@@ -72,10 +72,10 @@ module.exports = {
       var year = num.substring(6, 10);
       var month = num.substring(10, 12);
       var day = num.substring(12, 14);
-      var temp = new Date(year, parseInt(month, 10), parseInt(day, 10));
+      var temp = new Date(year, parseInt(month, 10) - 1, parseInt(day, 10));
 
       return (temp.getFullYear() === parseInt(year, 10) &&
-          temp.getMonth() === parseInt(month, 10) &&
+          temp.getMonth() + 1 === parseInt(month, 10) &&
           temp.getDate() === parseInt(day, 10));
     }
 
@@ -89,11 +89,11 @@ module.exports = {
         temp[17] = 10;
       }
 
-      for (var i = 0; i < temp.length; i++) {
+      for (var i = 0; i < temp.length - 1; i++) {
         sum += wc[i] * temp[i];
       }
 
-      return temp[17] === validCodeArr[sum % 11];
+      return temp[17] == validCodeArr[sum % 11];
     }
   },
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "is-valid",
-  "version": "0.1.0",
+  "version": "1.0.0",
   "description": "String validator for more",
   "main": "index.js",
   "scripts": {

test/test.js

@@ -5,7 +5,7 @@
  * @description
  * @author Fantasy <fantasyshao@icloud.com>
  * @create 2014-09-10
- * @update 2014-09-10
+ * @update 2014-09-16
  */
 
 var validator = require('../index');
@@ -142,4 +142,45 @@ describe('Validate', function () {
     });
   });
 
+  describe('#isVisaCard', function () {
+    it('should validate a Visa card', function () {
+      test({
+        validator: 'isVisaCard',
+        valid: [
+          '4012888888881881',
+          '4111111111111111',
+          '4222222222222'
+        ],
+        invalid: [
+          // mess
+          '1234567891012312',
+          // JCB
+          '3566002020360505',
+          // Master Card
+          '5555555555554444'
+        ]
+      });
+    });
+  });
+
+  describe('#isMasterCard', function () {
+    it('should validate a Master card', function () {
+      test({
+        validator: 'isMasterCard',
+        valid: [
+          '5555555555554444',
+          '5105105105105100'
+        ],
+        invalid: [
+          // mess
+          '1234567891012312',
+          // JCB
+          '3566002020360505',
+          // Visa
+          '4012888888881881'
+        ]
+      });
+    });
+  });
+
 });