Merge pull request #887 from edumoreira1506/fix/amount-return-number

Fix return of finance.amount

2 files changed, 16 insertions, 5 deletions

diff --git a/lib/finance.js b/lib/finance.js
index c16b9e49..917719d5 100644
--- a/lib/finance.js
+++ b/lib/finance.js
@@ -110,8 +110,9 @@ var Finance = function (faker) {
       dec = dec === undefined ? 2 : dec;
       symbol = symbol || '';
       var randValue = faker.random.number({ max: max, min: min, precision: Math.pow(10, -dec) });
+      var stringNumber = symbol + randValue.toFixed(dec);
 
-      return symbol + randValue.toFixed(dec);
+      return Number(stringNumber);
   };
 
   /**
diff --git a/test/finance.unit.js b/test/finance.unit.js
index 6c1cb142..432a9332 100644
--- a/test/finance.unit.js
+++ b/test/finance.unit.js
@@ -160,9 +160,9 @@ describe('finance.js', function () {
 
         });
 
-        it("should use the defaul decimal location when not passing arguments", function () {
+        it("should use the default decimal location when not passing arguments", function () {
 
-            var amount = faker.finance.amount();
+            var amount = faker.finance.amount().toString();
 
             var decimal = '.';
             var expected = amount.length - 3;
@@ -200,7 +200,7 @@ describe('finance.js', function () {
             var amount = faker.finance.amount(100, 100, 1);
 
             assert.ok(amount);
-            assert.strictEqual(amount , '100.0', "the amount should be equal 100.0");
+            assert.strictEqual(amount , 100.0, "the amount should be equal 100.0");
         });
 
         it("it should handle argument dec = 0", function () {
@@ -208,7 +208,17 @@ describe('finance.js', function () {
             var amount = faker.finance.amount(100, 100, 0);
 
             assert.ok(amount);
-            assert.strictEqual(amount , '100', "the amount should be equal 100");
+            assert.strictEqual(amount , 100, "the amount should be equal 100");
+        });
+
+        it("it should return a number", function() {
+
+            var amount = faker.finance.amount(100, 100, 0);
+
+            var typeOfAmount = typeof amount;
+
+            assert.ok(amount);
+            assert.strictEqual(typeOfAmount , 'number', "the amount type should be number");
         });
 
     });