StorybookのVRTにファイル一覧の同期チェックを追加

.github/workflows/test.yml

@@ -155,9 +155,12 @@ jobs:
       - name: Collect patch for snapshots
         if: needs.config.outputs.shouldUpdateSnapshots == 'true'
         run: |
-          git add --intent-to-add tests/  # git diff に表示されるようにする
-          git diff tests/  # ロギング用
-          git diff --binary tests/ > patch-${{ matrix.os }}.diff
+          # ログ用
+          git status
+
+          # git diff に表示されるようにする
+          git add --intent-to-add --all tests/
+          git diff --binary --cached tests/ > patch-${{ matrix.os }}.diff
 
       - name: Upload patch to artifact
         if: needs.config.outputs.shouldUpdateSnapshots == 'true'
@@ -184,14 +187,14 @@ jobs:
         uses: actions/download-artifact@v4
         with:
           pattern: updated-snapshots-*
-          path: patches
+          path: /tmp/patches
           merge-multiple: true
 
       - name: Commit updated snapshots
         id: commit-updated-snapshots
         run: |
           # パッチを適用
-          for patch in patches/*.diff; do
+          for patch in /tmp/patches/*.diff; do
             git apply --allow-empty $patch
             rm $patch
           done

tests/e2e/storybook/スクリーンショット.spec.mts

@@ -1,10 +1,12 @@
 // 起動中のStorybookで様々なStoryを表示し、スクリーンショットを撮って比較するVRT。
 // テスト自体はend-to-endではないが、Playwrightを使う関係でe2eディレクトリ内でテストしている。
+import fs from "fs/promises";
+import path from "path";
 import { test, expect, Locator } from "@playwright/test";
 import z from "zod";
 
 // Storybook 8.3.5時点でのindex.jsonのスキーマ。
-// もしスキーマが変わってテストが通らなくなった場合は、このスキーマを修正する。
+// もしスキーマが変わってテストが通らなくなった場合は、このスキーマを修正すること。
 const storybookIndexSchema = z.object({
   v: z.literal(5),
   entries: z.record(
@@ -19,12 +21,19 @@ const storybookIndexSchema = z.object({
 });
 type StorybookIndex = z.infer<typeof storybookIndexSchema>;
 type Story = StorybookIndex["entries"][string];
+type Theme = "light" | "dark";
+
+const toSnapshotFileName = (story: Story, theme: Theme) =>
+  `${story.id}-${theme}.png`;
 
 // テスト対象のStory一覧を取得する。
 // play-fnが付いているStoryはUnit Test用Storyとみなしてスクリーンショットを撮らない
 const getStoriesToTest = (index: StorybookIndex) =>
   Object.values(index.entries).filter(
-    (entry) => entry.type === "story" && !entry.tags.includes("play-fn"),
+    (entry) =>
+      entry.type === "story" &&
+      !entry.tags.includes("play-fn") &&
+      !entry.tags.includes("skip-screenshot"),
   );
 
 let index: StorybookIndex;
@@ -52,14 +61,11 @@ for (const story of currentStories) {
 for (const [story, stories] of Object.entries(allStories)) {
   test.describe(story, () => {
     for (const story of stories) {
-      if (story.tags.includes("skip-screenshot")) {
-        continue;
-      }
       test.describe(story.name, () => {
         for (const [theme, name] of [
           ["light", "ライト"],
           ["dark", "ダーク"],
-        ]) {
+        ] as const) {
           test(`テーマ：${name}`, async ({ page }) => {
             test.skip(
               process.platform !== "win32",
@@ -95,11 +101,40 @@ for (const [story, stories] of Object.entries(allStories)) {
               elementToScreenshot = root;
             }
             await expect(elementToScreenshot).toHaveScreenshot(
-              `${story.id}-${theme}.png`,
+              toSnapshotFileName(story, theme),
             );
           });
         }
       });
     }
   });
 }
+
+test("スクリーンショットの一覧に過不足が無い", async () => {
+  test.skip(process.platform !== "win32", "Windows以外のためスキップします");
+  const screenshotFiles = await fs.readdir(test.info().snapshotDir);
+  const screenshotPaths = screenshotFiles.map((file) =>
+    path.join(test.info().snapshotDir, file),
+  );
+
+  const expectedScreenshots = currentStories.flatMap((story) =>
+    (["light", "dark"] as const).map((theme) =>
+      test.info().snapshotPath(toSnapshotFileName(story, theme)),
+    ),
+  );
+
+  screenshotPaths.sort();
+  expectedScreenshots.sort();
+
+  // update-snapshotsが指定されていたら、余分なスクリーンショットを削除する。
+  // 指定されていなかったら、スクリーンショットの一覧が一致していることを確認する。
+  if (test.info().config.updateSnapshots === "all") {
+    for (const screenshot of screenshotPaths) {
+      if (!expectedScreenshots.includes(screenshot)) {
+        await fs.unlink(screenshot);
+      }
+    }
+  } else {
+    expect(screenshotPaths).toEqual(expectedScreenshots);
+  }
+});