Merge pull request #353 from ashpr/bitcoinaddress-fix

[api] [fix] finance.bitcoinAddress

Author: Marak

lib/finance.js

@@ -135,6 +135,21 @@ var Finance = function (faker) {
       return symbol;
   }
 
+  /**
+   * bitcoinAddress
+   *
+   * @method  faker.finance.bitcoinAddress
+   */
+  self.bitcoinAddress = function () {
+    var addressLength = faker.random.number({ min: 27, max: 34 });
+
+    var address = faker.random.arrayElement(['1', '3']);
+
+    for (var i = 0; i < addressLength - 1; i++)
+      address += faker.random.alphaNumeric().toUpperCase();
+
+    return address;
+  }
 }
 
 module['exports'] = Finance;

test/finance.unit.js

@@ -203,13 +203,11 @@ describe('finance.js', function () {
         });
     })
 
-    /*
     describe("bitcoinAddress()", function(){
         it("returns a random bitcoin address", function(){
             var bitcoinAddress = faker.finance.bitcoinAddress();
 
             assert.ok(bitcoinAddress.match(/^[A-Z0-9.]{27,34}$/));
         });
     });
-    */
 });