[ML] Functional tests - stabilize saved search tests (#59652)

pheyos · pheyos · commit cbf271e41f2a · 2020-03-10T18:11:58.000+01:00
This PR stabilizes the saved search functional UI tests by adding a retry to navigateToApp('ml')
diff --git a/x-pack/test/functional/apps/machine_learning/anomaly_detection/saved_search_job.ts b/x-pack/test/functional/apps/machine_learning/anomaly_detection/saved_search_job.ts
@@ -271,8 +271,7 @@ export default function({ getService }: FtrProviderContext) {
     },
   ];
 
-  // test failures, see #59354
-  describe.skip('saved search', function() {
+  describe('saved search', function() {
     this.tags(['smoke', 'mlqa']);
     before(async () => {
       await esArchiver.load('ml/farequote');
diff --git a/x-pack/test/functional/services/machine_learning/navigation.ts b/x-pack/test/functional/services/machine_learning/navigation.ts
@@ -17,7 +17,10 @@ export function MachineLearningNavigationProvider({
 
   return {
     async navigateToMl() {
-      return await PageObjects.common.navigateToApp('ml');
+      await retry.tryForTime(60 * 1000, async () => {
+        await PageObjects.common.navigateToApp('ml');
+        await testSubjects.existOrFail('mlPageOverview', { timeout: 2000 });
+      });
     },
 
     async assertTabsExist(tabTypeSubject: string, areaSubjects: string[]) {
