re-enable tests. retry on fail (#72061)

elastic · Jul 17, 2020 · 260eb13 · 260eb13
1 parent ac9178a
commit 260eb13
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/...nalytics/pages/analytics_creation/components/configuration_step/analysis_fields_table.tsx b/...nalytics/pages/analytics_creation/components/configuration_step/analysis_fields_table.tsx
@@ -172,6 +172,7 @@ export const AnalysisFieldsTable: FC<{
  return (
  <Fragment>
  <EuiFormRow
+ data-test-subj="mlAnalyticsCreateJobWizardIncludesTable"
  label={i18n.translate('xpack.ml.dataframe.analytics.create.includedFieldsLabel', {
  defaultMessage: 'Included fields',
  })}

diff --git a/x-pack/test/functional/apps/ml/data_frame_analytics/index.ts b/x-pack/test/functional/apps/ml/data_frame_analytics/index.ts
@@ -6,8 +6,7 @@
 import { FtrProviderContext } from '../../../ftr_provider_context';
 
 export default function ({ loadTestFile }: FtrProviderContext) {
- // flaky tests
- describe.skip('data frame analytics', function () {
+ describe('data frame analytics', function () {
  this.tags(['mlqa', 'skipFirefox']);
 
  loadTestFile(require.resolve('./outlier_detection_creation'));

diff --git a/x-pack/test/functional/services/ml/data_frame_analytics_creation.ts b/x-pack/test/functional/services/ml/data_frame_analytics_creation.ts
@@ -181,7 +181,11 @@ export function MachineLearningDataFrameAnalyticsCreationProvider(
  },
 
  async assertIncludeFieldsSelectionExists() {
- await testSubjects.existOrFail('mlAnalyticsCreateJobWizardIncludesSelect', { timeout: 5000 });
+ await testSubjects.existOrFail('mlAnalyticsCreateJobWizardIncludesTable', { timeout: 8000 });
+
+ await retry.tryForTime(8000, async () => {
+ await testSubjects.existOrFail('mlAnalyticsCreateJobWizardIncludesSelect');
+ });
  },
 
  // async assertIncludedFieldsSelection(expectedSelection: string[]) {