[api] [refactor] Replaced Math.Random calls with Random.number. Random ranges are now inclusive of max instead of exclusive. #88 #48

1 files changed, 20 insertions, 9 deletions

diff --git a/test/random.unit.js b/test/random.unit.js
index c50bfada..2403ab1a 100644
--- a/test/random.unit.js
+++ b/test/random.unit.js
@@ -6,21 +6,32 @@ if (typeof module !== 'undefined') {
 
 describe("random.js", function () {
   describe("number", function() {
-    it("returns a random number given a maximum value", function() {
-      var max = 10;
-      assert.ok(faker.random.number(max) < max);
-    });
-    it("returns a random number between a range", function() {
-      var min = 1;
+
+    it("returns a random number given a maximum value as Number", function() {
       var max = 10;
-      var randomNumber = faker.random.number(1, 10);
-      assert.ok( randomNumber >= min);
-      assert.ok( randomNumber <= max);
+      assert.ok(faker.random.number(max) <= max);
     });
+
     it("returns zero when given a max of zero.", function() {
       var max = 0;
       var randomNumber = faker.random.number(max);
       assert.ok( randomNumber === 0 );
     });
+
+
+    it("returns a random number given a maximum value as Object", function() {
+      var options = { max: 10 };
+      assert.ok(faker.random.number(options) < options.max);
+    });
+
+    it("returns a random number between a range", function() {
+      var options = { min: 1, max: 10 };
+      for(var i = 0; i < 100; i++) {
+        var randomNumber = faker.random.number(options);
+        assert.ok(randomNumber >= options.min);
+        assert.ok(randomNumber <= options.max);
+      }
+    });
   });
+
 });