[Discover] Unskip context navigation flaky tests (#188811)

- Closes #188396
elastic · Jul 23, 2024 · 9a44464 · 9a44464
1 parent 6cfa2c0
commit 9a44464
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/test/functional/apps/context/_context_navigation.ts b/test/functional/apps/context/_context_navigation.ts
@@ -36,8 +36,7 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
     });
   };
 
-  // FLAKY: https://github.com/elastic/kibana/issues/188396
-  describe.skip('discover - context - back navigation', function contextSize() {
+  describe('discover - context - back navigation', function contextSize() {
     before(async () => {
       await PageObjects.timePicker.setDefaultAbsoluteRangeViaUiSettings();
       await kibanaServer.uiSettings.update({

diff --git a/test/functional/services/data_grid.ts b/test/functional/services/data_grid.ts
@@ -324,9 +324,9 @@ export class DataGridService extends FtrService {
     });
 
     if (toggle) {
+      await toggle.scrollIntoViewIfNecessary();
+      await toggle.click();
       await this.retry.waitFor('doc viewer to open', async () => {
-        await toggle!.scrollIntoViewIfNecessary();
-        await toggle!.click();
         return this.isShowingDocViewer();
       });
     } else {