Merge pull request #353 from ashpr/bitcoinaddress-fix

[api] [fix] finance.bitcoinAddress

2 files changed, 15 insertions, 2 deletions

diff --git a/lib/finance.js b/lib/finance.js
index a7c7b873..1db94dcf 100644
--- a/lib/finance.js
+++ b/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;
diff --git a/test/finance.unit.js b/test/finance.unit.js
index 2c5442d8..6bbb51c6 100644
--- a/test/finance.unit.js
+++ b/test/finance.unit.js
@@ -203,7 +203,6 @@ describe('finance.js', function () {
         });
     })
 
-    /*
     describe("bitcoinAddress()", function(){
         it("returns a random bitcoin address", function(){
             var bitcoinAddress = faker.finance.bitcoinAddress();
@@ -211,5 +210,4 @@ describe('finance.js', function () {
             assert.ok(bitcoinAddress.match(/^[A-Z0-9.]{27,34}$/));
         });
     });
-    */
 });
\ No newline at end of file