Merge pull request #242 from DavidVujic/bump-deps

Update dependencies causing audit warnings on install

Author: DavidVujic

examples/index.js

@@ -7,10 +7,10 @@ const { addTask } = require('./addtask.js');
 const logger = require('./logger.js');
 const notifier = require('./notifier.js');
 
-notifier.on('connect', message => logger.log('connect', message));
-notifier.on('createNode', message => logger.log('createNode', message));
-notifier.on('addTask', message => logger.log('addTask', message));
-notifier.on('close', message => logger.log('close', message));
+notifier.on('connect', (message) => logger.log('connect', message));
+notifier.on('createNode', (message) => logger.log('createNode', message));
+notifier.on('addTask', (message) => logger.log('addTask', message));
+notifier.on('close', (message) => logger.log('close', message));
 
 notifier.on('onChildren', (children) => {
     children.forEach((child) => {

lib/constants.js

@@ -1,4 +1,4 @@
-const NativeZk = require('./../build/zookeeper.node').ZooKeeper;
+const NativeZk = require('../build/zookeeper.node').ZooKeeper;
 
 /** @returns {number} 1 */
 module.exports.ZOO_PERM_READ = NativeZk.ZOO_PERM_READ;

lib/promise.js

@@ -12,23 +12,23 @@ class ZkPromise extends Promise {
      */
     get(propertyName) {
         deprecationLog(ZkPromise.name, 'get');
-        return this.then(object => object[propertyName]);
+        return this.then((object) => object[propertyName]);
     }
 
     /**
      * @deprecated
      */
     put(propertyName, value) {
         deprecationLog(ZkPromise.name, 'put');
-        return this.then(object => object[propertyName] = value);
+        return this.then((object) => object[propertyName] = value);
     }
 
     /**
      * @deprecated
      */
     call(functionName, ...args) {
         deprecationLog(ZkPromise.name, 'call');
-        return this.then(object => object[functionName](...args));
+        return this.then((object) => object[functionName](...args));
     }
 
     /**

lib/zookeeper.js

@@ -6,7 +6,7 @@ const _ = require('lodash');
 const { normalize } = require('path');
 const util = require('util');
 
-const NativeZk = require('./../build/zookeeper.node').ZooKeeper;
+const NativeZk = require('../build/zookeeper.node').ZooKeeper;
 
 const Async = {
     apply,

package.json

@@ -27,22 +27,22 @@
         "client"
     ],
     "dependencies": {
-        "async": "2.6.x",
+        "async": "3.2.x",
         "decompress": "4.2.x",
         "decompress-targz": "4.1.x",
         "lodash": "4.x",
         "nan": "2.x",
         "shelljs": "0.8.x"
     },
     "devDependencies": {
-        "eslint": "5.x",
-        "eslint-config-airbnb-base": "13.x",
+        "eslint": "7.x",
+        "eslint-config-airbnb-base": "14.x",
         "eslint-plugin-import": "2.x",
-        "log4js": "4.x",
+        "log4js": "6.x",
         "proxyquire": "2.x",
-        "sinon": "7.x",
+        "sinon": "9.x",
         "tap-spec": "5.x",
-        "tape": "4.x",
+        "tape": "5.x",
         "webworker": "0.x"
     },
     "main": "lib/index",

tests/unit/util/promisetest.js

@@ -42,7 +42,7 @@ test('ZkPromise call', (t) => {
 test('ZkPromise addCallback', (t) => {
     t.plan(3);
 
-    const callback = sinon.stub().callsFake(value => value * 2);
+    const callback = sinon.stub().callsFake((value) => value * 2);
     const promise = ZkPromise.resolve(10);
 
     promise.addCallback(callback).then((actual) => {
@@ -55,7 +55,7 @@ test('ZkPromise addCallback', (t) => {
 test('ZkPromise addErrback', (t) => {
     t.plan(3);
 
-    const callback = sinon.stub().callsFake(value => value * 2);
+    const callback = sinon.stub().callsFake((value) => value * 2);
     const promise = ZkPromise.reject(10);
 
     promise.addErrback(callback).then((actual) => {
@@ -68,12 +68,12 @@ test('ZkPromise addErrback', (t) => {
 test('ZkPromise addBoth', (t) => {
     t.plan(7);
 
-    const callback = sinon.stub().callsFake(value => value * 2);
+    const callback = sinon.stub().callsFake((value) => value * 2);
     const errback = sinon.stub().callsFake(() => {
         throw new Error('errback!');
     });
     // eslint-disable-next-line max-len
-    const promise = succeeds => new ZkPromise((resolve, reject) => (succeeds ? resolve(10) : reject(10)));
+    const promise = (succeeds) => new ZkPromise((resolve, reject) => (succeeds ? resolve(10) : reject(10)));
 
     promise(true).addBoth(callback)
         .then((actual) => {
@@ -93,12 +93,12 @@ test('ZkPromise addBoth', (t) => {
 test('ZkPromise addCallbacks', (t) => {
     t.plan(9);
 
-    const callback = sinon.stub().callsFake(value => value * 2);
+    const callback = sinon.stub().callsFake((value) => value * 2);
     const errback = sinon.stub().callsFake(() => {
         throw new Error('errback!');
     });
     // eslint-disable-next-line max-len
-    const promise = succeeds => new ZkPromise((resolve, reject) => (succeeds ? resolve(10) : reject(10)));
+    const promise = (succeeds) => new ZkPromise((resolve, reject) => (succeeds ? resolve(10) : reject(10)));
 
     promise(true).addCallbacks(callback, errback)
         .then((actual) => {

tests/unit/zookeeper/nativeobjecttest.js

@@ -2,7 +2,6 @@ const test = require('tape');
 const NativeZk = require('../../../build/zookeeper.node').ZooKeeper;
 const ZooKeeper = require('../../../lib/zookeeper');
 
-
 test('ZooKeeper instance creates an instance of the native object', (t) => {
     t.plan(1);