[Lens][FTR] Fix afterAll issue (#189069)

## Summary Fixes #189056, #189057, #164623 Bring back the window handler for other tests correctly. --------- Co-authored-by: kibanamachine <[email protected]>
elastic · Jul 25, 2024 · c47f371 · c47f371
1 parent a35d317
commit c47f371
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/x-pack/test/functional/apps/lens/group1/ad_hoc_data_view.ts b/x-pack/test/functional/apps/lens/group1/ad_hoc_data_view.ts
@@ -73,8 +73,7 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
     expect(await dataViews.isAdHoc()).to.be(true);
   };
 
-  // Failing: See https://github.com/elastic/kibana/issues/164623
-  describe.skip('lens ad hoc data view tests', () => {
+  describe('lens ad hoc data view tests', () => {
     it('should allow building a chart based on ad hoc data view', async () => {
       await setupAdHocDataView();
       await PageObjects.lens.configureDimension({
@@ -223,6 +222,8 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
       expect(await dataViews.isAdHoc()).to.be(true);
 
       await browser.closeCurrentWindow();
+      const [lensHandle] = await browser.getAllWindowHandles();
+      await browser.switchToWindow(lensHandle);
     });
 
     // Failing: See https://github.com/elastic/kibana/issues/164623

diff --git a/x-pack/test/functional/apps/lens/group1/index.ts b/x-pack/test/functional/apps/lens/group1/index.ts
@@ -17,8 +17,7 @@ export default ({ getService, loadTestFile, getPageObjects }: FtrProviderContext
   const config = getService('config');
   let remoteEsArchiver;
 
-  // FLAKY: https://github.com/elastic/kibana/issues/189057
-  describe.skip('lens app - group 1', () => {
+  describe('lens app - group 1', () => {
     const esArchive = 'x-pack/test/functional/es_archives/logstash_functional';
     const localIndexPatternString = 'logstash-*';
     const remoteIndexPatternString = 'ftr-remote:logstash-*';

diff --git a/x-pack/test/functional/apps/lens/group1/multiple_data_views.ts b/x-pack/test/functional/apps/lens/group1/multiple_data_views.ts
@@ -45,8 +45,7 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
     ).to.eql(expectedData);
   }
 
-  // Failing: See https://github.com/elastic/kibana/issues/189056
-  describe.skip('lens with multiple data views', () => {
+  describe('lens with multiple data views', () => {
     const visTitle = 'xyChart with multiple data views';
 
     before(async () => {