From 2fc4b34c5a54624772b5d28151aa3f80c09cb364 Mon Sep 17 00:00:00 2001
From: David Taylor <david@taylorhq.com>
Date: Mon, 4 Aug 2025 11:01:09 +0100
Subject: [PATCH] DEV: Enable local-storage for smoke test (#34060)

The localStorage API has been a web standard for a very long time. It's
safe to assume it'll be available in all users browsers, so it makes
sense for the smoke test to match.

Also adds console logs to the smoke-test output, to make debugging
failures easier.
---
 test/smoke-test.mjs | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/test/smoke-test.mjs b/test/smoke-test.mjs
index 7665b80112b..6af0b2515d8 100644
--- a/test/smoke-test.mjs
+++ b/test/smoke-test.mjs
@@ -20,9 +20,15 @@ import puppeteer from "puppeteer-core";
     executablePath: Launcher.getInstallations()[0],
     // when debugging locally setting the SHOW_BROWSER env variable can be very helpful
     headless: process.env.SHOW_BROWSER === undefined,
-    args: ["--disable-local-storage", "--no-sandbox"],
+    args: ["--no-sandbox"],
   });
   const page = await browser.newPage();
+  page.on("console", (msg) => {
+    if (["error", "warning"].includes(msg.type())) {
+      console.log(`PAGE ${msg.type().toUpperCase()}: ${msg.text()}`);
+    }
+  });
+  page.on("pageerror", (err) => console.log(`PAGE ERROR: ${err.message}`));
 
   await page.setViewport({
     width: 1366,