test: add e2e tests for onBeforeSetupMiddleware option (#3710)

Author: snitin315

test/e2e/__snapshots__/on-before-setup-middleware.test.js.snap.webpack4

@@ -0,0 +1,21 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`onBeforeSetupMiddleware option should handle GET request to /before/some/path route: console messages 1`] = `Array []`;
+
+exports[`onBeforeSetupMiddleware option should handle GET request to /before/some/path route: page errors 1`] = `Array []`;
+
+exports[`onBeforeSetupMiddleware option should handle GET request to /before/some/path route: response headers content-type 1`] = `"text/html; charset=utf-8"`;
+
+exports[`onBeforeSetupMiddleware option should handle GET request to /before/some/path route: response status 1`] = `200`;
+
+exports[`onBeforeSetupMiddleware option should handle GET request to /before/some/path route: response text 1`] = `"before"`;
+
+exports[`onBeforeSetupMiddleware option should handle POST request to /before/some/path route: console messages 1`] = `Array []`;
+
+exports[`onBeforeSetupMiddleware option should handle POST request to /before/some/path route: page errors 1`] = `Array []`;
+
+exports[`onBeforeSetupMiddleware option should handle POST request to /before/some/path route: response headers content-type 1`] = `"text/html; charset=utf-8"`;
+
+exports[`onBeforeSetupMiddleware option should handle POST request to /before/some/path route: response status 1`] = `200`;
+
+exports[`onBeforeSetupMiddleware option should handle POST request to /before/some/path route: response text 1`] = `"brefore POST"`;

test/e2e/__snapshots__/on-before-setup-middleware.test.js.snap.webpack5

@@ -0,0 +1,21 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`onBeforeSetupMiddleware option should handle GET request to /before/some/path route: console messages 1`] = `Array []`;
+
+exports[`onBeforeSetupMiddleware option should handle GET request to /before/some/path route: page errors 1`] = `Array []`;
+
+exports[`onBeforeSetupMiddleware option should handle GET request to /before/some/path route: response headers content-type 1`] = `"text/html; charset=utf-8"`;
+
+exports[`onBeforeSetupMiddleware option should handle GET request to /before/some/path route: response status 1`] = `200`;
+
+exports[`onBeforeSetupMiddleware option should handle GET request to /before/some/path route: response text 1`] = `"before"`;
+
+exports[`onBeforeSetupMiddleware option should handle POST request to /before/some/path route: console messages 1`] = `Array []`;
+
+exports[`onBeforeSetupMiddleware option should handle POST request to /before/some/path route: page errors 1`] = `Array []`;
+
+exports[`onBeforeSetupMiddleware option should handle POST request to /before/some/path route: response headers content-type 1`] = `"text/html; charset=utf-8"`;
+
+exports[`onBeforeSetupMiddleware option should handle POST request to /before/some/path route: response status 1`] = `200`;
+
+exports[`onBeforeSetupMiddleware option should handle POST request to /before/some/path route: response text 1`] = `"brefore POST"`;

test/e2e/on-before-setup-middleware.test.js

@@ -0,0 +1,118 @@
+"use strict";
+
+const webpack = require("webpack");
+const Server = require("../../lib/Server");
+const config = require("../fixtures/client-config/webpack.config");
+const runBrowser = require("../helpers/run-browser");
+const port = require("../ports-map")["on-before-setup-middleware-option"];
+
+describe("onBeforeSetupMiddleware option", () => {
+  let compiler;
+  let server;
+  let page;
+  let browser;
+  let pageErrors;
+  let consoleMessages;
+
+  beforeEach(async () => {
+    compiler = webpack(config);
+    server = new Server(
+      {
+        onBeforeSetupMiddleware: (devServer) => {
+          if (!devServer) {
+            throw new Error("webpack-dev-server is not defined");
+          }
+
+          devServer.app.get("/before/some/path", (_, response) => {
+            response.send("before");
+          });
+
+          devServer.app.post("/before/some/path", (_, response) => {
+            response.send("brefore POST");
+          });
+        },
+        port,
+      },
+      compiler
+    );
+
+    await server.start();
+
+    ({ page, browser } = await runBrowser());
+
+    pageErrors = [];
+    consoleMessages = [];
+  });
+
+  afterEach(async () => {
+    await browser.close();
+    await server.stop();
+  });
+
+  it("should handle GET request to /before/some/path route", async () => {
+    page
+      .on("console", (message) => {
+        consoleMessages.push(message);
+      })
+      .on("pageerror", (error) => {
+        pageErrors.push(error);
+      });
+
+    const response = await page.goto(
+      `http://127.0.0.1:${port}/before/some/path`,
+      {
+        waitUntil: "networkidle0",
+      }
+    );
+
+    expect(response.headers()["content-type"]).toMatchSnapshot(
+      "response headers content-type"
+    );
+
+    expect(response.status()).toMatchSnapshot("response status");
+
+    expect(await response.text()).toMatchSnapshot("response text");
+
+    expect(consoleMessages.map((message) => message.text())).toMatchSnapshot(
+      "console messages"
+    );
+
+    expect(pageErrors).toMatchSnapshot("page errors");
+  });
+
+  it("should handle POST request to /before/some/path route", async () => {
+    await page.setRequestInterception(true);
+
+    page
+      .on("console", (message) => {
+        consoleMessages.push(message);
+      })
+      .on("pageerror", (error) => {
+        pageErrors.push(error);
+      })
+      .on("request", (interceptedRequest) => {
+        interceptedRequest.continue({ method: "POST" });
+      });
+
+    const response = await page.goto(
+      `http://127.0.0.1:${port}/before/some/path`,
+      {
+        waitUntil: "networkidle0",
+      }
+    );
+
+    expect(response.headers()["content-type"]).toMatchSnapshot(
+      "response headers content-type"
+    );
+
+    expect(response.status()).toMatchSnapshot("response status");
+
+    expect(await response.text()).toMatchSnapshot("response text");
+
+    expect(consoleMessages.map((message) => message.text())).toMatchSnapshot(
+      "console messages"
+    );
+
+    expect(pageErrors).toMatchSnapshot("page errors");
+  });
+});