Merge pull request #271 from rgis-app/remove-max-diff-pixel-ratio

Remove maxDiffPixelRatio tolerance from screenshot tests

Author: frewsxcv

www/playwright.config.ts

@@ -14,7 +14,8 @@ export default defineConfig({
 
   expect: {
     toHaveScreenshot: {
-      maxDiffPixelRatio: 0.03,
+      // Hardware GPU rendering is inherently non-deterministic between runs
+      maxDiffPixelRatio: 0.05,
       threshold: 0.2,
     },
   },

www/tests/fixtures/app-fixture.ts

@@ -86,9 +86,9 @@ export class AppPage {
     await this.waitForNextFrame();
   }
 
-  async expectScreenshot(name: string, options?: { maxDiffPixelRatio?: number }) {
+  async expectScreenshot(name: string) {
     await this.stabilizeForScreenshot();
-    await expect(this.page).toHaveScreenshot(name, options);
+    await expect(this.page).toHaveScreenshot(name);
   }
 
   async clickOnCanvas(xFrac: number, yFrac: number) {

www/tests/raster-layer.spec.ts

@@ -61,10 +61,6 @@ for (const filePath of geotiffFiles) {
 
     await appPage.loadGeoTIFFFile(`./dist/geotiff-test-data/${filePath}`);
 
-    // Remote raster data can have minor non-deterministic rendering differences
-    const options = filePath.startsWith("real_data/")
-      ? { maxDiffPixelRatio: 0.05 }
-      : undefined;
-    await appPage.expectScreenshot(snapshotName(filePath), options);
+    await appPage.expectScreenshot(snapshotName(filePath));
   });
 }