test(e2e): snapshot the logging=log module detail and drop the covered unit tests

A logging=log e2e run now snapshots the collapsed "Updated modules:"
group and its per-module entries straight from the browser console
(puppeteer surfaces group frames as console events). The jsdom
equivalents — the process-update logging describe and the
building-message log test, whose output already appears verbatim in the
info-level e2e snapshot — are removed.

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

Author: bjohansebas

test/client.test.js

@@ -115,17 +115,6 @@ describe("client", () => {
       jest.restoreAllMocks();
     });
 
-    it("logs the changed file on building messages", () => {
-      EventSourceStub.lastInstance().onmessage(
-        makeMessage({ action: "building", file: "/src/index.js" }),
-      );
-      expect(
-        console.info.mock.calls.some(([msg]) =>
-          msg.includes("rebuilding (/src/index.js changed)"),
-        ),
-      ).toBe(true);
-    });
-
     it("calls subscribeAll handler on default messages", () => {
       const spy = jest.fn();
       client.subscribeAll(spy);

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

@@ -10,6 +10,20 @@ You may need an appropriate loader to handle this file type, currently no loader
 ]
 `;
 
+exports[`client logging (browser) logging=log adds the collapsed per-module detail 1`] = `
+[
+  "[webpack-dev-middleware] connected",
+  "[webpack-dev-middleware] bundle rebuilding (<fixture>/app.js changed)",
+  "[webpack-dev-middleware] bundle rebuilt in Xms",
+  "[webpack-dev-middleware] Checking for updates on the server...",
+  "[webpack-dev-middleware] Hot updated 1 modules.",
+  "[webpack-dev-middleware] Updated modules:",
+  "[webpack-dev-middleware]  - ./app.js",
+  "console.groupEnd",
+  "[webpack-dev-middleware] App is up to date.",
+]
+`;
+
 exports[`client logging (browser) logging=none silences the whole cycle 1`] = `[]`;
 
 exports[`client logging (browser) logging=warn keeps warnings only 1`] = `

test/e2e/logging.test.js

@@ -70,6 +70,24 @@ describe("client logging (browser)", () => {
     expect(normalizeConsole(console_.messages)).toMatchSnapshot();
   });
 
+  it("logging=log adds the collapsed per-module detail", async () => {
+    hotApp = await createHotApp({ query: "?logging=log", code: app("v1") });
+    ({ page, browser } = await runBrowser());
+    const console_ = collectConsole(page);
+
+    await page.goto(hotApp.url);
+    await waitForAppText(page, "v1");
+    await console_.waitFor("connected");
+
+    hotApp.edit(app("v2"));
+    await waitForAppText(page, "v2");
+    await console_.waitFor("App is up to date");
+
+    // Includes the "Updated modules:" collapsed group and its " - ./app.js"
+    // entry, which the info level gates off.
+    expect(normalizeConsole(console_.messages)).toMatchSnapshot();
+  });
+
   it("logging=none silences the whole cycle", async () => {
     hotApp = await createHotApp({ query: "?logging=none", code: app("v1") });
     ({ page, browser } = await runBrowser());

test/process-update.test.js

@@ -220,69 +220,4 @@ describe("process-update", () => {
 
     expect(reloadPage).toHaveBeenCalledTimes(1);
   });
-
-  describe("logging", () => {
-    /**
-     * @param {jest.Mock} spy console spy
-     * @param {string} text expected substring
-     * @returns {boolean} true when some call contains the text
-     */
-    function logged(spy, text) {
-      return spy.mock.calls.some((call) => call.join(" ").includes(text));
-    }
-
-    beforeEach(() => {
-      if (typeof console.groupCollapsed !== "function") {
-        console.groupCollapsed = () => {};
-      }
-      if (typeof console.groupEnd !== "function") {
-        console.groupEnd = () => {};
-      }
-      jest.spyOn(console, "groupCollapsed").mockImplementation(() => {});
-      jest.spyOn(console, "groupEnd").mockImplementation(() => {});
-    });
-
-    it("logs a one-line summary at the default info level", async () => {
-      applyUpdate = loadApplyUpdate(
-        makeFakeHot({
-          checkResult: ["./a.js", "./b.js"],
-          applyImpl: () => Promise.resolve(["./a.js", "./b.js"]),
-        }),
-      );
-
-      applyUpdate("new-hash", { reload: true });
-      globalThis.__webpack_hash__ = "new-hash";
-      await flushPromises();
-
-      expect(logged(console.info, "Hot updated 2 modules.")).toBe(true);
-      // Groups are gated below the "log" level.
-      expect(console.groupCollapsed).not.toHaveBeenCalled();
-      expect(logged(console.log, "./a.js")).toBe(false);
-    });
-
-    it("adds a collapsed group with the module detail at the log level", async () => {
-      applyUpdate = loadApplyUpdate(
-        makeFakeHot({
-          checkResult: ["./a.js", "./b.js"],
-          applyImpl: () => Promise.resolve(["./a.js", "./b.js"]),
-        }),
-      );
-      setLogLevel("log");
-
-      applyUpdate("new-hash", { reload: true });
-      globalThis.__webpack_hash__ = "new-hash";
-      await flushPromises();
-
-      expect(logged(console.info, "Hot updated 2 modules.")).toBe(true);
-      expect(
-        logged(
-          /** @type {jest.Mock} */ (console.groupCollapsed),
-          "Updated modules:",
-        ),
-      ).toBe(true);
-      expect(logged(console.log, "./a.js")).toBe(true);
-      expect(logged(console.log, "./b.js")).toBe(true);
-      expect(console.groupEnd).toHaveBeenCalled();
-    });
-  });
 });