aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAnthony van der Hoorn <[email protected]>2014-07-23 01:02:03 +0100
committerAnthony van der Hoorn <[email protected]>2014-07-23 01:02:03 +0100
commit45160e59709716801d08dc211cc64435a789b604 (patch)
tree431124b56f0d1b506174e134526086911a728e6b
parent6bed149da3df03edbe9a87145c8471d965d1bc3b (diff)
downloadfaker-45160e59709716801d08dc211cc64435a789b604.tar.xz
faker-45160e59709716801d08dc211cc64435a789b604.zip
Updates to switch changes over to using 2 args to support min/max
-rw-r--r--lib/random.js11
-rw-r--r--test/random.unit.js7
2 files changed, 8 insertions, 10 deletions
diff --git a/lib/random.js b/lib/random.js
index 63347278..6205dc09 100644
--- a/lib/random.js
+++ b/lib/random.js
@@ -1,13 +1,12 @@
var definitions = require('./definitions');
var random = {
// returns a single random number based on a max number or range
- number: function (range) {
- if(Array.isArray(range)) {
- var min = range[0];
- var max = range[1];
- return Math.floor((Math.random() * max) + min);
+ number: function (min, max) {
+ if (max === undefined) {
+ max = min;
+ min = 0;
}
- return Math.floor(Math.random() * range);
+ return Math.floor((Math.random() * max) + min);
},
// takes an array and returns the array randomly sorted
diff --git a/test/random.unit.js b/test/random.unit.js
index be180b17..89f5b786 100644
--- a/test/random.unit.js
+++ b/test/random.unit.js
@@ -11,10 +11,9 @@ describe("random.js", function () {
assert.ok(faker.random.number(max) < max);
});
it("returns a random number between a range", function() {
- var range = [1, 10];
- var min = range[0];
- var max = range[1]
- var randomNumber = faker.random.number(range);
+ var min = 1;
+ var max = 10;
+ var randomNumber = faker.random.number(1, 10);
assert.ok( randomNumber >= min);
assert.ok( randomNumber <= max);
});