From 55acc06923086364fc06e4c355ccfdea46d4bc8c Mon Sep 17 00:00:00 2001 From: Quynh Nguyen Date: Thu, 27 Apr 2023 19:44:35 -0500 Subject: [PATCH] Stabilize creation_index_pattern tests --- .../index_pattern/creation_index_pattern.ts | 4 ++-- .../functional/services/ml/field_stats_flyout.ts | 15 ++++++++------- 2 files changed, 10 insertions(+), 9 deletions(-) diff --git a/x-pack/test/functional/apps/transform/creation/index_pattern/creation_index_pattern.ts b/x-pack/test/functional/apps/transform/creation/index_pattern/creation_index_pattern.ts index e5319cb4d60a4..69406c5830421 100644 --- a/x-pack/test/functional/apps/transform/creation/index_pattern/creation_index_pattern.ts +++ b/x-pack/test/functional/apps/transform/creation/index_pattern/creation_index_pattern.ts @@ -22,8 +22,7 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) { const transform = getService('transform'); const pageObjects = getPageObjects(['discover']); - // Failing: See https://github.com/elastic/kibana/issues/151889 - describe.skip('creation_index_pattern', function () { + describe('creation_index_pattern', function () { before(async () => { await esArchiver.loadIfNeeded('x-pack/test/functional/es_archives/ml/ecommerce'); await transform.testResources.createIndexPatternIfNeeded('ft_ecommerce', 'order_date'); @@ -35,6 +34,7 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) { after(async () => { await transform.api.cleanTransformIndices(); await transform.testResources.deleteIndexPatternByTitle('ft_ecommerce'); + await transform.securityUI.logout(); }); const fieldStatsEntries = [ diff --git a/x-pack/test/functional/services/ml/field_stats_flyout.ts b/x-pack/test/functional/services/ml/field_stats_flyout.ts index f039a5c7d7a68..5ef970e7839e3 100644 --- a/x-pack/test/functional/services/ml/field_stats_flyout.ts +++ b/x-pack/test/functional/services/ml/field_stats_flyout.ts @@ -70,11 +70,13 @@ export function MachineLearningFieldStatsFlyoutProvider({ getService }: FtrProvi ) { const selector = `~${testSubj} > ~mlInspectFieldStatsButton-${fieldName}`; - await retry.tryForTime(2000, async () => { + await retry.tryForTime(10 * 1000, async () => { await testSubjects.existOrFail(selector); await testSubjects.click(selector); - await testSubjects.existOrFail('mlFieldStatsFlyout'); - await testSubjects.existOrFail(`mlFieldStatsFlyoutContent ${fieldName}-title`); + await testSubjects.existOrFail('mlFieldStatsFlyout', { timeout: 500 }); + await testSubjects.existOrFail(`mlFieldStatsFlyoutContent ${fieldName}-title`, { + timeout: 500, + }); }); await this.assertFieldStatContentByType(testSubj, fieldName, fieldType); if (Array.isArray(expectedTopValuesContent)) { @@ -89,15 +91,14 @@ export function MachineLearningFieldStatsFlyoutProvider({ getService }: FtrProvi fieldType: 'keyword' | 'date' | 'number', expectedTopValuesContent?: string[] ) { - await retry.tryForTime(3000, async () => { - const selector = `mlInspectFieldStatsButton-${fieldName}`; + const selector = `mlInspectFieldStatsButton-${fieldName}`; + await retry.tryForTime(20 * 1000, async () => { const fieldTarget = await testSubjects.find(parentComboBoxSelector); await comboBox.openOptionsList(fieldTarget); await testSubjects.existOrFail(selector); await testSubjects.click(selector); - await testSubjects.existOrFail('mlFieldStatsFlyout'); - + await testSubjects.existOrFail('mlFieldStatsFlyout', { timeout: 500 }); await testSubjects.existOrFail(`mlFieldStatsFlyoutContent ${fieldName}-title`); await this.assertFieldStatContentByType(parentComboBoxSelector, fieldName, fieldType);