From 7bd60e0ddbc2a67d23c0ec26c1b055cfd5d77c90 Mon Sep 17 00:00:00 2001
From: Samuel Reichert <samuelreichertt@gmail.com>
Date: Tue, 26 May 2026 16:05:24 +0200
Subject: [PATCH 1/6] test(video-player): wait for widget after tab clicks, fix
 poster race

---
 .../video-player-web/e2e/VideoPlayer.spec.js  | 26 +++++++++++++++----
 1 file changed, 21 insertions(+), 5 deletions(-)

diff --git a/packages/pluggableWidgets/video-player-web/e2e/VideoPlayer.spec.js b/packages/pluggableWidgets/video-player-web/e2e/VideoPlayer.spec.js
index 8ba37034a7..78d72d2223 100644
--- a/packages/pluggableWidgets/video-player-web/e2e/VideoPlayer.spec.js
+++ b/packages/pluggableWidgets/video-player-web/e2e/VideoPlayer.spec.js
@@ -1,5 +1,5 @@
 import { test, expect } from "@mendix/run-e2e/fixtures";
-import { waitForDataReady } from "@mendix/run-e2e/mendix-helpers";
+import { waitForWidget } from "@mendix/run-e2e/mendix-helpers";
 
 test.describe("Video Player", () => {
     test.beforeEach(async ({ page }) => {
@@ -41,6 +41,7 @@ test.describe("Tab page", () => {
 
     test("renders youtube video", async ({ page }) => {
         await page.locator(".mx-name-tabPage1").click();
+        await waitForWidget(page, "videoPlayer1");
         await expect(
             page.locator(".widget-video-player.widget-video-player-container.mx-name-videoPlayer1.size-box iframe")
         ).toBeVisible();
@@ -52,6 +53,7 @@ test.describe("Tab page", () => {
 
     test("renders vimeo video", async ({ page }) => {
         await page.locator(".mx-name-tabPage5").click();
+        await waitForWidget(page, "videoPlayer5");
         await expect(
             page.locator(".widget-video-player.widget-video-player-container.mx-name-videoPlayer5.size-box iframe")
         ).toBeVisible();
@@ -63,6 +65,7 @@ test.describe("Tab page", () => {
 
     test("renders dailymotion video", async ({ page }) => {
         await page.locator(".mx-name-tabPage4").click();
+        await waitForWidget(page, "videoPlayer4");
         await expect(
             page.locator(".widget-video-player.widget-video-player-container.mx-name-videoPlayer4.size-box iframe")
         ).toBeVisible();
@@ -74,6 +77,7 @@ test.describe("Tab page", () => {
 
     test("renders html5 video", async ({ page }) => {
         await page.locator(".mx-name-tabPage3").click();
+        await waitForWidget(page, "videoPlayer3");
         await expect(
             page.locator(".widget-video-player.widget-video-player-container.mx-name-videoPlayer3.size-box video")
         ).toBeVisible();
@@ -112,7 +116,19 @@ test.describe("External video", () => {
         await widget.scrollIntoViewIfNeeded();
         await expect(widget).toBeVisible();
         await expect(videoLocator).toHaveAttribute("poster", /.+/);
-        await waitForDataReady(page);
+        // Wait for poster image to decode in-page before screenshotting.
+        // page.evaluate with a separate Image() is unreliable — the promise can
+        // resolve before the <video> element itself has rendered the poster frame.
+        await videoLocator.evaluate(el =>
+            el.poster
+                ? new Promise(r => {
+                      const i = new Image();
+                      i.onload = i.onerror = r;
+                      i.src = el.poster;
+                      if (i.complete) r();
+                  })
+                : Promise.resolve()
+        );
         await expect(widget).toHaveScreenshot("videoPlayerExternalPoster.png");
     });
 
@@ -122,21 +138,21 @@ test.describe("External video", () => {
         });
 
         test("renders video aspect ratio correctly", async ({ page }) => {
-            await expect(page.locator(".mx-name-videoPlayer1")).toBeVisible();
+            await waitForWidget(page, "videoPlayer1");
             const videoElement = await page.locator(".mx-name-videoPlayer1").first().boundingBox();
             const { width, height } = videoElement;
             const aspectRatio = Number(width / height);
             expect(aspectRatio).toBeCloseTo(16 / 9, 0.1);
 
             await page.locator(".mx-name-tabPage2").click();
-            await expect(page.locator(".mx-name-videoPlayer3")).toBeVisible();
+            await waitForWidget(page, "videoPlayer3");
             const videoElement2 = await page.locator(".mx-name-videoPlayer3").first().boundingBox();
             const { width: width2, height: height2 } = videoElement2;
             const aspectRatio2 = Number(width2 / height2);
             expect(aspectRatio2).toBeCloseTo(3 / 2, 0.1);
 
             await page.locator(".mx-name-tabPage3").click();
-            await expect(page.locator(".mx-name-videoPlayer5")).toBeVisible();
+            await waitForWidget(page, "videoPlayer5");
             const videoElement3 = await page.locator(".mx-name-videoPlayer5").first().boundingBox();
             const { width: width3, height: height3 } = videoElement3;
             expect(width3).toEqual(height3);

From b351f83c1c1e4111e7d6018db548be4a23094a00 Mon Sep 17 00:00:00 2001
From: Samuel Reichert <samuelreichertt@gmail.com>
Date: Thu, 28 May 2026 14:00:03 +0200
Subject: [PATCH 2/6] test(accessibility-helper): add waitForMendixApp calls to
 improve stability of multiple target tests

---
 .../e2e/AccessibilityHelper.spec.js                         | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/packages/pluggableWidgets/accessibility-helper-web/e2e/AccessibilityHelper.spec.js b/packages/pluggableWidgets/accessibility-helper-web/e2e/AccessibilityHelper.spec.js
index 12467203d1..e9318c3f75 100644
--- a/packages/pluggableWidgets/accessibility-helper-web/e2e/AccessibilityHelper.spec.js
+++ b/packages/pluggableWidgets/accessibility-helper-web/e2e/AccessibilityHelper.spec.js
@@ -58,6 +58,8 @@ test.describe("with single target", () => {
     test.describe("with multiple targets", () => {
         test("sets attributes when condition is true", async ({ page }) => {
             await page.click(".mx-name-actionButton2");
+            await waitForMendixApp(page);
+            await expect(page.locator(".mx-name-actionButton2")).not.toHaveAttribute("data-disabled", "true");
             await page.click(".mx-name-actionButton2");
             await waitForMendixApp(page);
             await page.click(".mx-name-radioButtons2 input:first-child");
@@ -89,6 +91,8 @@ test.describe("with single target", () => {
 
         test("updates target attributes using a NF", async ({ page }) => {
             await page.click(".mx-name-actionButton2");
+            await waitForMendixApp(page);
+            await expect(page.locator(".mx-name-actionButton2")).not.toHaveAttribute("data-disabled", "true");
             await page.click(".mx-name-actionButton2");
             await waitForMendixApp(page);
             await page.click(".mx-name-radioButtons2 input:first-child");
@@ -118,6 +122,8 @@ test.describe("with single target", () => {
             page
         }) => {
             await page.click(".mx-name-actionButton2");
+            await waitForMendixApp(page);
+            await expect(page.locator(".mx-name-actionButton2")).not.toHaveAttribute("data-disabled", "true");
             await page.click(".mx-name-actionButton2");
             await waitForMendixApp(page);
             await page.click(".mx-name-radioButtons2 input:first-child");

From 13c48e7c7a6e0414ba0228d54f0148de8f2f4f8c Mon Sep 17 00:00:00 2001
From: Samuel Reichert <samuelreichertt@gmail.com>
Date: Thu, 28 May 2026 15:14:49 +0200
Subject: [PATCH 3/6] test(accessibility-helper): increase timeout for
 actionButton2 disabled-state assertions

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../e2e/AccessibilityHelper.spec.js                  | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/packages/pluggableWidgets/accessibility-helper-web/e2e/AccessibilityHelper.spec.js b/packages/pluggableWidgets/accessibility-helper-web/e2e/AccessibilityHelper.spec.js
index e9318c3f75..495b35ca9f 100644
--- a/packages/pluggableWidgets/accessibility-helper-web/e2e/AccessibilityHelper.spec.js
+++ b/packages/pluggableWidgets/accessibility-helper-web/e2e/AccessibilityHelper.spec.js
@@ -59,7 +59,9 @@ test.describe("with single target", () => {
         test("sets attributes when condition is true", async ({ page }) => {
             await page.click(".mx-name-actionButton2");
             await waitForMendixApp(page);
-            await expect(page.locator(".mx-name-actionButton2")).not.toHaveAttribute("data-disabled", "true");
+            await expect(page.locator(".mx-name-actionButton2")).not.toHaveAttribute("data-disabled", "true", {
+                timeout: 10000
+            });
             await page.click(".mx-name-actionButton2");
             await waitForMendixApp(page);
             await page.click(".mx-name-radioButtons2 input:first-child");
@@ -92,7 +94,9 @@ test.describe("with single target", () => {
         test("updates target attributes using a NF", async ({ page }) => {
             await page.click(".mx-name-actionButton2");
             await waitForMendixApp(page);
-            await expect(page.locator(".mx-name-actionButton2")).not.toHaveAttribute("data-disabled", "true");
+            await expect(page.locator(".mx-name-actionButton2")).not.toHaveAttribute("data-disabled", "true", {
+                timeout: 10000
+            });
             await page.click(".mx-name-actionButton2");
             await waitForMendixApp(page);
             await page.click(".mx-name-radioButtons2 input:first-child");
@@ -123,7 +127,9 @@ test.describe("with single target", () => {
         }) => {
             await page.click(".mx-name-actionButton2");
             await waitForMendixApp(page);
-            await expect(page.locator(".mx-name-actionButton2")).not.toHaveAttribute("data-disabled", "true");
+            await expect(page.locator(".mx-name-actionButton2")).not.toHaveAttribute("data-disabled", "true", {
+                timeout: 10000
+            });
             await page.click(".mx-name-actionButton2");
             await waitForMendixApp(page);
             await page.click(".mx-name-radioButtons2 input:first-child");

From 06ca11fa411e3ac8d92cfe9c7b4f74ff06c049e6 Mon Sep 17 00:00:00 2001
From: Samuel Reichert <samuelreichertt@gmail.com>
Date: Fri, 29 May 2026 11:29:38 +0200
Subject: [PATCH 4/6] test(accessibility-helper): remove spurious double-click
 and disabled-state assertion
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

actionButton2 stays data-disabled after its action runs by design — asserting
it re-enables was semantically wrong and caused permanent failures. All six
multiple-targets tests now use single-click + waitForMendixApp, matching the
proven pattern from the three passing tests.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../e2e/AccessibilityHelper.spec.js               | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/packages/pluggableWidgets/accessibility-helper-web/e2e/AccessibilityHelper.spec.js b/packages/pluggableWidgets/accessibility-helper-web/e2e/AccessibilityHelper.spec.js
index 495b35ca9f..3588a2cfa0 100644
--- a/packages/pluggableWidgets/accessibility-helper-web/e2e/AccessibilityHelper.spec.js
+++ b/packages/pluggableWidgets/accessibility-helper-web/e2e/AccessibilityHelper.spec.js
@@ -57,11 +57,6 @@ test.describe("with single target", () => {
     });
     test.describe("with multiple targets", () => {
         test("sets attributes when condition is true", async ({ page }) => {
-            await page.click(".mx-name-actionButton2");
-            await waitForMendixApp(page);
-            await expect(page.locator(".mx-name-actionButton2")).not.toHaveAttribute("data-disabled", "true", {
-                timeout: 10000
-            });
             await page.click(".mx-name-actionButton2");
             await waitForMendixApp(page);
             await page.click(".mx-name-radioButtons2 input:first-child");
@@ -92,11 +87,6 @@ test.describe("with single target", () => {
         });
 
         test("updates target attributes using a NF", async ({ page }) => {
-            await page.click(".mx-name-actionButton2");
-            await waitForMendixApp(page);
-            await expect(page.locator(".mx-name-actionButton2")).not.toHaveAttribute("data-disabled", "true", {
-                timeout: 10000
-            });
             await page.click(".mx-name-actionButton2");
             await waitForMendixApp(page);
             await page.click(".mx-name-radioButtons2 input:first-child");
@@ -125,11 +115,6 @@ test.describe("with single target", () => {
         test("sets target attributes even though target is conditionally shown after being hidden", async ({
             page
         }) => {
-            await page.click(".mx-name-actionButton2");
-            await waitForMendixApp(page);
-            await expect(page.locator(".mx-name-actionButton2")).not.toHaveAttribute("data-disabled", "true", {
-                timeout: 10000
-            });
             await page.click(".mx-name-actionButton2");
             await waitForMendixApp(page);
             await page.click(".mx-name-radioButtons2 input:first-child");

From e89192df9931b2e45f9c13c4a75295a5552531af Mon Sep 17 00:00:00 2001
From: Samuel Reichert <samuelreichertt@gmail.com>
Date: Fri, 29 May 2026 12:37:19 +0200
Subject: [PATCH 5/6] test(video-player): wait for layout to stabilize before
 taking screenshot of external video poster

---
 .../pluggableWidgets/video-player-web/e2e/VideoPlayer.spec.js   | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/packages/pluggableWidgets/video-player-web/e2e/VideoPlayer.spec.js b/packages/pluggableWidgets/video-player-web/e2e/VideoPlayer.spec.js
index 78d72d2223..5991b77a55 100644
--- a/packages/pluggableWidgets/video-player-web/e2e/VideoPlayer.spec.js
+++ b/packages/pluggableWidgets/video-player-web/e2e/VideoPlayer.spec.js
@@ -129,6 +129,8 @@ test.describe("External video", () => {
                   })
                 : Promise.resolve()
         );
+        // Wait two animation frames so the browser flushes layout and paints the poster frame.
+        await page.evaluate(() => new Promise(r => requestAnimationFrame(() => requestAnimationFrame(r))));
         await expect(widget).toHaveScreenshot("videoPlayerExternalPoster.png");
     });
 

From 84e9b4649b44c6e52d91264effeb93c1a1a620b6 Mon Sep 17 00:00:00 2001
From: Samuel Reichert <samuelreichertt@gmail.com>
Date: Fri, 29 May 2026 15:10:36 +0200
Subject: [PATCH 6/6] test(video-player): add waitFrames helper to improve
 animation frame handling

---
 automation/run-e2e/lib/mendix-helpers.mjs                   | 6 ++++++
 .../video-player-web/e2e/VideoPlayer.spec.js                | 4 ++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/automation/run-e2e/lib/mendix-helpers.mjs b/automation/run-e2e/lib/mendix-helpers.mjs
index b558f3ee04..fa7a3ac1d3 100644
--- a/automation/run-e2e/lib/mendix-helpers.mjs
+++ b/automation/run-e2e/lib/mendix-helpers.mjs
@@ -41,6 +41,12 @@ export async function navigateToPage(page, path, timeout = 30_000) {
     await waitForMendixApp(page, timeout);
 }
 
+export async function waitFrames(page, n = 2) {
+    for (let i = 0; i < n; i++) {
+        await page.evaluate(() => new Promise(r => requestAnimationFrame(r)));
+    }
+}
+
 export async function checkAccessibility(page, selector, options = {}) {
     const AxeBuilder = (await import("@axe-core/playwright")).default;
     let builder = new AxeBuilder({ page }).withTags(options.tags || ["wcag21aa"]);
diff --git a/packages/pluggableWidgets/video-player-web/e2e/VideoPlayer.spec.js b/packages/pluggableWidgets/video-player-web/e2e/VideoPlayer.spec.js
index 5991b77a55..bcd66d23ed 100644
--- a/packages/pluggableWidgets/video-player-web/e2e/VideoPlayer.spec.js
+++ b/packages/pluggableWidgets/video-player-web/e2e/VideoPlayer.spec.js
@@ -1,5 +1,5 @@
 import { test, expect } from "@mendix/run-e2e/fixtures";
-import { waitForWidget } from "@mendix/run-e2e/mendix-helpers";
+import { waitForWidget, waitFrames } from "@mendix/run-e2e/mendix-helpers";
 
 test.describe("Video Player", () => {
     test.beforeEach(async ({ page }) => {
@@ -130,7 +130,7 @@ test.describe("External video", () => {
                 : Promise.resolve()
         );
         // Wait two animation frames so the browser flushes layout and paints the poster frame.
-        await page.evaluate(() => new Promise(r => requestAnimationFrame(() => requestAnimationFrame(r))));
+        await waitFrames(page, 2);
         await expect(widget).toHaveScreenshot("videoPlayerExternalPoster.png");
     });