test: mock console.error for proxy option tests and verify error logging

Author: bjohansebas

test/server/proxy-option.test.js

@@ -139,6 +139,7 @@ const proxyWithRouterAsObject = [
 describe("proxy option", () => {
   let proxyServer1;
   let proxyServer2;
+  let consoleMock;
 
   async function listenProxyServers() {
     const proxyApp1 = express();
@@ -834,16 +835,9 @@ describe("proxy option", () => {
   describe("should work and respect `logger` option", () => {
     let server;
     let req;
-    let customLogProvider;
 
     beforeAll(async () => {
-      customLogProvider = {
-        log: jest.fn(),
-        debug: jest.fn(),
-        info: jest.fn(),
-        warn: jest.fn(),
-        error: jest.fn(),
-      };
+      consoleMock = jest.spyOn(console, "error");
 
       const compiler = webpack([config, config]);
 
@@ -853,7 +847,7 @@ describe("proxy option", () => {
             {
               context: "/my-path",
               target: "http://unknown:1234",
-              logger: customLogProvider,
+              logger: console,
             },
           ],
           port: port3,
@@ -869,6 +863,7 @@ describe("proxy option", () => {
     });
 
     afterAll(async () => {
+      consoleMock.mockRestore();
       await server.stop();
       await closeProxyServers();
     });
@@ -877,24 +872,18 @@ describe("proxy option", () => {
       it("respects a proxy option when a request path is matched", async () => {
         await req.get("/my-path");
 
-        expect(customLogProvider.error).toHaveBeenCalledTimes(1);
+        expect(consoleMock).toHaveBeenCalledTimes(1);
+        expect(consoleMock).toHaveBeenCalledWith("custom error log");
       });
     });
   });
 
   describe("should work and respect the `infrastructureLogging.level` option", () => {
     let server;
     let req;
-    let customLogProvider;
 
     beforeAll(async () => {
-      customLogProvider = {
-        log: jest.fn(),
-        debug: jest.fn(),
-        info: jest.fn(),
-        warn: jest.fn(),
-        error: jest.fn(),
-      };
+      consoleMock = jest.spyOn(console, "error");
 
       const compiler = webpack({
         ...config,
@@ -907,7 +896,6 @@ describe("proxy option", () => {
             {
               context: "/my-path",
               target: "http://unknown:1234",
-              logger: customLogProvider,
             },
           ],
           port: port3,
@@ -923,6 +911,7 @@ describe("proxy option", () => {
     });
 
     afterAll(async () => {
+      consoleMock.mockRestore();
       await server.stop();
       await closeProxyServers();
     });
@@ -931,24 +920,17 @@ describe("proxy option", () => {
       it("respects a proxy option when a request path is matched", async () => {
         await req.get("/my-path");
 
-        expect(customLogProvider.error).toHaveBeenCalledTimes(1);
+        expect(consoleMock).toHaveBeenCalledTimes(1);
       });
     });
   });
 
   describe("should work and respect the `infrastructureLogging.level` option with `none` value", () => {
     let server;
     let req;
-    let customLogProvider;
 
     beforeAll(async () => {
-      customLogProvider = {
-        log: jest.fn(),
-        debug: jest.fn(),
-        info: jest.fn(),
-        warn: jest.fn(),
-        error: jest.fn(),
-      };
+      consoleMock = jest.spyOn(console, "error");
 
       const compiler = webpack({
         ...config,
@@ -961,7 +943,6 @@ describe("proxy option", () => {
             {
               context: "/my-path",
               target: "http://unknown:1234",
-              logger: customLogProvider,
             },
           ],
           port: port3,
@@ -975,14 +956,16 @@ describe("proxy option", () => {
     });
 
     afterAll(async () => {
+      consoleMock.mockRestore();
       await server.stop();
     });
 
     describe("target", () => {
       it("respects a proxy option when a request path is matched", async () => {
         await req.get("/my-path");
 
-        expect(customLogProvider.error).toHaveBeenCalledTimes(0);
+        // Since infrastructureLogging.level is set to "none", no error logs should be produced but http-proxy-middleware still calls the error logger though no show.
+        expect(consoleMock).toHaveBeenCalledTimes(0);
       });
     });
   });