Start server before bundling

Fixes logging issue

Author: devongovett

src/Bundler.js

@@ -459,8 +459,9 @@ class Bundler extends EventEmitter {
   }
 
   async serve(port = 1234) {
+    let server = await Server.serve(this, port);
     this.bundle();
-    return await Server.serve(this, port);
+    return server;
   }
 }
 

src/Server.js

@@ -60,25 +60,28 @@ async function serve(bundler, port) {
   let freePort = await getPort({port});
   let server = http.createServer(middleware(bundler)).listen(freePort);
 
-  server.on('error', err => {
-    bundler.logger.error(new Error(serverErrors(err, server.address().port)));
-  });
+  return new Promise((resolve, reject) => {
+    server.on('error', err => {
+      bundler.logger.error(new Error(serverErrors(err, server.address().port)));
+      reject(err);
+    });
 
-  server.once('listening', connection => {
-    let addon =
-      server.address().port !== port
-        ? `- ${bundler.logger.chalk.red(
-            `configured port ${port} could not be used.`
-          )}`
-        : '';
-    bundler.logger.persistent(
-      `Server running at ${bundler.logger.chalk.cyan(
-        `http://localhost:${server.address().port}`
-      )} ${addon}\n`
-    );
-  });
+    server.once('listening', connection => {
+      let addon =
+        server.address().port !== port
+          ? `- ${bundler.logger.chalk.red(
+              `configured port ${port} could not be used.`
+            )}`
+          : '';
+      bundler.logger.persistent(
+        `Server running at ${bundler.logger.chalk.cyan(
+          `http://localhost:${server.address().port}`
+        )} ${addon}`
+      );
 
-  return server;
+      resolve(server);
+    });
+  });
 }
 
 exports.middleware = middleware;