From 44acc5cb004b63b166e7d423714b39b6f331179a Mon Sep 17 00:00:00 2001
From: Pavel Djundik <github@xpaw.me>
Date: Fri, 6 Oct 2017 12:53:08 +0300
Subject: [PATCH] Teardown sockets in tests

---
 src/plugins/auth/ldap.js  |  1 +
 src/server.js             |  2 ++
 test/plugins/auth/ldap.js | 14 ++++++--------
 test/server.js            | 11 +++++++++--
 4 files changed, 18 insertions(+), 10 deletions(-)

diff --git a/src/plugins/auth/ldap.js b/src/plugins/auth/ldap.js
index 059d23ce..54d8f472 100644
--- a/src/plugins/auth/ldap.js
+++ b/src/plugins/auth/ldap.js
@@ -91,6 +91,7 @@ function advancedLdapAuth(user, password, callback) {
 						callback(false);
 					});
 					res.on("end", function() {
+						ldapclient.unbind();
 						if (!found) {
 							callback(false);
 						}
diff --git a/src/server.js b/src/server.js
index de8c90e5..82982e66 100644
--- a/src/server.js
+++ b/src/server.js
@@ -177,6 +177,8 @@ module.exports = function() {
 		process.on("SIGINT", exitGracefully);
 		process.on("SIGTERM", exitGracefully);
 	});
+
+	return server;
 };
 
 function getClientIp(request) {
diff --git a/test/plugins/auth/ldap.js b/test/plugins/auth/ldap.js
index 0a4917fc..f2dcb180 100644
--- a/test/plugins/auth/ldap.js
+++ b/test/plugins/auth/ldap.js
@@ -109,29 +109,28 @@ function testLdapAuth() {
 }
 
 describe("LDAP authentication plugin", function() {
-	before(function(done) {
+	before((done) => {
 		this.server = startLdapServer(done);
 	});
-	after(function(done) {
+
+	after(() => {
 		this.server.close();
-		done();
 	});
 
-	beforeEach(function(done) {
+	beforeEach(() => {
 		Helper.config.public = false;
 		Helper.config.ldap.enable = true;
 		Helper.config.ldap.url = "ldap://localhost:" + String(serverPort);
 		Helper.config.ldap.primaryKey = primaryKey;
-		done();
 	});
 
 	describe("LDAP authentication availability", function() {
-		it("checks that the configuration is correctly tied to isEnabled()", function(done) {
+		it("checks that the configuration is correctly tied to isEnabled()", function() {
 			Helper.config.ldap.enable = true;
 			expect(ldapAuth.isEnabled()).to.equal(true);
+
 			Helper.config.ldap.enable = false;
 			expect(ldapAuth.isEnabled()).to.equal(false);
-			done();
 		});
 	});
 
@@ -145,4 +144,3 @@ describe("LDAP authentication plugin", function() {
 		testLdapAuth();
 	});
 });
-
diff --git a/test/server.js b/test/server.js
index ec2a31a8..bbdb8674 100644
--- a/test/server.js
+++ b/test/server.js
@@ -6,8 +6,15 @@ const request = require("request");
 const io = require("socket.io-client");
 
 describe("Server", () => {
-	const server = require("../src/server");
-	server();
+	let server;
+
+	before(() => {
+		server = require("../src/server")();
+	});
+
+	after((done) => {
+		server.close(done);
+	});
 
 	const webURL = `http://${Helper.config.host}:${Helper.config.port}/`;