feat: update bonjour.test.js

mahdikhashan · mahdikhashan · commit ab7cf84eda03 · 2024-07-29T01:02:44.000+02:00
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-object-should-apply-bonjour-options-console-messages-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-object-should-apply-bonjour-options-console-messages-snap.webpack5
@@ -0,0 +1 @@
+["[webpack-dev-server] Server started: Hot Module Replacement enabled, Live Reloading enabled, Progress disabled, Overlay enabled.","[HMR] Waiting for update signal from WDS...","Hey."]
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-object-should-apply-bonjour-options-page-errors-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-object-should-apply-bonjour-options-page-errors-snap.webpack5
@@ -0,0 +1 @@
+[]
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-object-should-apply-bonjour-options-response-status-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-object-should-apply-bonjour-options-response-status-snap.webpack5
@@ -0,0 +1 @@
+200
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-object-should-apply-bonjour-options-undefined-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-object-should-apply-bonjour-options-undefined-snap.webpack5
@@ -0,0 +1 @@
+200
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-true-should-call-bonjour-with-correct-params-console-messages-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-true-should-call-bonjour-with-correct-params-console-messages-snap.webpack5
@@ -0,0 +1 @@
+["[webpack-dev-server] Server started: Hot Module Replacement enabled, Live Reloading enabled, Progress disabled, Overlay enabled.","[HMR] Waiting for update signal from WDS...","Hey."]
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-true-should-call-bonjour-with-correct-params-page-errors-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-true-should-call-bonjour-with-correct-params-page-errors-snap.webpack5
@@ -0,0 +1 @@
+[]
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-true-should-call-bonjour-with-correct-params-response-status-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-true-should-call-bonjour-with-correct-params-response-status-snap.webpack5
@@ -0,0 +1 @@
+200
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-true-should-call-bonjour-with-correct-params-undefined-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-as-true-should-call-bonjour-with-correct-params-undefined-snap.webpack5
@@ -0,0 +1 @@
+200
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-bonjour-object-and-server-option-should-apply-bonjour-options-console-messages-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-bonjour-object-and-server-option-should-apply-bonjour-options-console-messages-snap.webpack5
@@ -0,0 +1 @@
+["[webpack-dev-server] Server started: Hot Module Replacement enabled, Live Reloading enabled, Progress disabled, Overlay enabled.","[HMR] Waiting for update signal from WDS...","Hey."]
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-bonjour-object-and-server-option-should-apply-bonjour-options-page-errors-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-bonjour-object-and-server-option-should-apply-bonjour-options-page-errors-snap.webpack5
@@ -0,0 +1 @@
+[]
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-bonjour-object-and-server-option-should-apply-bonjour-options-response-status-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-bonjour-object-and-server-option-should-apply-bonjour-options-response-status-snap.webpack5
@@ -0,0 +1 @@
+200
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-bonjour-object-and-server-option-should-apply-bonjour-options-undefined-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-bonjour-object-and-server-option-should-apply-bonjour-options-undefined-snap.webpack5
@@ -0,0 +1 @@
+200
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-with-server-option-should-call-bonjour-with-https-type-console-messages-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-with-server-option-should-call-bonjour-with-https-type-console-messages-snap.webpack5
@@ -0,0 +1 @@
+["[webpack-dev-server] Server started: Hot Module Replacement enabled, Live Reloading enabled, Progress disabled, Overlay enabled.","[HMR] Waiting for update signal from WDS...","Hey."]
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-with-server-option-should-call-bonjour-with-https-type-page-errors-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-with-server-option-should-call-bonjour-with-https-type-page-errors-snap.webpack5
@@ -0,0 +1 @@
+[]
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-with-server-option-should-call-bonjour-with-https-type-response-status-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-with-server-option-should-call-bonjour-with-https-type-response-status-snap.webpack5
@@ -0,0 +1 @@
+200
diff --git a/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-with-server-option-should-call-bonjour-with-https-type-undefined-snap.webpack5 b/test/e2e/__snapshots__/bonjour.test.js/bonjour-option-with-server-option-should-call-bonjour-with-https-type-undefined-snap.webpack5
@@ -0,0 +1 @@
+200
diff --git a/test/e2e/bonjour.test.js b/test/e2e/bonjour.test.js
@@ -2,46 +2,46 @@
 
 const os = require("os");
 const webpack = require("webpack");
-const { describe, test, beforeEach, beforeAll, afterEach } = require("@playwright/test");
 const sinon = require("sinon");
 const bonjourService = require("bonjour-service");
+const { test } = require("../helpers/playwright-test");
 const { expect } = require("../helpers/playwright-custom-expects");
 const Server = require("../../lib/Server");
 const config = require("../fixtures/simple-config/webpack.config");
 const port = require("../ports-map").bonjour;
 
-describe("bonjour option", { tag: "@flaky" }, () => {
+test.describe("bonjour option", { tag: "@flaky" }, () => {
   let mockPublish;
   let mockUnpublishAll;
   let mockDestroy;
 
-  beforeAll(() => {
+  test.beforeAll(() => {
     mockPublish = sinon.stub();
     mockUnpublishAll = sinon.stub().callsFake((callback) => {
       callback();
     });
     mockDestroy = sinon.stub();
 
-    sinon.stub(bonjourService, 'Bonjour').returns({
+    sinon.stub(bonjourService, "Bonjour").returns({
       publish: mockPublish,
       unpublishAll: mockUnpublishAll,
       destroy: mockDestroy,
-    })
+    });
   });
 
-  afterEach(() => {
+  test.afterEach(() => {
     mockPublish.resetHistory();
     mockUnpublishAll.resetHistory();
     mockDestroy.resetHistory();
-  })
+  });
 
-  describe("as true", () => {
+  test.describe("as true", () => {
     let compiler;
     let server;
     let pageErrors;
     let consoleMessages;
 
-    beforeEach(async () => {
+    test.beforeEach(async () => {
       compiler = webpack(config);
 
       server = new Server({ port, bonjour: true }, compiler);
@@ -52,7 +52,7 @@ describe("bonjour option", { tag: "@flaky" }, () => {
       consoleMessages = [];
     });
 
-    afterEach(async () => {
+    test.afterEach(async () => {
       await server.stop();
     });
 
@@ -71,35 +71,35 @@ describe("bonjour option", { tag: "@flaky" }, () => {
 
       expect(mockPublish.callCount).toBe(1);
 
-      expect(mockPublish.calledWith(
-        {
+      expect(
+        mockPublish.calledWith({
           name: `Webpack Dev Server ${os.hostname()}:${port}`,
           port,
           type: "http",
           subtypes: ["webpack"],
-        }
-      )).toBeTruthy();
+        }),
+      ).toBeTruthy();
 
       expect(mockUnpublishAll.callCount).toBe(0);
       expect(mockDestroy.callCount).toBe(0);
 
-      expect(response.status()).toMatchSnapshotWithArray();
+      expect(response.status()).toMatchSnapshotWithArray("response status");
 
       expect(
-        consoleMessages.map((message) => message.text()))
-      .toMatchSnapshotWithArray();
+        consoleMessages.map((message) => message.text()),
+      ).toMatchSnapshotWithArray("console messages");
 
-      expect(pageErrors).toMatchSnapshotWithArray();
+      expect(pageErrors).toMatchSnapshotWithArray("page errors");
     });
   });
 
-  describe("with 'server' option", () => {
+  test.describe("with 'server' option", () => {
     let compiler;
     let server;
     let pageErrors;
     let consoleMessages;
 
-    beforeEach(async () => {
+    test.beforeEach(async () => {
       compiler = webpack(config);
 
       server = new Server({ bonjour: true, port, server: "https" }, compiler);
@@ -110,7 +110,7 @@ describe("bonjour option", { tag: "@flaky" }, () => {
       consoleMessages = [];
     });
 
-    afterEach(async () => {
+    test.afterEach(async () => {
       await server.stop();
     });
 
@@ -129,33 +129,35 @@ describe("bonjour option", { tag: "@flaky" }, () => {
 
       expect(mockPublish.callCount).toBe(1);
 
-      expect(mockPublish.calledWith({
-        name: `Webpack Dev Server ${os.hostname()}:${port}`,
-        port,
-        type: "https",
-        subtypes: ["webpack"],
-      })).toBeTruthy();
+      expect(
+        mockPublish.calledWith({
+          name: `Webpack Dev Server ${os.hostname()}:${port}`,
+          port,
+          type: "https",
+          subtypes: ["webpack"],
+        }),
+      ).toBeTruthy();
 
       expect(mockUnpublishAll.callCount).toBe(0);
       expect(mockDestroy.callCount).toBe(0);
 
-      expect(response.status()).toMatchSnapshotWithArray();
+      expect(response.status()).toMatchSnapshotWithArray("response status");
 
       expect(
-        consoleMessages.map((message) => message.text()))
-      .toMatchSnapshotWithArray();
+        consoleMessages.map((message) => message.text()),
+      ).toMatchSnapshotWithArray("console messages");
 
-      expect(pageErrors).toMatchSnapshotWithArray();
+      expect(pageErrors).toMatchSnapshotWithArray("page errors");
     });
   });
 
-  describe("as object", () => {
+  test.describe("as object", () => {
     let compiler;
     let server;
     let pageErrors;
     let consoleMessages;
 
-    beforeEach(async () => {
+    test.beforeEach(async () => {
       compiler = webpack(config);
 
       server = new Server(
@@ -175,7 +177,7 @@ describe("bonjour option", { tag: "@flaky" }, () => {
       consoleMessages = [];
     });
 
-    afterEach(async () => {
+    test.afterEach(async () => {
       await server.stop();
     });
 
@@ -194,34 +196,36 @@ describe("bonjour option", { tag: "@flaky" }, () => {
 
       expect(mockPublish.callCount).toBe(1);
 
-      expect(mockPublish.calledWith({
-        name: `Webpack Dev Server ${os.hostname()}:${port}`,
-        port,
-        type: "https",
-        protocol: "udp",
-        subtypes: ["webpack"],
-      })).toBeTruthy();
+      expect(
+        mockPublish.calledWith({
+          name: `Webpack Dev Server ${os.hostname()}:${port}`,
+          port,
+          type: "https",
+          protocol: "udp",
+          subtypes: ["webpack"],
+        }),
+      ).toBeTruthy();
 
       expect(mockUnpublishAll.callCount).toBe(0);
       expect(mockDestroy.callCount).toBe(0);
 
-      expect(response.status()).toMatchSnapshotWithArray();
+      expect(response.status()).toMatchSnapshotWithArray("response status");
 
       expect(
-        consoleMessages.map((message) => message.text()))
-      .toMatchSnapshotWithArray();
+        consoleMessages.map((message) => message.text()),
+      ).toMatchSnapshotWithArray("console messages");
 
-      expect(pageErrors).toMatchSnapshotWithArray();
+      expect(pageErrors).toMatchSnapshotWithArray("page errors");
     });
   });
 
-  describe("bonjour object and 'server' option", () => {
+  test.describe("bonjour object and 'server' option", () => {
     let compiler;
     let server;
     let pageErrors;
     let consoleMessages;
 
-    beforeEach(async () => {
+    test.beforeEach(async () => {
       compiler = webpack(config);
 
       server = new Server(
@@ -244,7 +248,7 @@ describe("bonjour option", { tag: "@flaky" }, () => {
       consoleMessages = [];
     });
 
-    afterEach(async () => {
+    test.afterEach(async () => {
       await server.stop();
     });
 
@@ -263,24 +267,26 @@ describe("bonjour option", { tag: "@flaky" }, () => {
 
       expect(mockPublish.callCount).toBe(1);
 
-      expect(mockPublish.calledWith({
-        name: `Webpack Dev Server ${os.hostname()}:${port}`,
-        port,
-        type: "http",
-        protocol: "udp",
-        subtypes: ["webpack"],
-      })).toBeTruthy();
+      expect(
+        mockPublish.calledWith({
+          name: `Webpack Dev Server ${os.hostname()}:${port}`,
+          port,
+          type: "http",
+          protocol: "udp",
+          subtypes: ["webpack"],
+        }),
+      ).toBeTruthy();
 
       expect(mockUnpublishAll.callCount).toBe(0);
       expect(mockDestroy.callCount).toBe(0);
 
-      expect(response.status()).toMatchSnapshotWithArray();
+      expect(response.status()).toMatchSnapshotWithArray("response status");
 
       expect(
-        consoleMessages.map((message) => message.text()))
-      .toMatchSnapshotWithArray();
+        consoleMessages.map((message) => message.text()),
+      ).toMatchSnapshotWithArray("console messages");
 
-      expect(pageErrors).toMatchSnapshotWithArray();
+      expect(pageErrors).toMatchSnapshotWithArray("page errors");
     });
   });
 });

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+["[webpack-dev-server] Server started: Hot Module Replacement enabled, Live Reloading enabled, Progress disabled, Overlay enabled.","[HMR] Waiting for update signal from WDS...","Hey."]`