@camunda/e2e-test-suite 0.0.1040 → 0.0.1041

package/dist/pages/8.10/UtilitiesPage.js

@@ -100,8 +100,19 @@ async function waitForLoginForm(page, loginPage, totalTimeout = 90000, roundTime
     }
     throw new Error(`Auth0 login form did not render within ${totalTimeout}ms: ${lastState}`);
 }
+// Tracks which worker-derived `timeout` values have already paid the
+// stagger sleep below, so a worker only pays it once per process lifetime
+// rather than on every call -- see the comment at the call site.
+const staggeredWorkerTimeouts = new Set();
 async function loginWithRetry(page, loginPage, testUser, timeout, maxRetries = 5) {
     let lastError;
+    // `timeout` is a stable per-worker identifier (derived from workerIndex),
+    // and a worker is a long-lived process across every test/repeat it runs, so
+    // the stagger below only needs to happen for that worker's first call in
+    // the whole job. A --repeat-each validation pass otherwise pays it on
+    // every repeat instead of just the first (team-test-automation#78).
+    const isWorkersFirstCall = !staggeredWorkerTimeouts.has(timeout);
+    staggeredWorkerTimeouts.add(timeout);
     for (let attempt = 0; attempt < maxRetries; attempt++) {
         try {
             await page.context().clearCookies();
@@ -150,7 +161,9 @@ async function loginWithRetry(page, loginPage, testUser, timeout, maxRetries = 5
             await page
                 .waitForLoadState('networkidle', { timeout: 30000 })
                 .catch(() => { });
-            await (0, sleep_1.sleep)(timeout);
+            if (isWorkersFirstCall) {
+                await (0, sleep_1.sleep)(timeout);
+            }
             await waitForLoginForm(page, loginPage);
             await loginPage.loginWithTestUser(testUser);
             return;

package/dist/pages/8.7/UtilitiesPage.js

@@ -7,11 +7,28 @@ const HomePage_1 = require("./HomePage");
 const sleep_1 = require("../../utils/sleep");
 const fileUpload_1 = require("../../utils/fileUpload");
 const mailSlurpClient_1 = require("../../utils/mailSlurpClient");
+// Tracks which worker-derived `timeout` values have already paid the
+// pre-flight stagger below, so a worker only pays it once per process
+// lifetime rather than on every call -- see the comment at the call sites.
+const staggeredWorkerTimeouts = new Set();
 async function loginWithRetry(page, loginPage, testUser, timeout, maxRetries = 5, skipOrgAssertion = false) {
+    // Callers derive `timeout` as (workerIndex + 1) * 1000, so it's a stable
+    // per-worker identifier: a Playwright worker is a long-lived process that
+    // calls loginWithRetry once per test/repeat it runs, and this stagger only
+    // needs to happen for that worker's FIRST call in the whole job -- by the
+    // time it calls again, it's no longer starting simultaneously with anyone
+    // else's first call, which is the only scenario the stagger protects
+    // against. Without this, a --repeat-each validation pass pays the full
+    // stagger on every single repeat instead of once, which is pure dead time
+    // on the critical path (team-test-automation#78).
+    const isWorkersFirstCall = !staggeredWorkerTimeouts.has(timeout);
+    staggeredWorkerTimeouts.add(timeout);
     // Pre-flight worker stagger: spread the initial Auth0 hits across parallel
     // workers BEFORE the first goto, otherwise all workers slam the tenant at
     // the same instant and several get throttled into the retry loop.
-    await (0, sleep_1.sleep)(timeout);
+    if (isWorkersFirstCall) {
+        await (0, sleep_1.sleep)(timeout);
+    }
     for (let attempt = 0; attempt < maxRetries; attempt++) {
         try {
             await page.context().clearCookies();
@@ -36,7 +53,9 @@ async function loginWithRetry(page, loginPage, testUser, timeout, maxRetries = 5
             await page
                 .waitForLoadState('networkidle', { timeout: 30000 })
                 .catch(() => { });
-            await (0, sleep_1.sleep)(timeout);
+            if (isWorkersFirstCall) {
+                await (0, sleep_1.sleep)(timeout);
+            }
             // Always detect SSO re-auth: after a logout + clearCookies, Auth0 may
             // silently log the previous user back in without showing the login form.
             // Force logout until the Auth0 email input appears.

package/dist/pages/8.8/UtilitiesPage.js

@@ -32,14 +32,26 @@ async function forceLogoutIfNeeded(page) {
     }
 }
 exports.forceLogoutIfNeeded = forceLogoutIfNeeded;
+// Tracks which worker-derived `timeout` values have already paid the
+// pre-flight stagger below, so a worker only pays it once per process
+// lifetime rather than on every call -- see the comment at the call site.
+const staggeredWorkerTimeouts = new Set();
 async function loginWithRetry(page, loginPage, testUser, timeout, maxRetries = 5) {
     let lastError;
     // Stagger the first navigation across parallel workers. Without this, every
     // worker's initial page.goto('/') fires within the same second -- with 18+
     // workers logging in at once, that burst is enough to make the navigation
     // itself time out well before any login form ever renders. Retries already
-    // back off via randomSleep below, so this only needs to run once.
-    await (0, sleep_1.sleep)(timeout);
+    // back off via randomSleep below, so this only needs to run once -- and,
+    // since `timeout` is a stable per-worker identifier (derived from
+    // workerIndex) and a worker is a long-lived process across every
+    // test/repeat it runs, "once" means once per worker for the whole job, not
+    // once per call. A --repeat-each validation pass otherwise pays this on
+    // every repeat instead of just the first (team-test-automation#78).
+    if (!staggeredWorkerTimeouts.has(timeout)) {
+        staggeredWorkerTimeouts.add(timeout);
+        await (0, sleep_1.sleep)(timeout);
+    }
     for (let attempt = 0; attempt < maxRetries; attempt++) {
         try {
             await page.context().clearCookies();

package/dist/pages/8.9/UtilitiesPage.js

@@ -8,8 +8,19 @@ const sleep_1 = require("../../utils/sleep");
 const randomSleep_1 = require("../../utils/randomSleep");
 const fileUpload_1 = require("../../utils/fileUpload");
 const mailSlurpClient_1 = require("../../utils/mailSlurpClient");
+// Tracks which worker-derived `timeout` values have already paid the
+// stagger sleep below, so a worker only pays it once per process lifetime
+// rather than on every call -- see the comment at the call site.
+const staggeredWorkerTimeouts = new Set();
 async function loginWithRetry(page, loginPage, testUser, timeout, maxRetries = 5) {
     let lastError;
+    // `timeout` is a stable per-worker identifier (derived from workerIndex),
+    // and a worker is a long-lived process across every test/repeat it runs, so
+    // the stagger below only needs to happen for that worker's first call in
+    // the whole job. A --repeat-each validation pass otherwise pays it on
+    // every repeat instead of just the first (team-test-automation#78).
+    const isWorkersFirstCall = !staggeredWorkerTimeouts.has(timeout);
+    staggeredWorkerTimeouts.add(timeout);
     for (let attempt = 0; attempt < maxRetries; attempt++) {
         try {
             await page.context().clearCookies();
@@ -34,7 +45,9 @@ async function loginWithRetry(page, loginPage, testUser, timeout, maxRetries = 5
             await page
                 .waitForLoadState('networkidle', { timeout: 30000 })
                 .catch(() => { });
-            await (0, sleep_1.sleep)(timeout);
+            if (isWorkersFirstCall) {
+                await (0, sleep_1.sleep)(timeout);
+            }
             // The identity provider occasionally returns a generic error / rate
             // limit page that has none of the usual auth controls. Bail out early
             // when that page is visible so the outer retry can back off.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@camunda/e2e-test-suite",
-  "version": "0.0.1040",
+  "version": "0.0.1041",
   "description": "End-to-end test helpers for Camunda 8",
   "repository": {
     "type": "git",