Merge pull request #11 from slacroix/feature-tedious

 Removed dependency on seriate. Now uses tedious directly.

Author: digitalBush

README.md

@@ -6,7 +6,7 @@ NHibernate-style hi/lo ID generator for node.js
 
 ### Requiring & Configuring
 
-node-hilo exports a factory function that takes an instance of [seriate](https://github.com/leankit-labs/seriate), and a configuration object:
+node-hilo exports a factory function that takes a configuration object:
 ```javascript
 /*
 	The configuration argument can contain the following:
@@ -23,23 +23,18 @@ node-hilo exports a factory function that takes an instance of [seriate](https:/
 		}
 	}
 */
-var hilo = require( "node-hilo" )( seriate, configuration );
+var hilo = require( "node-hilo" )( configuration );
 ```
 
 ### How to Use
 
-node-hilo exports two module members: a `nextId` method and a read-only property called `hival`. You will likely *never* need to care about the `hival` value - it's there for diagnostics and testing. The `nextId` method returns a promise, with the newly generated ID being passed to the success callback:
+node-hilo exports three module members: a `nextId` method, a `nextIds` method and a read-only property called `hival`. You will likely *never* need to care about the `hival` value - it's there for diagnostics and testing. The `nextId` method returns a promise, with the newly generated ID being passed to the success callback:
 
 ```javascript
-hilo.nextId(
-	function( id ){
-		// Now you can use the id on your object, etc.
-		api.saveNewThing( id, newThing );
-	},
-	function( err) {
-		console.log( "O NOES!", err );
-	}
-);
+const id = await hilo.nextId();
+
+// block of 100 ids
+const ids = await hilo.nextIds( 100 )
 ```
 
 ## The More You Know...

package-lock.json

@@ -40,23 +40,24 @@
         "cover:show": "open \"file://$PWD/coverage/index.html\""
     },
     "dependencies": {
-        "big-integer": "^1.4.4",
-        "lodash": "^4.17.10",
-        "machina": "^2.0.2"
+        "big-integer": "^1.6.40",
+        "lodash": "^4.17.11",
+        "machina": "^4.0.2",
+        "tedious": "^4.1.1"
     },
     "devDependencies": {
-        "chai": "^4.1.2",
+        "chai": "^4.2.0",
         "chai-as-promised": "^7.1.1",
+        "dirty-chai": "^2.0.1",
         "eslint": "^4.19.1",
         "eslint-config-leankit": "^4.5.0",
         "mocha": "^5.2.0",
-        "nodemon": "^1.17.5",
-        "nyc": "^11.8.0",
+        "nodemon": "^1.18.7",
+        "nyc": "^11.9.0",
         "processhost": "^0.2.1",
-        "proxyquire": "^2.0.1",
-        "seriate": "^0.13.1",
-        "sinon": "^5.0.10",
-        "sinon-chai": "^3.1.0"
+        "proxyquire": "^2.1.0",
+        "sinon": "^5.1.1",
+        "sinon-chai": "^3.3.0"
     },
     "engines": {
         "node": ">=8.0.0"

package.json

@@ -13,6 +13,7 @@ module.exports = {
 		bigInt: true,
 		should: true,
 		getExpected: true,
-		getHiloInstance: true
+		proxyquire: true,
+		expect: true
 	}
 };

spec/.eslintrc.js

@@ -0,0 +1,16 @@
+describe( "HiloGenerationError", () => {
+	let subject;
+
+	beforeEach( () => {
+		const HiloGenerationError = require( "../../src/HiloGenerationError" );
+		subject = new HiloGenerationError( "MESSAGE" );
+	} );
+
+	it( "should set the error message", () => {
+		subject.message.should.eql( "MESSAGE" );
+	} );
+
+	it( "should set the error name", () => {
+		subject.name.should.eql( "HiloGenerationError" );
+	} );
+} );

spec/behavior/HiloGenerationError.spec.js

@@ -0,0 +1,49 @@
+describe( "createTediousConfig", () => {
+	let subject;
+	beforeEach( () => {
+		subject = require( "../../src/createTediousConfig" );
+	} );
+
+	it( "should set all properties", () => {
+		subject( {
+			user: "USER",
+			password: "PASSWORD",
+			server: "SERVER",
+			domain: "DOMAIN",
+			host: "HOST",
+			port: "PORT",
+			database: "DATABASE",
+			connectTimeout: "CONNECTTIMEOUT",
+			requestTimeout: "REQUESTTIMEOUT",
+			encrypt: "ENCRYPT"
+		} ).should.eql( {
+			server: "SERVER",
+			authentication: {
+				type: "ntlm",
+				options: {
+					userName: "USER",
+					password: "PASSWORD",
+					domain: "DOMAIN"
+				}
+			},
+			options: {
+				port: "PORT",
+				database: "DATABASE",
+				connectTimeout: "CONNECTTIMEOUT",
+				requestTimeout: "REQUESTTIMEOUT",
+				encrypt: "ENCRYPT",
+				rowCollectionOnRequestCompletion: true,
+				useColumnNames: true,
+				abortTransactionOnError: true
+			}
+		} );
+	} );
+
+	it( "should set the auth to default when domain is not provided", () => {
+		subject( {} ).authentication.type.should.eql( "default" );
+	} );
+
+	it( "should set the server name from the host when server is not provided", () => {
+		subject( { host: "HOST" } ).server.should.eql( "HOST" );
+	} );
+} );