test(e2e): Skip shopify image cache test when shopify is unconfigured (#1068)

The shopify-image-cache e2e test depends on real products rendering on
/store, which requires SHOPIFY_STORE_DOMAIN and SHOPIFY_STOREFRONT_ACCESS_TOKEN.
CI does not set these, so the store page falls back to a "Store Configuration
Required" panel and the test fails on a missing img[src*="cdn.shopify.com"]
locator.

Fixes:
- Detect the fallback panel at runtime and call test.skip with a clear message
  pointing at the env vars required to run the test meaningfully.
- Forward SHOPIFY_STORE_DOMAIN and SHOPIFY_STOREFRONT_ACCESS_TOKEN from the
  parent environment to the spawned web server in playwright.config.ts when
  available, so future CI runs (with secrets set) or local runs without
  reuseExistingServer actually exercise the Shopify path.

Note: a separate service-worker timing bug exists on the offline-reload step
(page.reload after setOffline returns ERR_INTERNET_DISCONNECTED when the SW
has not finished caching the HTML). That is unrelated to Shopify config and
will be filed as a follow-up.

Author: jeromehardaway

playwright.config.ts

@@ -70,6 +70,16 @@ export default defineConfig({
         timeout: process.env.CI ? 30000 : 120000,
         env: {
             NEXTAUTH_SECRET: process.env.NEXTAUTH_SECRET || "test-secret-for-playwright",
+            // Forward Shopify creds when present so /store can render real
+            // products for e2e tests that depend on them. When unset (most CI
+            // runs), the Shopify-dependent specs skip themselves at runtime.
+            ...(process.env.SHOPIFY_STORE_DOMAIN && {
+                SHOPIFY_STORE_DOMAIN: process.env.SHOPIFY_STORE_DOMAIN,
+            }),
+            ...(process.env.SHOPIFY_STOREFRONT_ACCESS_TOKEN && {
+                SHOPIFY_STOREFRONT_ACCESS_TOKEN:
+                    process.env.SHOPIFY_STOREFRONT_ACCESS_TOKEN,
+            }),
         },
     },
 });

tests/e2e/shopify-image-cache.spec.ts

@@ -15,6 +15,19 @@ test.describe("Shopify Image Caching", () => {
     // Wait for the page to be fully loaded
     await page.waitForLoadState("networkidle");
 
+    // When SHOPIFY_STORE_DOMAIN / SHOPIFY_STOREFRONT_ACCESS_TOKEN aren't set,
+    // /store renders a "Store Configuration Required" fallback with no
+    // products. Skip the test at runtime rather than fail — this test
+    // requires a live Shopify connection to be meaningful.
+    const storeUnconfigured = await page
+      .getByText("Store Configuration Required")
+      .isVisible()
+      .catch(() => false);
+    test.skip(
+      storeUnconfigured,
+      "Shopify not configured — set SHOPIFY_STORE_DOMAIN + SHOPIFY_STOREFRONT_ACCESS_TOKEN to run this test"
+    );
+
     // Find the first product image from Shopify
     const productImage = page.locator('img[src*="cdn.shopify.com"]').first();
     await expect(productImage).toBeVisible();