PRO-16178 fix: use lexical steps stack via AsyncLocalStorage

Author: uid11

README.md

@@ -333,7 +333,7 @@ If the mapping returns `undefined`, the log entry is not skipped, but is printed
 For example, if it is equal to three, the test will be run no more than three times.
 
 `navigationTimeout: number`: default timeout for navigation to url
-(`navigateToPage`, `navigateToUrl` actions) in milliseconds.
+(`navigateToUrl`, `setHeadersAndNavigateToUrl` actions) in milliseconds.
 
 `overriddenConfigFields: PlaywrightTestConfig | null`: if not `null`, then this value will override
 fields of internal `Playwright` config.

autotests/pageObjects/pages/E2edReportExample/E2edReportExample.ts

@@ -21,6 +21,11 @@ type CustomPageParams =
  * The e2ed report example page.
  */
 export class E2edReportExample extends Page<CustomPageParams> {
+  /**
+   * Page navigation timeout.
+   */
+  static override readonly navigationTimeout = 5_000;
+
   /**
    * Page header.
    */
@@ -47,11 +52,6 @@ export class E2edReportExample extends Page<CustomPageParams> {
   readonly navigationRetriesButtonSelected: Selector =
     this.navigationRetriesButton.filterByLocatorParameter('selected', 'true');
 
-  /**
-   * Page navigation timeout.
-   */
-  override readonly navigationTimeout = 5_000;
-
   /**
    * Cookies that we set (additionally) on a page before navigating to it.
    */

autotests/pageObjects/pages/Main.ts

@@ -15,6 +15,11 @@ type CustomPageParams = Partial<RouteParams> | undefined;
  * The Main (index) page.
  */
 export class Main extends Page<CustomPageParams> {
+  /**
+   * Page navigation timeout.
+   */
+  static override readonly navigationTimeout = 12_000;
+
   /**
    * Body selector.
    */

…/tests/switchingPagesForRequests.skip.ts …tests/tests/switchingPagesForRequests.tsautotests/tests/switchingPagesForRequests.skip.ts renamed to autotests/tests/switchingPagesForRequests.ts autotests/tests/switchingPagesForRequests.skip.ts renamed to autotests/tests/switchingPagesForRequests.ts

@@ -24,6 +24,11 @@ import type {
  * Abstract page with base methods.
  */
 export abstract class Page<PageParams = undefined> {
+  /**
+   * Default timeout for navigation to url (`navigateToPage`, `navigateToUrl` actions) in milliseconds.
+   */
+  static readonly navigationTimeout: number = 8_000;
+
   /**
    * Type of page parameters.
    */
@@ -40,12 +45,6 @@ export abstract class Page<PageParams = undefined> {
    */
   readonly maxIntervalBetweenRequestsInMs: number;
 
-  /**
-   * Default timeout for navigation to url (`navigateToPage`, `navigateToUrl` actions) in milliseconds.
-   * The default value is taken from the corresponding field of the pack config.
-   */
-  readonly navigationTimeout: number;
-
   /**
    * Immutable page parameters.
    */
@@ -67,12 +66,10 @@ export abstract class Page<PageParams = undefined> {
     this.pageParams = pageParams as PageParams;
 
     const {
-      navigationTimeout,
       waitForAllRequestsComplete: {maxIntervalBetweenRequestsInMs},
     } = getFullPackConfig();
 
     this.maxIntervalBetweenRequestsInMs = maxIntervalBetweenRequestsInMs;
-    this.navigationTimeout = navigationTimeout;
   }
 
   /**
@@ -132,7 +129,7 @@ export abstract class Page<PageParams = undefined> {
   ): Promise<NavigationReturn> {
     const navigationReturn = await navigateToUrl(url, {
       skipLogs: true,
-      timeout: this.navigationTimeout,
+      timeout: (this.constructor as typeof Page).navigationTimeout,
       ...options,
     });
     const {statusCode} = navigationReturn;

…tests/switchingPagesForResponses.skip.ts …ests/tests/switchingPagesForResponses.tsautotests/tests/switchingPagesForResponses.skip.ts renamed to autotests/tests/switchingPagesForResponses.ts autotests/tests/switchingPagesForResponses.skip.ts renamed to autotests/tests/switchingPagesForResponses.ts

@@ -23,33 +23,35 @@ Modules in the dependency graph should only import the modules above them:
 16. `utils/error`
 17. `utils/asserts`
 18. `utils/object`
-19. `utils/uiMode`
-20. `utils/runLabel`
-21. `utils/clone`
-22. `utils/notIncludedInPackTests`
-23. `utils/userland`
-24. `utils/fn`
-25. `utils/environment`
-26. `utils/packCompiler`
-27. `config`
-28. `utils/config`
-29. `utils/generalLog`
-30. `utils/testFilePaths`
-31. `utils/exit`
-32. `utils/promise`
-33. `utils/resourceUsage`
-34. `utils/fs`
-35. `utils/getGlobalErrorHandler`
-36. `utils/tests`
-37. `utils/end`
-38. `utils/pack`
-39. `useContext`
-40. `context`
-41. `utils/apiStatistics`
-42. `utils/selectors`
-43. `selectors`
-44. `utils/log`
-45. `utils/waitForEvents`
-46. `utils/expect`
-47. `expect`
-48. ...
+19. `utils/step`
+20. `utils/uiMode`
+21. `utils/runLabel`
+22. `utils/clone`
+23. `utils/notIncludedInPackTests`
+24. `utils/userland`
+25. `utils/fn`
+26. `utils/environment`
+27. `utils/packCompiler`
+28. `config`
+29. `utils/config`
+30. `utils/generalLog`
+31. `utils/testFilePaths`
+32. `utils/exit`
+33. `utils/promise`
+34. `utils/resourceUsage`
+35. `utils/fs`
+36. `utils/getGlobalErrorHandler`
+37. `utils/tests`
+38. `utils/end`
+39. `utils/pack`
+40. `useContext`
+41. `context`
+42. `utils/apiStatistics`
+43. `utils/selectors`
+44. `selectors`
+45. `utils/log`
+46. `step`
+47. `utils/waitForEvents`
+48. `utils/expect`
+49. `expect`
+50. ...

src/Page.ts

@@ -1,4 +1,4 @@
-import {LogEventStatus, LogEventType} from '../../constants/internal';
+import {ADDITIONAL_STEP_TIMEOUT, LogEventStatus, LogEventType} from '../../constants/internal';
 import {step} from '../../step';
 import {assertValueIsDefined} from '../../utils/asserts';
 import {getDocumentUrl} from '../../utils/document';
@@ -38,7 +38,11 @@ export const assertPage = async <SomePageClass extends AnyPageClassType>(
 
       return {documentUrl, isMatch, logEventStatus, routeParams};
     },
-    {payload: {pageParams}, type: LogEventType.InternalAction},
+    {
+      payload: {pageParams},
+      timeout: PageClass.navigationTimeout + ADDITIONAL_STEP_TIMEOUT,
+      type: LogEventType.InternalAction,
+    },
   );
 
   assertValueIsDefined(page, 'page is defined', {name: PageClass.name, pageParams});

src/README.md

@@ -1,4 +1,4 @@
-import {LogEventType} from '../../constants/internal';
+import {ADDITIONAL_STEP_TIMEOUT, LogEventType} from '../../constants/internal';
 import {step} from '../../step';
 import {addPageToApiStatistics} from '../../utils/apiStatistics';
 import {assertValueIsDefined} from '../../utils/asserts';
@@ -56,7 +56,11 @@ export const navigateToPage = async <SomePageClass extends AnyPageClassType>(
 
       return {documentUrl, isMatch, pageInstanceCreatedInMs, routeParams, url};
     },
-    {payload: {pageParams}, type: LogEventType.InternalAction},
+    {
+      payload: {pageParams},
+      timeout: PageClass.navigationTimeout + ADDITIONAL_STEP_TIMEOUT,
+      type: LogEventType.InternalAction,
+    },
   );
 
   assertValueIsDefined(page, 'page is defined', {name: PageClass.name, pageParams});