test: disable to output logs (#2109)

Author: hiroppy

test/client/clients/SockJSClient.test.js

@@ -7,10 +7,13 @@ const SockJSClient = require('../../../client-src/clients/SockJSClient');
 const port = require('../../ports-map').sockJSClient;
 
 describe('SockJSClient', () => {
+  let consoleMock;
   let socketServer;
   let listeningApp;
 
   beforeAll((done) => {
+    consoleMock = jest.spyOn(console, 'log').mockImplementation();
+
     // eslint-disable-next-line new-cap
     const app = new express();
 
@@ -24,6 +27,10 @@ describe('SockJSClient', () => {
     });
   });
 
+  afterAll(() => {
+    consoleMock.mockRestore();
+  });
+
   describe('client', () => {
     it('should open, receive message, and close', (done) => {
       socketServer.on('connection', (connection) => {

test/e2e/ClientOptions.test.js

@@ -33,6 +33,7 @@ describe('Client code', () => {
       watchOptions: {
         poll: true,
       },
+      quiet: true,
     };
     testServer.startAwaitingCompilation(config, options, done);
   });
@@ -95,6 +96,7 @@ describe('Client complex inline script path', () => {
       },
       public: 'myhost.test',
       sockPath: '/foo/test/bar/',
+      quiet: true,
     };
     testServer.startAwaitingCompilation(config, options, done);
   });
@@ -135,6 +137,7 @@ describe('Client complex inline script path with sockPort', () => {
       },
       sockPath: '/foo/test/bar/',
       sockPort: port3,
+      quiet: true,
     };
     testServer.startAwaitingCompilation(config, options, done);
   });
@@ -178,6 +181,7 @@ describe('Client complex inline script path with sockPort, no sockPath', () => {
         poll: true,
       },
       sockPort: port3,
+      quiet: true,
     };
     testServer.startAwaitingCompilation(config, options, done);
   });
@@ -215,6 +219,7 @@ describe('Client complex inline script path with sockHost', () => {
         poll: true,
       },
       sockHost: 'myhost.test',
+      quiet: true,
     };
     testServer.startAwaitingCompilation(config, options, done);
   });
@@ -246,6 +251,7 @@ describe('Client console.log', () => {
   const baseOptions = {
     port: port2,
     host: '0.0.0.0',
+    quiet: true,
   };
   const cases = [
     {

test/e2e/Progress.test.js

@@ -5,20 +5,31 @@
 */
 const fs = require('fs');
 const { resolve } = require('path');
-const testServer = require('../helpers/test-server');
 const reloadConfig = require('../fixtures/reload-config/webpack.config');
 const runBrowser = require('../helpers/run-browser');
 const port = require('../ports-map').Progress;
 
 const cssFilePath = resolve(__dirname, '../fixtures/reload-config/main.css');
 
 describe('client progress', () => {
+  let testServer;
+  let processStderrMock;
+
   describe('using hot', () => {
     beforeAll((done) => {
+      // ProgressPlugin uses process.stderr and reset webpack
+      jest.resetModules();
+      processStderrMock = jest
+        .spyOn(process.stderr, 'write')
+        .mockImplementation();
+      // eslint-disable-next-line global-require
+      testServer = require('../helpers/test-server');
+
       fs.writeFileSync(
         cssFilePath,
         'body { background-color: rgb(0, 0, 255); }'
       );
+
       const options = {
         port,
         host: '0.0.0.0',
@@ -29,12 +40,14 @@ describe('client progress', () => {
           poll: 500,
         },
       };
+
       testServer.startAwaitingCompilation(reloadConfig, options, done);
     });
 
     afterAll((done) => {
       fs.unlinkSync(cssFilePath);
       testServer.close(done);
+      processStderrMock.mockRestore();
     });
 
     describe('on browser client', () => {

test/helpers/test-server.js

@@ -3,6 +3,7 @@
 /* eslint-disable
   no-undefined
 */
+
 const webpack = require('webpack');
 const Server = require('../../lib/Server');
 

test/options.test.js

@@ -12,6 +12,16 @@ const config = require('./fixtures/simple-config/webpack.config');
 describe('options', () => {
   jest.setTimeout(20000);
 
+  let consoleMock;
+
+  beforeAll(() => {
+    consoleMock = jest.spyOn(console, 'warn').mockImplementation();
+  });
+
+  afterAll(() => {
+    consoleMock.mockRestore();
+  });
+
   it('should match properties and errorMessage', () => {
     const properties = Object.keys(options.properties);
     const messages = Object.keys(options.errorMessage.properties);

test/server/Server.test.js

@@ -10,6 +10,11 @@ const port = require('../ports-map').Server;
 
 jest.mock('sockjs/lib/transport');
 
+const baseDevConfig = {
+  port,
+  quiet: true,
+};
+
 describe('Server', () => {
   describe('sockjs', () => {
     it('add decorateConnection', () => {
@@ -22,10 +27,12 @@ describe('Server', () => {
   describe('addEntries', () => {
     it('add hot option', (done) => {
       const compiler = webpack(config);
-      const server = new Server(compiler, {
-        hot: true,
-        port,
-      });
+      const server = new Server(
+        compiler,
+        Object.assign({}, baseDevConfig, {
+          hot: true,
+        })
+      );
 
       expect(
         server.middleware.context.compiler.options.entry.map((p) => {
@@ -45,10 +52,12 @@ describe('Server', () => {
 
     it('add hotOnly option', (done) => {
       const compiler = webpack(config);
-      const server = new Server(compiler, {
-        hotOnly: true,
-        port,
-      });
+      const server = new Server(
+        compiler,
+        Object.assign({}, baseDevConfig, {
+          hotOnly: true,
+        })
+      );
 
       expect(
         server.middleware.context.compiler.options.entry.map((p) => {
@@ -95,7 +104,7 @@ describe('Server', () => {
       });
 
       const compiler = webpack(config);
-      const server = new Server(compiler, { port });
+      const server = new Server(compiler, baseDevConfig);
 
       compiler.hooks.done.tap('webpack-dev-server', (s) => {
         const output = server.getStats(s);
@@ -139,7 +148,7 @@ describe('Server', () => {
     describe('Testing callback functions on calling invalidate without callback', () => {
       it('should be `noop` (the default callback function)', (done) => {
         const compiler = webpack(config);
-        const server = new Server(compiler, { port });
+        const server = new Server(compiler, baseDevConfig);
 
         server.invalidate();
         expect(server.middleware.context.callbacks[0]).toBe(noop);
@@ -156,7 +165,7 @@ describe('Server', () => {
       it('should be `callback` function', (done) => {
         const compiler = webpack(config);
         const callback = jest.fn();
-        const server = new Server(compiler, { port });
+        const server = new Server(compiler, baseDevConfig);
 
         server.invalidate(callback);
 

test/server/open-option.test.js

@@ -20,6 +20,7 @@ describe('open option', () => {
     const server = new Server(compiler, {
       open: true,
       port,
+      quiet: true,
     });
 
     compiler.hooks.done.tap('webpack-dev-server', () => {

test/server/profile-option.test.js

@@ -22,6 +22,7 @@ describe('profile', () => {
         // profile will only have an effect when progress is enabled
         progress: true,
         profile: true,
+        quiet: true,
       });
 
       compiler.hooks.done.tap('webpack-dev-server', () => {

test/server/progress-option.test.js

@@ -20,6 +20,7 @@ describe('progress', () => {
       const server = new Server(compiler, {
         port,
         progress: true,
+        quiet: true,
       });
 
       compiler.hooks.done.tap('webpack-dev-server', () => {

test/server/serverMode-option.test.js

@@ -17,6 +17,7 @@ describe('serverMode option', () => {
   let server;
   let req;
   let getSocketServerImplementation;
+  let consoleMock;
 
   const serverModes = [
     {
@@ -45,6 +46,14 @@ describe('serverMode option', () => {
     },
   ];
 
+  beforeAll(() => {
+    consoleMock = jest.spyOn(console, 'log').mockImplementation();
+  });
+
+  afterAll(() => {
+    consoleMock.mockRestore();
+  });
+
   describe('is passed to getSocketServerImplementation correctly', () => {
     beforeEach(() => {
       jest.mock('../../lib/utils/getSocketServerImplementation');