Fix unittests (#43)

* Fix unittest
* Update sailor version to 2.6.18

Author: Olha Virolainen

CHANGELOG.md

@@ -1,3 +1,7 @@
+## 1.2.7 (October 27, 2020)
+* Update sailor version to 2.6.18
+* Fix unit-tests
+
 ## 1.2.6 (October 19, 2020)
 * Update sailor version to 2.6.17
 

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "webhook",
-  "version": "1.2.6",
+  "version": "1.2.7",
   "description": "elastic.io webhook component",
   "scripts": {
     "pretest": "eslint spec lib verifyCredentials.js --fix",
@@ -13,7 +13,7 @@
   "license": "Apache License 2.0",
   "dependencies": {
     "elasticio-node": "0.0.9",
-    "elasticio-sailor-nodejs": "2.6.17",
+    "elasticio-sailor-nodejs": "2.6.18",
     "node-uuid": "1.4.8",
     "request": "2.88.x"
   },

package.json

@@ -15,7 +15,7 @@ describe('Test Webhook', () => {
   const webhookReturnObj = { message: 'ok', other: 'returned' };
   let self;
 
-  it('PUT No Auth', () => {
+  it('PUT No Auth', (done) => {
     const nockObj = nock('http://www.example.com')
       .put('/test', {
         k1: 'v1',
@@ -52,10 +52,11 @@ describe('Test Webhook', () => {
       expect(self.calledTwice).to.be.true;
       expect(self.args[0][1].body).to.eql(webhookReturnObj);
       expect(self.args[1][0]).to.eql('end');
+      done();
     });
   });
 
-  it('PUT Auth', () => {
+  it('PUT Auth', (done) => {
     const nockObj = nock('http://www.example.com')
       .put('/test', {
         k1: 'v1',
@@ -94,10 +95,11 @@ describe('Test Webhook', () => {
       expect(self.calledTwice).to.be.true;
       expect(self.args[0][1].body).to.eql(webhookReturnObj);
       expect(self.args[1][0]).to.eql('end');
+      done();
     });
   });
 
-  it('POST and get text/html response', () => {
+  it('POST and get text/html response', (done) => {
     const nockObj = nock('http://www.example.com')
       .post('/test', {
         k1: 'v1',
@@ -135,10 +137,11 @@ describe('Test Webhook', () => {
         responseBody: '{"message":"ok","other":"returned"}',
       });
       expect(self.args[1][0]).to.eql('end');
+      done();
     });
   });
 
-  it('GET No Auth No QMark', () => {
+  it('GET No Auth No QMark', (done) => {
     const nockObj = nock('http://www.example.com')
       .get('/test?k1=v1&k2=v2')
       .reply(200, webhookReturnObj);
@@ -172,10 +175,11 @@ describe('Test Webhook', () => {
       expect(self.args[0][0]).to.eql('data');
       expect(self.args[0][1].body).to.eql(webhookReturnObj);
       expect(self.args[1][0]).to.eql('end');
+      done();
     });
   });
 
-  it('GET Auth QMark', () => {
+  it('GET Auth QMark', (done) => {
     const nockObj = nock('http://www.example.com')
       .get('/test?k1=v1&k2=v2')
       .matchHeader('X-Api-Secret', 'theSecret')
@@ -213,10 +217,11 @@ describe('Test Webhook', () => {
       expect(self.args[0][0]).to.eql('data');
       expect(self.args[0][1].body).to.eql(webhookReturnObj);
       expect(self.args[1][0]).to.eql('end');
+      done();
     });
   });
 
-  it('404', () => {
+  it('404', (done) => {
     const nockObj = nock('http://www.example.com')
       .get('/test?k1=v1&k2=v2')
       .matchHeader('X-Api-Secret', 'theSecret')
@@ -252,6 +257,7 @@ describe('Test Webhook', () => {
       expect(self.args[0][0]).to.eql('error');
       expect(self.args[0][1].message).to.eql('Endpoint responds with 404');
       expect(self.args[1][0]).to.eql('end');
+      done();
     });
   });