Merge pull request #268 from thefrontside/cl/ldap-server-start

Don't report server started until listener callback is invoked

.changes/ldap-await-server-init.md

@@ -0,0 +1,6 @@
+---
+"@simulacrum/ldap-simulator": patch
+---
+
+Don't resolve `createLDAPServer()` until server is known to be accepting
+connections.

.circleci/config.yml

@@ -1,6 +1,6 @@
 version: 2.1
 orbs:
-  node: circleci/node@4.3.0
+  node: circleci/node@5.1.0
 jobs:
   test:
     docker:

packages/ldap/src/index.ts

@@ -139,15 +139,23 @@ filter: ${req.filter.toString()}
         }
       });
 
-      server.listen(port, function () {
-        logger.log(dedent`LDAP test server running on port ${port});
+      yield new Promise<void>((resolve, reject) => {
+        server.listen(port, function (err: Error) {
+          if (err) {
+            reject(err);
+          } else {
+            resolve();
+          }
+        });
+      });
+
+      logger.log(dedent`LDAP test server running on port ${port}
 
 BindDN: bindDn = ${bindDn} cn=${bindDn},${baseDN}
 Bind Password: ${bindPassword}
 
 UserBaseDN:    ${bindDn}
 `);
-      });
 
       yield spawn(function* shutdown() {
         try {