fix(Server): return socket processing to bin

Author: hiroppy

bin/webpack-dev-server.js

@@ -192,7 +192,19 @@ function startDevServer(config, options) {
       }
     });
 
-    runServer();
+    server.listen(options.socket, options.host, (err) => {
+      if (err) {
+        throw err;
+      }
+      // chmod 666 (rw rw rw)
+      const READ_WRITE = 438;
+
+      fs.chmod(options.socket, READ_WRITE, (err) => {
+        if (err) {
+          throw err;
+        }
+      });
+    });
   } else if (options.port) {
     runServer();
   } else {

lib/Server.js

@@ -741,6 +741,21 @@ class Server {
     return false;
   }
 
+  showStatus() {
+    const suffix =
+      this.options.inline !== false || this.options.lazy === true
+        ? '/'
+        : '/webpack-dev-server/';
+    const uri = `${createDomain(this.options, this.listeningApp)}${suffix}`;
+
+    status(
+      uri,
+      this.options,
+      this.log,
+      this.options.stats && this.options.stats.colors
+    );
+  }
+
   // delegate listen call and init sockjs
   listen(port, hostname, fn) {
     this.hostname = hostname;
@@ -816,36 +831,7 @@ class Server {
         runBonjour(this.options);
       }
 
-      const showStatus = () => {
-        const suffix =
-          this.options.inline !== false || this.options.lazy === true
-            ? '/'
-            : '/webpack-dev-server/';
-
-        const uri = `${createDomain(this.options, this.listeningApp)}${suffix}`;
-
-        status(
-          uri,
-          this.options,
-          this.log,
-          this.options.stats && this.options.stats.colors
-        );
-      };
-
-      if (this.options.socket) {
-        // chmod 666 (rw rw rw)
-        const READ_WRITE = 438;
-
-        fs.chmod(this.options.socket, READ_WRITE, (err) => {
-          if (err) {
-            throw err;
-          }
-
-          showStatus();
-        });
-      } else {
-        showStatus();
-      }
+      this.showStatus();
 
       if (fn) {
         fn.call(this.listeningApp, err);

test/cli.test.js

@@ -3,6 +3,7 @@
 /* eslint-disable
   array-bracket-spacing,
 */
+const { unlink } = require('fs');
 const path = require('path');
 const execa = require('execa');
 const runDevServer = require('./helpers/run-webpack-dev-server');
@@ -21,7 +22,7 @@ describe('CLI', () => {
     runDevServer('--progress')
       .then((output) => {
         expect(output.code).toEqual(0);
-        expect(output.stderr.indexOf('0% compiling') >= 0).toBe(true);
+        expect(output.stderr.includes('0% compiling')).toBe(true);
         done();
       })
       .catch(done);
@@ -31,7 +32,7 @@ describe('CLI', () => {
     runDevServer('--bonjour')
       .then((output) => {
         expect(output.code).toEqual(0);
-        expect(output.stdout.indexOf('Bonjour') >= 0).toBe(true);
+        expect(output.stdout.includes('Bonjour')).toBe(true);
         done();
       })
       .catch(done);
@@ -41,7 +42,7 @@ describe('CLI', () => {
     runDevServer('--https')
       .then((output) => {
         expect(output.code).toEqual(0);
-        expect(output.stdout.indexOf('Project is running at') >= 0).toBe(true);
+        expect(output.stdout.includes('Project is running at')).toBe(true);
         done();
       })
       .catch(done);
@@ -53,7 +54,7 @@ describe('CLI', () => {
     )
       .then((output) => {
         expect(output.code).toEqual(0);
-        expect(output.stdout.indexOf('Project is running at') >= 0).toBe(true);
+        expect(output.stdout.includes('Project is running at')).toBe(true);
         done();
       })
       .catch(done);
@@ -82,6 +83,20 @@ describe('CLI', () => {
       .catch(done);
   });
 
+  it('--socket', (done) => {
+    const socketPath = './webpack.sock';
+
+    runDevServer(`--socket ${socketPath}`)
+      .then((output) => {
+        expect(output.code).toEqual(0);
+        expect(output.stdout.includes(socketPath)).toBe(true);
+        unlink(socketPath, () => {
+          done();
+        });
+      })
+      .catch(done);
+  });
+
   it('should exit the process when SIGINT is detected', (done) => {
     const cliPath = path.resolve(__dirname, '../bin/webpack-dev-server.js');
     const examplePath = path.resolve(__dirname, '../examples/cli/public');