ldap test: more unbinding

src/test/ldapserver-test.js

@@ -37,28 +37,29 @@ async function ldapSearch(dn, opts) {
     return new Promise((resolve, reject) => {
         const client = ldap.createClient({ url: 'ldap://127.0.0.1:' + constants.LDAP_PORT });
 
+        function done(error, result) {
+            client.unbind();
+            if (error) reject(error); else resolve(result);
+        }
+
         client.search(dn, opts, function (error, result) {
-            if (error) return reject(error);
+            if (error) return done(error);
 
             let entries = [];
 
             result.on('searchEntry', function (entry) { entries.push(entry.object); });
 
-            result.on('error', function (error) {
-                client.unbind();
-                reject(error);
-            });
+            result.on('error', done);
 
             result.on('end', function (result) {
-                if (result.status !== 0) return reject(new Error(`Unexpected status: ${result.status}`));
-                resolve(entries);
+                if (result.status !== 0) return done(new Error(`Unexpected status: ${result.status}`));
+                done(null, entries);
             });
         });
     });
-
 }
 
-describe('Ldap', function () {
+describe('Ldap Server', function () {
     const { setup, cleanup, admin, user, app, domain, auditSource } = common;
     let group, group2;
     const mockApp = Object.assign({}, app);