test: move test case to appropriate place (#3871)

* chore: remove duplicate test case

* test: move test case to appropriate place

Author: snitin315

test/server/client-option.test.js

@@ -105,26 +105,6 @@ describe("client option", () => {
 
       await server.stop();
     });
-
-    it("disables hot entry", async () => {
-      const compiler = webpack(config);
-
-      server = new Server(
-        {
-          hot: false,
-          port,
-        },
-        compiler
-      );
-
-      await server.start();
-
-      const res = await request(server.app).get("/main.js");
-
-      expect(res.text).not.toMatch(/webpack\/hot\/dev-server\.js/);
-
-      await server.stop();
-    });
   });
 
   describe("webSocketTransport", () => {

test/server/hot-option.test.js

@@ -34,6 +34,35 @@ describe("hot option", () => {
     });
   });
 
+  describe("simple config with hot: false", () => {
+    beforeAll(async () => {
+      const compiler = webpack(config);
+
+      server = new Server(
+        {
+          port,
+          hot: false,
+        },
+        compiler
+      );
+
+      await server.start();
+
+      req = request(server.app);
+    });
+
+    afterAll(async () => {
+      await server.stop();
+    });
+
+    it("should not include hot script in the bundle", async () => {
+      const response = await req.get("/main.js");
+
+      expect(response.status).toEqual(200);
+      expect(response.text).not.toContain("webpack/hot/dev-server.js");
+    });
+  });
+
   describe("simple hot-only config entries", () => {
     beforeAll(async () => {
       const compiler = webpack(config);