Move lenght checks to Account

Author: bartekn

src/account.js

@@ -10,7 +10,10 @@ export class Account {
     */
     static isValidAddress(address) {
         try {
-            decodeCheck("accountId", address);
+            let decoded = decodeCheck("accountId", address);
+            if (decoded.length !== 32) {
+                return false;
+            }
         } catch(err) {
             return false;
         }

src/strkey.js

@@ -32,10 +32,6 @@ export function decodeCheck(versionByteName, encoded) {
     throw new Error(`invalid version byte. expected ${expectedVersion}, got ${versionByte}`);
   }
 
-  if (decoded.length !== 35) {
-    throw new Error(`Decoded length is invalid. Expected 35, got ${decoded.length}`);
-  }
-
   let expectedChecksum = calculateChecksum(payload);
 
   if (!verifyChecksum(expectedChecksum, checksum)) {

test/unit/account_test.js

@@ -23,6 +23,11 @@ describe('Account.isValidAddress', function() {
 
   it("returns false for invalid address", function() {
     var addresses = [
+      'GBPXX0A5N4JYPESHAADMQKBPWZWQDQ64ZV6ZL2S3LAGW4SY7NTCMWIVL',
+      'GCFZB6L25D26RQFDWSSBDEYQ32JHLRMTT44ZYE3DZQUTYOL7WY43PLBG++',
+      'GADE5QJ2TY7S5ZB65Q43DFGWYWCPHIYDJ2326KZGAGBN7AE5UY6JVDRRA',
+      'GB6OWYST45X57HCJY5XWOHDEBULB6XUROWPIKW77L5DSNANBEQGUPADT2',
+      'GB6OWYST45X57HCJY5XWOHDEBULB6XUROWPIKW77L5DSNANBEQGUPADT2T',
       'GDXIIZTKTLVYCBHURXL2UPMTYXOVNI7BRAEFQCP6EZCY4JLKY4VKFNLT',
       'gWRYUerEKuz53tstxEuR3NCkiQDcV4wzFHmvLnZmj7PUqxW2wt',
       'test',