test(e2e): snapshot the console wherever it is asserted

hot.statsOptions warnings-filtering runs in the browser — its snapshot
is a lone "connected", the warning never entering the payload, while
the overlay={"warnings":false} snapshot shows the warning reaching the
console and stopping there. Three hot unit tests fall away: the
late-connect sync and post-close handling are covered by the e2e and
framework suites, and statsOptions forwarding by the new run.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: bjohansebas

test/e2e/__snapshots__/overlay.test.js.snap.webpack5

@@ -1,5 +1,20 @@
 // Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
+exports[`error overlay (browser) keeps warnings out of the payload with hot.statsOptions 1`] = `
+[
+  "[webpack-dev-middleware] connected",
+]
+`;
+
+exports[`error overlay (browser) overlay={"warnings":false} suppresses warnings (dev-server shape) 1`] = `
+[
+  "[webpack-dev-middleware] connected",
+  "[webpack-dev-middleware] bundle has 1 warnings",
+  "[webpack-dev-middleware] ./app.js 5:10-22
+Critical dependency: the request of a dependency is an expression",
+]
+`;
+
 exports[`error overlay (browser) paginates multiple problems with a counter 1`] = `
 [
   "[webpack-dev-middleware] connected",

test/e2e/overlay.test.js

@@ -261,6 +261,41 @@ describe("error overlay (browser)", () => {
     await console_.waitFor("Critical dependency");
 
     expect(await page.$(`#${OVERLAY_ID}`)).toBeNull();
+    expect(normalizeConsole(console_.messages)).toMatchSnapshot();
+  });
+
+  it("keeps warnings out of the payload with hot.statsOptions", async () => {
+    hotApp = await createHotApp({
+      // The README's documented recipe: keep warnings in the build output
+      // but out of the SSE payload entirely.
+      hot: { statsOptions: { warnings: false } },
+      code: `
+        document.getElementById("app").textContent = "v1";
+        const dep = "./nothing";
+        try {
+          require(dep);
+        } catch (err) {
+          // expected
+        }
+        if (module.hot) {
+          module.hot.accept();
+        }
+      `,
+    });
+    ({ page, browser } = await runBrowser());
+    const console_ = collectConsole(page);
+
+    await page.goto(hotApp.url);
+    await console_.waitFor("connected");
+
+    // The sync arrived (connected implies it); give rendering a beat.
+    await new Promise((resolve) => {
+      setTimeout(resolve, 500);
+    });
+
+    expect(await page.$(`#${OVERLAY_ID}`)).toBeNull();
+    // Just the connect — the warning never entered the payload.
+    expect(normalizeConsole(console_.messages)).toMatchSnapshot();
   });
 
   it("applies a warnings filter function from the query (dev-server parity)", async () => {

test/hot.test.js

@@ -366,44 +366,6 @@ describe("createHot", () => {
     hot.close();
   });
 
-  it("sends a sync payload to a client that connects after a build", () => {
-    const compiler = makeFakeCompiler();
-    const hot = createHot(compiler, {});
-
-    // A build finishes BEFORE anyone connects.
-    compiler.emitDone(makeFakeStats());
-
-    const { writes } = attachClient({ handler: hot.handle });
-
-    expect(writes.some((w) => w.includes('"action":"sync"'))).toBe(true);
-
-    hot.close();
-  });
-
-  it("responds 404 instead of hanging when a client connects after close()", () => {
-    const compiler = makeFakeCompiler();
-    const hot = createHot(compiler, {});
-
-    hot.close();
-
-    /** @type {number | undefined} */
-    let statusCode;
-    let ended = false;
-    const res = {
-      writeHead: (code) => {
-        statusCode = code;
-      },
-      end: () => {
-        ended = true;
-      },
-    };
-
-    hot.handle({}, res);
-
-    expect(statusCode).toBe(404);
-    expect(ended).toBe(true);
-  });
-
   it("does not re-send the catch-up sync to already connected clients", () => {
     const compiler = makeFakeCompiler();
     const hot = createHot(compiler, {});
@@ -479,35 +441,6 @@ describe("createHot", () => {
     hot.close();
   });
 
-  it("forwards custom statsOptions to stats.toJson", () => {
-    const compiler = makeFakeCompiler();
-    const hot = createHot(compiler, {
-      statsOptions: { modules: true, ids: true },
-    });
-    attachClient({ handler: hot.handle });
-
-    /** @type {EXPECTED_OBJECT} */
-    let receivedOptions;
-
-    compiler.emitDone({
-      toJson(statsOptions) {
-        receivedOptions = statsOptions;
-        return {
-          time: 1,
-          hash: "h",
-          warnings: [],
-          errors: [],
-          modules: [],
-        };
-      },
-      compilation: undefined,
-    });
-
-    expect(receivedOptions).toMatchObject({ modules: true, ids: true });
-
-    hot.close();
-  });
-
   it("publishes throttled progress events when progress is enabled", () => {
     const compiler = makeFakeCompiler();
     /** @type {EXPECTED_OBJECT} */