Merge branch 'deprecate-old' into dev

.travis.yml

@@ -1,3 +1,3 @@
 language: node_js
 node_js:
-  - "6"
+  - "stable"

.vs/VSWorkspaceState.json

@@ -0,0 +1,6 @@
+{
+  "ExpandedNodes": [
+    ""
+  ],
+  "PreviewInSolutionExplorer": false
+}

deprecated/v1.js

@@ -0,0 +1,21 @@
+'use strict'
+// Through v0.1.6
+
+const Purify = require('purify-int')
+const { deprecate } = require('../util')
+
+function ceil(max) {
+	const safeMax = Purify.asInt(max);
+	deprecate("make-random.ceil()", "0.1.6");
+	return Math.ceil(Math.random() * safeMax);
+}
+function floor(max) {
+	const safeMax = Purify.asInt(max);
+	deprecate("make-random.floor()", "0.1.6");
+	return Math.floor(Math.random() * safeMax);
+}
+
+module.exports = {
+	floor,
+	ceil
+}

index.js

@@ -3,22 +3,9 @@
 const Purify = require('purify-int')
 const Crypto = require('crypto')
 
-const makeRandom = {
-	ceil(max) {
-		const safeMax = Purify.asInt(max);
-		deprecate("make-random.ceil()", "0.1.6");
-		return Math.ceil(Math.random() * safeMax);
-	},
-	floor(max) {
-		const safeMax = Purify.asInt(max);
-		deprecate("make-random.floor()", "0.1.6");
-		return Math.floor(Math.random() * safeMax);
-	}
-};
+const { floor, ceil } = require('./deprecated/v1')
 
-function deprecate(func, version) {
-	console.log(`${func} is deprecated as of ${version} and may be removed at any time.
-	Please convert to make-random-legacy if this function is still needed.`)
+module.exports = {
+	floor,
+	ceil
 }
-
-module.exports = makeRandom;

package.json

@@ -4,7 +4,7 @@
   "description": "💯 Make a random number",
   "main": "index.js",
   "scripts": {
-    "test": "mocha"
+    "test": "mocha --recursive"
   },
   "repository": {
     "type": "git",

test/main/index.js

@@ -0,0 +1,46 @@
+'use strict';
+
+const chai = require('chai');
+const expect = chai.expect;
+const { ceil, floor } = require('../../index');
+
+describe('makeRandom ceil()', () => {
+  it(`should have the method 'ceil'`, ()=> {
+    expect(ceil).to.be.a('function');
+  });
+  it(`should return a number`, () => {
+    expect(ceil(10)).to.be.a('number');
+    expect(ceil('10')).to.be.an('number');
+  });
+  it(`should not return a negative zero`, () => {
+    expect(ceil(-0)).to.equal(0);
+  });
+
+	it(`should return 0 if a number or number string is not passed to ceil()`, () => {
+    expect(ceil('test')).to.equal(0);
+    expect(ceil(true)).to.equal(0);
+    expect(ceil({})).to.equal(0);
+    expect(ceil([])).to.equal(0);
+    expect(ceil(() => {})).to.equal(0);
+  });
+});
+
+describe('makeRandom floor()', () => {
+  it(`should have the method 'floor'`, ()=> {
+    expect(floor).to.be.a('function');
+  });
+  it(`should return a number`, () => {
+    expect(floor(10)).to.be.a('number');
+    expect(floor('10')).to.be.an('number');
+  });
+  it(`should not return a negative zero`, () => {
+    expect(floor(-0)).to.equal(0);
+  });
+  it(`should return 0 if a number or number string is not passed to ceil()`, () => {
+    expect(floor('test')).to.equal(0);
+    expect(floor(true)).to.equal(0);
+    expect(floor({})).to.equal(0);
+    expect(floor([])).to.equal(0);
+    expect(floor(() => {})).to.equal(0);
+	});
+});

util/index.js

@@ -0,0 +1,8 @@
+function deprecate(func, version) {
+	console.log(`${func} is deprecated as of ${version} and may be removed at any time.
+	Please convert to make-random-legacy if this function is still needed.`)
+}
+
+module.exports = {
+	deprecate
+}