Merge 516fee2 into 00f21b6

.eslintignore

@@ -1,2 +1,2 @@
 node_modules
-lib/migrationTemplate.js
+lib/migrationTemplates/

.nycrc

@@ -7,7 +7,6 @@
 		"testUtils/**",
 		"tap-snapshots/**",
 		"coverage/**",
-		"migrations/**",
-		"lib/migrationTemplate.js"
+		"migrations/**"
 	]
-}
+}

README.md

@@ -18,13 +18,32 @@ which can be used for dropping index in safe way (contrasting to
 function returns promise and can be used that way instead of providing
 callback.
 
+east mongo package also provides following migration templates:
+
+* lib/migrationTemplates/promises.js - default migration template, uses
+built-in `Promise` in `migrate`, `rollback` functions.
+* lib/migrationTemplates/async.js - migration template that uses async
+functions to describe `migrate`, `rollback` functions.
+
+Default migration template will be used if `template` is not set. To get path
+of another template `require.resolve` could be used, e.g. at `.eastrc`:
+
+```js
+	module.exports = {
+		template: require.resolve('east-mongo/lib/migrationTemplates/async.js')
+	}
+```
 
 [![Npm version](https://img.shields.io/npm/v/east-mongo.svg)](https://www.npmjs.org/package/east-mongo)
 [![Build Status](https://travis-ci.org/okv/east-mongo.svg?branch=master)](https://travis-ci.org/okv/east-mongo)
 [![Coverage Status](https://coveralls.io/repos/github/okv/east-mongo/badge.svg?branch=master)](https://coveralls.io/github/okv/east-mongo?branch=master)
 [![Known Vulnerabilities](https://snyk.io/test/npm/east-mongo/badge.svg)](https://snyk.io/test/npm/east-mongo)
 
 
+## Node.js compatibility
+
+east mongo requires node.js >= 4 to work.
+
 ## Installation
 
 mongodb adapter requires `mongodb` package as peer dependency (versions 2.x and

lib/adapter.js

@@ -18,7 +18,7 @@ function Adapter(params) {
 }
 
 Adapter.prototype.getTemplatePath = function getTemplatePath() {
-	return path.join(__dirname, 'migrationTemplate.js');
+	return path.join(__dirname, 'migrationTemplates', 'promises.js');
 };
 
 // mongodb driver 2.x returns `db`, 3.x returns `client` as a result of

lib/migrationTemplates/async.js

@@ -0,0 +1,6 @@
+
+exports.migrate = async ({db}) => {
+};
+
+exports.rollback = async ({db}) => {
+};

lib/migrationTemplates/promises.js

@@ -0,0 +1,12 @@
+
+exports.migrate = function(params) {
+	const db = params.db;
+
+	return Promise.resolve();
+};
+
+exports.rollback = function(params) {
+	const db = params.db;
+
+	return Promise.resolve();
+};

package.json

@@ -29,6 +29,7 @@
   "devDependencies": {
     "blue-tape": "1.0.0",
     "coveralls": "3.0.9",
+    "es6-promisify": "6.0.2",
     "eslint": "4.19.1",
     "eslint-config-airbnb-base": "12.1.0",
     "eslint-plugin-import": "2.18.2",

test/adapter/getTemplatePath/withSuitableParams.test.js

@@ -22,7 +22,7 @@ test(
 
 		assert.equal(
 			templatePath,
-			'../../../lib/migrationTemplate.js',
+			'../../../lib/migrationTemplates/promises.js',
 			'should return path to migration template'
 		);
 

test/lib/migrationTemplates.test.js

@@ -0,0 +1,68 @@
+'use strict';
+
+const testUtils = require('../utils');
+const pathUtils = require('path');
+const fs = require('fs');
+const promisify = require('es6-promisify').promisify;
+
+const test = testUtils.test;
+
+test('migrationTemplates', (assert) => {
+	const migrationTemplatesPath = pathUtils.resolve(
+		__dirname,
+		'../../lib/migrationTemplates'
+	);
+	const tryMigrationTemplate = (filename) => {
+		return Promise.resolve()
+			// eslint-disable-next-line import/no-dynamic-require
+			.then(() => require(pathUtils.join(migrationTemplatesPath, filename)))
+			.then((migration) => {
+				return Promise.all([
+					migration.migrate({}),
+					migration.rollback({})
+				]);
+			});
+	};
+	const readdir = promisify(fs.readdir);
+	const nodeMajor = Number(process.versions.node.split('.')[0]);
+
+	return Promise.resolve()
+		.then(() => readdir(migrationTemplatesPath))
+		.then((filenames) => {
+			assert.equal(
+				filenames.indexOf('promises.js') !== -1,
+				true,
+				'should contain promises migration template'
+			);
+			assert.equal(
+				filenames.indexOf('async.js') !== -1,
+				true,
+				'should contain promises migration template'
+			);
+			assert.equal(
+				filenames.length,
+				2,
+				'should contain only 2 templates'
+			);
+
+			if (nodeMajor >= 8) {
+				return Promise.all([
+					tryMigrationTemplate('promises.js'),
+					tryMigrationTemplate('async.js')
+				]);
+			} else {
+				return Promise.all([
+					tryMigrationTemplate('promises.js')
+				]);
+			}
+		})
+		.then(() => {
+			if (nodeMajor >= 8) {
+				assert.pass('promises template should be requireble');
+				assert.pass('async template should be requireble');
+			} else {
+				assert.pass('promises template should be requireble');
+				assert.skip('do not require async template on nodejs < 8');
+			}
+		});
+});

test/lib/module.test.js

@@ -7,14 +7,14 @@ const packageJson = require('../../package.json');
 
 const test = testUtils.test;
 
-test('module', (t) => {
-	t.equal(libModule, Adapter, 'should export adapter class');
+test('module', (assert) => {
+	assert.equal(libModule, Adapter, 'should export adapter class');
 
-	t.equal(
+	assert.equal(
 		packageJson.main,
 		'lib/index.js',
 		'should be set as main in package.json'
 	);
 
-	t.end();
+	assert.end();
 });