feat: improve error handling within startCallback and endCallback (#3969)

Author: tehraven

lib/Server.js

@@ -2175,7 +2175,9 @@ class Server {
   }
 
   startCallback(callback) {
-    this.start().then(() => callback(null), callback);
+    this.start()
+      .then(() => callback(null), callback)
+      .catch(callback);
   }
 
   async stop() {
@@ -2235,7 +2237,9 @@ class Server {
   }
 
   stopCallback(callback) {
-    this.stop().then(() => callback(null), callback);
+    this.stop()
+      .then(() => callback(null), callback)
+      .catch(callback);
   }
 
   // TODO remove in the next major release

test/e2e/api.test.js

@@ -80,6 +80,29 @@ describe("API", () => {
     });
   });
 
+  it(`should catch errors within startCallback`, async () => {
+    const compiler = webpack(config);
+    const server = new Server(
+      { port, static: "https://absolute-url.com/somewhere" },
+      compiler
+    );
+
+    await new Promise((resolve) => {
+      server.startCallback((err) => {
+        expect(err.message).toEqual(
+          "Using a URL as static.directory is not supported"
+        );
+        resolve();
+      });
+    });
+
+    await new Promise((resolve) => {
+      server.stopCallback(() => {
+        resolve();
+      });
+    });
+  });
+
   it(`should work when using configured manually`, async () => {
     const compiler = webpack({
       ...config,