Include Palette Manager runtime files in Playwright V8 coverage report - PR_26126_058-playwright-coverage-include-palette-manager

Author: DavidQ

docs/dev/reports/playwright_v8_coverage_report.txt

@@ -1,7 +1,8 @@
 Playwright V8 Coverage Report
 
-PR: PR_26126_057-playwright-coverage-formatting
+PR: PR_26126_058-playwright-coverage-include-palette-manager
 Coverage source: Playwright/Chromium built-in V8 coverage.
+Coverage scope: all collected runtime JavaScript under src/, tools/, and common/.
 Dependencies: no new npm packages.
 Thresholds: none enforced.
 Note: coverage is an advisory baseline only for this PR.
@@ -58,35 +59,51 @@ Files with executed line/function counts where available:
 (33%) src/engine/rendering/VectorDrawing.js - executed lines 22/22; executed functions 1/3
 (33%) src/engine/runtime/MobileRuntimeTweaks.js - executed lines 21/21; executed functions 1/3
 (33%) src/engine/scene/SceneTransitionController.js - executed lines 22/22; executed functions 1/3
+(36%) tools/palette-manager-v2/controls/PaletteImportExportControl.js - executed lines 67/67; executed functions 4/11
 (38%) src/engine/camera/Camera3D.js - executed lines 48/48; executed functions 3/8
+(38%) tools/palette-manager-v2/modules/PaletteValidationService.js - executed lines 88/88; executed functions 3/8
+(39%) tools/palette-manager-v2/modules/PaletteManagerApp.js - executed lines 934/934; executed functions 43/110
 (40%) src/engine/scene/Scene.js - executed lines 12/12; executed functions 2/5
 (42%) tools/preview-generator-v2/PreviewGeneratorV2Capture.js - executed lines 477/477; executed functions 10/24
+(44%) tools/palette-manager-v2/controls/PaletteEditorControl.js - executed lines 225/225; executed functions 14/32
 (47%) src/engine/runtime/RuntimeMonitoringHooks.js - executed lines 191/191; executed functions 8/17
 (50%) src/engine/camera/ZoneCameraSystem.js - executed lines 26/26; executed functions 1/2
 (50%) src/engine/config/ConfigStore.js - executed lines 46/46; executed functions 3/6
 (50%) src/engine/input/ActionInputDebugOverlay.js - executed lines 51/51; executed functions 1/2
 (50%) src/engine/release/DistributionPackager.js - executed lines 32/32; executed functions 1/2
 (50%) src/engine/rendering/LayeredRenderSystem.js - executed lines 14/14; executed functions 1/2
 (50%) src/engine/rendering/SpriteRenderSystem.js - executed lines 48/48; executed functions 1/2
+(50%) tools/common/PaletteUsageService.js - executed lines 5/5; executed functions 1/2
+(50%) tools/palette-manager-v2/main.js - executed lines 88/88; executed functions 3/6
+(50%) tools/palette-manager-v2/modules/PaletteHistoryStack.js - executed lines 54/54; executed functions 5/10
 (50%) tools/toolRegistry.js - executed lines 425/425; executed functions 4/8
+(55%) tools/palette-manager-v2/modules/SwatchRow.js - executed lines 115/115; executed functions 6/11
 (56%) src/engine/logging/Logger.js - executed lines 56/56; executed functions 5/9
+(56%) tools/palette-manager-v2/modules/paletteUtils.js - executed lines 41/41; executed functions 5/9
 (57%) src/engine/runtime/FullscreenService.js - executed lines 118/118; executed functions 8/14
 (58%) src/engine/core/Engine.js - executed lines 323/323; executed functions 7/12
 (60%) src/engine/core/RuntimeMetrics.js - executed lines 48/48; executed functions 3/5
 (60%) src/engine/persistence/StorageService.js - executed lines 49/49; executed functions 3/5
 (61%) src/engine/runtime/gameImageConvention.js - executed lines 207/207; executed functions 11/18
+(65%) tools/palette-manager-v2/controls/SourcePaletteBrowserControl.js - executed lines 121/121; executed functions 15/23
 (67%) src/shared/number/numberUtils.js - executed lines 14/14; executed functions 2/3
 (67%) tools/preview-generator-v2/PreviewGeneratorV2ShellControl.js - executed lines 117/117; executed functions 8/12
+(71%) tools/palette-manager-v2/controls/PaletteValidationErrorControl.js - executed lines 30/30; executed functions 5/7
 (73%) src/engine/theme/mount-shared-header.js - executed lines 143/143; executed functions 8/11
 (75%) src/engine/core/FixedTicker.js - executed lines 36/36; executed functions 3/4
 (75%) src/engine/theme/Theme.js - executed lines 26/26; executed functions 3/4
+(75%) tools/palette-manager-v2/controls/UserPaletteControl.js - executed lines 95/95; executed functions 12/16
+(75%) tools/palette-manager-v2/paletteManagerShell.js - executed lines 160/160; executed functions 12/16
 (80%) src/engine/core/FrameClock.js - executed lines 32/32; executed functions 4/5
+(80%) src/engine/theme/accordionV2/accordionV2.js - executed lines 26/26; executed functions 4/5
 (81%) tools/preview-generator-v2/PreviewGeneratorV2App.js - executed lines 539/539; executed functions 47/58
+(87%) tools/common/PaletteSortService.js - executed lines 103/103; executed functions 13/15
 (100%) src/engine/audio/index.js - executed lines 15/15; executed functions 1/1
 (100%) src/engine/camera/index.js - executed lines 10/10; executed functions 1/1
 (100%) src/engine/config/index.js - executed lines 7/7; executed functions 1/1
 (100%) src/engine/input/index.js - executed lines 13/13; executed functions 1/1
 (100%) src/engine/logging/index.js - executed lines 8/8; executed functions 1/1
+(100%) src/engine/paletteList.js - executed lines 1205/1205; executed functions 15/15
 (100%) src/engine/persistence/index.js - executed lines 11/11; executed functions 1/1
 (100%) src/engine/release/index.js - executed lines 12/12; executed functions 1/1
 (100%) src/engine/rendering/index.js - executed lines 11/11; executed functions 1/1
@@ -119,3 +136,4 @@ Uncovered or low-coverage changed JS files:
 
 Changed JS files considered:
 (0%) tests/helpers/playwrightV8CoverageReporter.mjs - changed JS file not collected as browser runtime coverage
+(0%) tests/playwright/PreviewGeneratorV2Baseline.spec.mjs - changed JS file not collected as browser runtime coverage

tests/helpers/playwrightV8CoverageReporter.mjs

@@ -46,8 +46,9 @@ export class PlaywrightV8CoverageReporter {
     const reportLines = [
       "Playwright V8 Coverage Report",
       "",
-      "PR: PR_26126_057-playwright-coverage-formatting",
+      "PR: PR_26126_058-playwright-coverage-include-palette-manager",
       "Coverage source: Playwright/Chromium built-in V8 coverage.",
+      "Coverage scope: all collected runtime JavaScript under src/, tools/, and common/.",
       "Dependencies: no new npm packages.",
       "Thresholds: none enforced.",
       "Note: coverage is an advisory baseline only for this PR.",

tests/playwright/PreviewGeneratorV2Baseline.spec.mjs

@@ -113,6 +113,13 @@ async function openPreviewGenerator(page, { withFakeRepo = false, validSampleIds
   return server;
 }
 
+async function openPaletteManager(page) {
+  const server = await startRepoServer();
+  await coverageReporter.start(page);
+  await page.goto(`${server.baseUrl}/tools/palette-manager-v2/index.html`, { waitUntil: "networkidle" });
+  return server;
+}
+
 async function expectAccordionToggles(page, contentId) {
   const header = page.locator(`.accordion-v2__header[aria-controls="${contentId}"]`);
   const content = page.locator(`#${contentId}`);
@@ -409,4 +416,31 @@ test.describe("Preview Generator V2 baseline", () => {
       await server.close();
     }
   });
+
+  test("loads Palette Manager V2 so coverage includes Palette Manager runtime files", async ({ page }) => {
+    const server = await openPaletteManager(page);
+    const pageErrors = [];
+
+    page.on("pageerror", (error) => {
+      pageErrors.push(error.message);
+    });
+
+    try {
+      await expect(page.locator(".palette-manager-v2.app-shell")).toBeVisible();
+      await expect(page.locator("h1", { hasText: "Palette Manager V2" })).toBeVisible();
+      await expect(page.locator('nav[aria-label="menuSample"]')).toBeVisible();
+      await expect(page.locator("#sourcePaletteSelect")).toBeVisible();
+      await expect(page.locator("#sourceSwatchList .palette-manager-v2__source-tile").first()).toBeVisible();
+      await expect(page.locator("#importPaletteButton")).toBeVisible();
+      await expect(page.locator("#copyPaletteButton")).toBeVisible();
+      await expect(page.locator("#exportPaletteButton")).toBeVisible();
+      await expect(page.locator("#paletteStatus")).toContainText("Ready.");
+      await page.waitForFunction(() => Boolean(globalThis.paletteManagerV2App?.getPaletteValue));
+
+      expect(pageErrors).toEqual([]);
+    } finally {
+      await coverageReporter.stop(page);
+      await server.close();
+    }
+  });
 });