Unskip Serverless Observability Tests / serverless observability UI n…

…avigation active sidenav section is auto opened on load (#162323) ## Summary close #162781 Simulating flaky test run with for loop: #162323 (comment)
elastic · Aug 22, 2023 · 3458d7b · 3458d7b
1 parent 79f8dcd
commit 3458d7b
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/x-pack/test_serverless/functional/page_objects/svl_common_navigation.ts b/x-pack/test_serverless/functional/page_objects/svl_common_navigation.ts
@@ -104,13 +104,17 @@ export function SvlCommonNavigationProvider(ctx: FtrProviderContext) {
  },
  async expectSectionOpen(sectionId: NavigationId) {
  await this.expectSectionExists(sectionId);
- const isOpen = await this.isSectionOpen(sectionId);
- expect(isOpen).to.be(true);
+ await retry.waitFor(`section ${sectionId} to be open`, async () => {
+ const isOpen = await this.isSectionOpen(sectionId);
+ return isOpen;
+ });
  },
  async expectSectionClosed(sectionId: NavigationId) {
  await this.expectSectionExists(sectionId);
- const isOpen = await this.isSectionOpen(sectionId);
- expect(isOpen).to.be(false);
+ await retry.waitFor(`section ${sectionId} to be closed`, async () => {
+ const isOpen = await this.isSectionOpen(sectionId);
+ return !isOpen;
+ });
  },
  async openSection(sectionId: NavigationId) {
  await this.expectSectionExists(sectionId);

diff --git a/x-pack/test_serverless/functional/test_suites/observability/navigation.ts b/x-pack/test_serverless/functional/test_suites/observability/navigation.ts
@@ -69,8 +69,7 @@ export default function ({ getPageObject, getService }: FtrProviderContext) {
  await expectNoPageReload();
  });
 
- // FLAKY/BUG?: https://github.com/elastic/kibana/issues/162781
- it.skip('active sidenav section is auto opened on load', async () => {
+ it('active sidenav section is auto opened on load', async () => {
  await svlCommonNavigation.sidenav.openSection('project_settings_project_nav');
  await svlCommonNavigation.sidenav.clickLink({ deepLinkId: 'management' });
  await browser.refresh();