dx: steadier test

Author: rejetto

e2e/frontend.spec.ts

@@ -1,5 +1,6 @@
 import { test, expect, Page } from '@playwright/test'
 import fs from 'fs'
+import { wait } from '../src/cross'
 
 const username = 'rejetto'
 const password = 'password'
@@ -287,8 +288,9 @@ test('admin1', async ({ page }) => {
   await screenshot(page);
 });
 
-function screenshot(page: Page, selectorForMask='') {
+async function screenshot(page: Page, selectorForMask='') {
   if (selectorForMask)
     selectorForMask = ',' + selectorForMask
+  await wait(1000) // this accounts especially for our DataTable component which takes time to set the layout
   return expect(page).toHaveScreenshot({ fullPage: true, mask: [page.locator(`.maskInTests${selectorForMask}`)] });
 }

e2e/screenshot.css

@@ -3,4 +3,9 @@
 main *:hover { background: none !important; }
 
 .hideInTests, /* use for elements with variable size, where masking would produce changes anyway */
-.list-wrapper:not([uri="/"]) .entry-ts { display: none } /* ignore changing timestamps inside folders */
+.list-wrapper:not([uri="/"]) .entry-ts { display: none } /* ignore changing timestamps inside folders */
+
+.MuiTooltip-tooltip { /* tooltips are subjected to small displacements */
+    visibility: hidden !important;
+    pointer-events: none !important;
+}

playwright.config.ts

@@ -13,7 +13,7 @@ import { defineConfig, devices } from '@playwright/test';
  */
 export default defineConfig({
   testDir: './e2e',
-  timeout: 15_000,
+  timeout: 20_000,
   fullyParallel: true, // Run tests in files in parallel
   forbidOnly: !!process.env.CI, // Fail the build on CI if you accidentally left test.only in the source code.
   retries: process.env.CI ? 2 : 0, // Retry on CI only