fix: automatic open the test with error from last retry in HTML report

autotests/tests/main/exists.ts

@@ -89,6 +89,7 @@ test('exists', {meta: {testId: '1'}, testIdleTimeout: 12_000, testTimeout: 20_00
 
   const successfulResponsePromise = waitForResponse(
     ({statusCode}) => statusCode === OK_STATUS_CODE,
+    {includeNavigationRequest: true},
   );
 
   await pressKey('Enter');

src/utils/report/client/chooseTestRun.tsx

@@ -1,15 +1,15 @@
-/* eslint-disable no-console */
-
 import {assertValueIsDefined as clientAssertValueIsDefined} from './assertValueIsDefined';
 import {
   MaybeApiStatistics as clientMaybeApiStatistics,
   TestRunDetails as clientTestRunDetails,
 } from './render';
+import {setDomElementsToClientState as clientSetDomElementsToClientState} from './setDomElementsToClientState';
 
 import type {ReportClientState, RunHash} from '../../../types/internal';
 
 const assertValueIsDefined: typeof clientAssertValueIsDefined = clientAssertValueIsDefined;
 const MaybeApiStatistics = clientMaybeApiStatistics;
+const setDomElementsToClientState = clientSetDomElementsToClientState;
 const TestRunDetails = clientTestRunDetails;
 
 declare const jsx: JSX.Runtime;
@@ -20,15 +20,13 @@ declare const reportClientState: ReportClientState;
  * This base client function should not use scope variables (except other base functions).
  * @internal
  */
-// eslint-disable-next-line max-statements
+// eslint-disable-next-line complexity, max-statements
 export const chooseTestRun = (runHash: RunHash): void => {
+  setDomElementsToClientState();
+
   const {e2edRightColumnContainer} = reportClientState;
 
   if (!e2edRightColumnContainer) {
-    console.error(
-      'Cannot find right column container (id="e2edRightColumnContainer"). Probably page not yet completely loaded. Please try click again later',
-    );
-
     return;
   }
 
@@ -46,6 +44,7 @@ export const chooseTestRun = (runHash: RunHash): void => {
     e2edRightColumnContainer.firstElementChild as HTMLElement | null;
 
   if (!previousTestRunDetailsElement) {
+    // eslint-disable-next-line no-console
     console.error(
       'Cannot find first child element in right column container (id="e2edRightColumnContainer"). Probably page not yet completely loaded. Please try click again later',
     );
@@ -77,6 +76,7 @@ export const chooseTestRun = (runHash: RunHash): void => {
     const fullTestRun = fullTestRuns.find((testRun) => testRun.runHash === runHash);
 
     if (fullTestRun === undefined) {
+      // eslint-disable-next-line no-console
       console.error(
         `Cannot find test run with hash ${runHash} in JSON report data. Probably JSON report data for this test run not yet loaded. Please try click again later`,
       );

src/utils/report/client/index.ts

@@ -56,4 +56,6 @@ export {
   sanitizeValue,
 } from './sanitizeHtml';
 /** @internal */
+export {setDomElementsToClientState} from './setDomElementsToClientState';
+/** @internal */
 export {setReadJsonReportDataObservers} from './setReadJsonReportDataObservers';

src/utils/report/client/onDomContentLoad.ts

@@ -1,29 +1,20 @@
 import {readJsonReportData as clientReadJsonReportData} from './readJsonReportData';
+import {setDomElementsToClientState as clientSetDomElementsToClientState} from './setDomElementsToClientState';
 
 import type {ReportClientState} from '../../../types/internal';
 
 declare const reportClientState: ReportClientState;
 
 const readJsonReportData = clientReadJsonReportData;
+const setDomElementsToClientState = clientSetDomElementsToClientState;
 
 /**
  * `DOMContentLoaded` handler for report page.
  * This client function should not use scope variables (except global functions).
  * @internal
  */
 export const onDomContentLoad = (): void => {
-  const e2edRightColumnContainer = document.getElementById('e2edRightColumnContainer') ?? undefined;
-
-  if (!e2edRightColumnContainer) {
-    // eslint-disable-next-line no-console
-    console.error(
-      'Cannot find right column container (id="e2edRightColumnContainer") after DOMContentLoaded.',
-    );
-  } else {
-    Object.assign<ReportClientState, Partial<ReportClientState>>(reportClientState, {
-      e2edRightColumnContainer,
-    });
-  }
+  setDomElementsToClientState({afterDomContentLoad: true});
 
   readJsonReportData(true);
 

src/utils/report/client/setDomElementsToClientState.ts

@@ -0,0 +1,35 @@
+import type {ReportClientState} from '../../../types/internal';
+
+declare const reportClientState: ReportClientState;
+
+type Options = Readonly<{afterDomContentLoad?: boolean}> | undefined;
+
+/**
+ * Set dynamic DOM elements to `reportClientState`.
+ * This client function should not use scope variables (except global functions).
+ * @internal
+ */
+export const setDomElementsToClientState = ({afterDomContentLoad = false}: Options = {}): void => {
+  let {e2edRightColumnContainer} = reportClientState;
+
+  if (e2edRightColumnContainer) {
+    return;
+  }
+
+  e2edRightColumnContainer = document.getElementById('e2edRightColumnContainer') ?? undefined;
+
+  if (e2edRightColumnContainer) {
+    Object.assign<ReportClientState, Partial<ReportClientState>>(reportClientState, {
+      e2edRightColumnContainer,
+    });
+
+    return;
+  }
+
+  const messageTail = afterDomContentLoad
+    ? ' after DOMContentLoaded'
+    : '. Probably page not yet completely loaded. Please try click again later';
+
+  // eslint-disable-next-line no-console
+  console.error(`Cannot find right column container (id="e2edRightColumnContainer")${messageTail}`);
+};