ci: update playwright config

SSShooter · Jan 13, 2024 · fe6af7c · fe6af7c
1 parent 82d4767
commit fe6af7c
Show file tree

Hide file tree

Showing 16 changed files with 7 additions and 4 deletions.
diff --git a/playwright.config.ts b/playwright.config.ts
@@ -10,14 +10,16 @@ import { defineConfig, devices } from '@playwright/test'
  * See https://playwright.dev/docs/test-configuration.
  */
 export default defineConfig({
+  snapshotPathTemplate: '{testDir}/__screenshots__/{testFilePath}/{arg}{ext}',
   timeout: 10000,
   testDir: './tests',
   /* Run tests in files in parallel */
   fullyParallel: true,
   /* Fail the build on CI if you accidentally left test.only in the source code. */
   forbidOnly: !!process.env.CI,
   /* Retry on CI only */
-  retries: process.env.CI ? 2 : 0,
+  // retries: process.env.CI ? 2 : 0,
+  retries: 0,
   /* Opt out of parallel tests on CI. */
   workers: process.env.CI ? 1 : undefined,
   /* Reporter to use. See https://playwright.dev/docs/test-reporters */

diff --git a/tests/MindElixirFixture.ts b/tests/MindElixirFixture.ts
@@ -48,6 +48,8 @@ export class MindElixirFixture {
     })
   }
   async toHaveScreenshot(locator?: Locator) {
-    await expect(locator || this.page.locator('me-nodes')).toHaveScreenshot()
+    await expect(locator || this.page.locator('me-nodes')).toHaveScreenshot({
+      maxDiffPixelRatio: 0.08,
+    })
   }
 }
diff --git a/tests/__screenshots__/drag-and-drop.spec.ts/DnD-move-after-1.png b/tests/__screenshots__/drag-and-drop.spec.ts/DnD-move-after-1.png
diff --git a/tests/__screenshots__/drag-and-drop.spec.ts/DnD-move-before-1.png b/tests/__screenshots__/drag-and-drop.spec.ts/DnD-move-before-1.png
diff --git a/...napshots/DnD-move-in-1-chromium-win32.png → ..._/drag-and-drop.spec.ts/DnD-move-in-1.png b/...napshots/DnD-move-in-1-chromium-win32.png → ..._/drag-and-drop.spec.ts/DnD-move-in-1.png
diff --git a/...ec.ts-snapshots/Add-Before-1-chromium.png → ...ts__/interaction.spec.ts/Add-Before-1.png b/...ec.ts-snapshots/Add-Before-1-chromium.png → ...ts__/interaction.spec.ts/Add-Before-1.png
diff --git a/tests/__screenshots__/interaction.spec.ts/Add-Child-1.png b/tests/__screenshots__/interaction.spec.ts/Add-Child-1.png
diff --git a/...ec.ts-snapshots/Add-Parent-1-chromium.png → ...ts__/interaction.spec.ts/Add-Parent-1.png b/...ec.ts-snapshots/Add-Parent-1-chromium.png → ...ts__/interaction.spec.ts/Add-Parent-1.png
diff --git a/...c.ts-snapshots/Add-Sibling-1-chromium.png → ...s__/interaction.spec.ts/Add-Sibling-1.png b/...c.ts-snapshots/Add-Sibling-1-chromium.png → ...s__/interaction.spec.ts/Add-Sibling-1.png
diff --git a/...-snapshots/Clear-and-reset-1-chromium.png → ...interaction.spec.ts/Clear-and-reset-1.png b/...-snapshots/Clear-and-reset-1-chromium.png → ...interaction.spec.ts/Clear-and-reset-1.png
diff --git a/...pec.ts-snapshots/Edit-Node-1-chromium.png → ...ots__/interaction.spec.ts/Edit-Node-1.png b/...pec.ts-snapshots/Edit-Node-1-chromium.png → ...ots__/interaction.spec.ts/Edit-Node-1.png
diff --git a/...c.ts-snapshots/Remove-Node-1-chromium.png → ...s__/interaction.spec.ts/Remove-Node-1.png b/...c.ts-snapshots/Remove-Node-1-chromium.png → ...s__/interaction.spec.ts/Remove-Node-1.png
diff --git a/tests/drag-and-drop.spec.ts-snapshots/DnD-move-after-1-chromium-win32.png b/tests/drag-and-drop.spec.ts-snapshots/DnD-move-after-1-chromium-win32.png
diff --git a/tests/drag-and-drop.spec.ts-snapshots/DnD-move-before-1-chromium-win32.png b/tests/drag-and-drop.spec.ts-snapshots/DnD-move-before-1-chromium-win32.png
diff --git a/tests/interaction.spec.ts-snapshots/Add-Child-1-chromium.png b/tests/interaction.spec.ts-snapshots/Add-Child-1-chromium.png
diff --git a/tests/mind-elixir-test.ts b/tests/mind-elixir-test.ts
@@ -8,9 +8,8 @@ type MyFixtures = {
 // Extend base test by providing "todoPage" and "settingsPage".
 // This new "test" can be used in multiple test files, and each of them will get the fixtures.
 export const test = base.extend<MyFixtures>({
-  me: async ({ page }, use, testInfo) => {
+  me: async ({ page }, use) => {
     // Set up the fixture.
-    testInfo.snapshotSuffix = ''
     const me = new MindElixirFixture(page)
     await me.goto()