Rename all snapshots to "-light"

frontend/test/playwright/playwright.config.ts

@@ -21,6 +21,8 @@ const pwCommand = process.env.FASTSTART !== "false" ? "dev" : "prod:playwright"
 
 const config: PlaywrightTestConfig = {
   forbidOnly: !!process.env.CI,
+  snapshotPathTemplate:
+    "{snapshotDir}/{testFileDir}/{testFileName}-snapshots/{arg}{ext}",
   webServer: {
     command: `pnpm exec npm-run-all -p -r talkback ${pwCommand}`,
     timeout: process.env.CI ? 60_000 * 5 : 60_000 * 10, // 5 minutes in CI, 10 in other envs

frontend/test/playwright/utils/breakpoints.ts

@@ -17,7 +17,7 @@ type ExpectSnapshot = <T extends ScreenshotAble>(
 
 type BreakpointBlock = (options: {
   getConfigValues: (name: string) => {
-    name: `${typeof name}-${Breakpoint}.png`
+    name: `${typeof name}-${Breakpoint}-light.png`
   }
   breakpoint: Breakpoint
   expectSnapshot: ExpectSnapshot
@@ -88,7 +88,7 @@ const makeBreakpointDescribe =
       })
 
       const getConfigValues = (name: string) => ({
-        name: `${name}-${breakpoint}.png` as const,
+        name: `${name}-${breakpoint}-light.png` as const,
       })
 
       const expectSnapshot = async <T extends ScreenshotAble>(

frontend/test/playwright/visual-regression/components/content-report-form.spec.ts

@@ -46,7 +46,7 @@ test.describe("content report form", () => {
 
       await button.click()
 
-      await expectSnapshot("content-report", page, undefined, {
+      await expectSnapshot("content-report-ltr", page, undefined, {
         maxDiffPixelRatio: 0.1,
       })
     })

frontend/test/playwright/visual-regression/components/global-audio-player.spec.ts

@@ -32,7 +32,7 @@ for (const dir of languageDirections) {
       // To make the tests consistent, set the played area to the same position
       await page.mouse.click(170, 650)
 
-      await expectSnapshot(`global-audio-player-on-search-${dir}.png`, page)
+      await expectSnapshot(`global-audio-player-on-search-${dir}`, page)
     })
   })
 }

frontend/test/playwright/visual-regression/components/header.spec.ts

@@ -18,7 +18,7 @@ test.describe.configure({ mode: "parallel" })
 const headerSelector = ".main-header"
 
 for (const dir of languageDirections) {
-  test.describe(`header-${dir}`, () => {
+  test.describe(dir, () => {
     breakpoints.describeEvery(({ breakpoint, expectSnapshot }) => {
       test.beforeEach(async ({ page }) => {
         await preparePageForTests(page, breakpoint, { dismissFilter: false })

frontend/test/playwright/visual-regression/pages/pages.spec.ts

@@ -79,7 +79,9 @@ test.describe("layout color is set correctly", () => {
       await page.waitForURL(/ar\/search/)
       await expect(getH1(page, "Cat")).toBeVisible()
 
-      expect(await page.screenshot()).toMatchSnapshot("search-page-rtl-lg.png")
+      expect(await page.screenshot()).toMatchSnapshot(
+        "search-page-rtl-lg-light.png"
+      )
     })
 
     test("change language on homepage and go to content page", async ({
@@ -97,7 +99,7 @@ test.describe("layout color is set correctly", () => {
       await page.mouse.move(100, 100)
 
       expect(await page.screenshot({ fullPage: true })).toMatchSnapshot(
-        "about-ltr-lg.png",
+        "about-ltr-lg-light.png",
         { maxDiffPixelRatio: 0.01 }
       )
     })