[Discover] Unskip main functional tests (#84300) (#84536)

elastic · Nov 30, 2020 · f1ac252 · f1ac252
1 parent 7e2c65b
commit f1ac252
Showing 1 changed file with 10 additions and 10 deletions.
diff --git a/test/functional/apps/discover/_discover.js b/test/functional/apps/discover/_discover.js
@@ -32,17 +32,14 @@ export default function ({ getService, getPageObjects }) {
     defaultIndex: 'logstash-*',
   };
 
-  // Failing: See https://github.com/elastic/kibana/issues/82915
-  describe.skip('discover app', function describeIndexTests() {
+  describe('discover test', function describeIndexTests() {
     before(async function () {
-      // delete .kibana index and update configDoc
-      await kibanaServer.uiSettings.replace(defaultSettings);
-
       log.debug('load kibana index with default index pattern');
       await esArchiver.load('discover');
 
       // and load a set of makelogs data
       await esArchiver.loadIfNeeded('logstash_functional');
+      await kibanaServer.uiSettings.replace(defaultSettings);
       log.debug('discover');
       await PageObjects.common.navigateToApp('discover');
       await PageObjects.timePicker.setDefaultAbsoluteRange();
@@ -340,10 +337,13 @@ export default function ({ getService, getPageObjects }) {
 
         const getRequestTimestamp = async () => {
           const requestStats = await inspector.getTableData();
-          const requestTimestamp = requestStats.filter((r) =>
-            r[0].includes('Request timestamp')
-          )[0][1];
-          return requestTimestamp;
+          const requestStatsRow = requestStats.filter(
+            (r) => r && r[0] && r[0].includes('Request timestamp')
+          );
+          if (!requestStatsRow || !requestStatsRow[0] || !requestStatsRow[0][1]) {
+            return '';
+          }
+          return requestStatsRow[0][1];
         };
 
         const requestTimestampBefore = await getRequestTimestamp();
@@ -352,7 +352,7 @@ export default function ({ getService, getPageObjects }) {
           log.debug(
             `Timestamp before: ${requestTimestampBefore}, Timestamp after: ${requestTimestampAfter}`
           );
-          return requestTimestampBefore !== requestTimestampAfter;
+          return requestTimestampAfter && requestTimestampBefore !== requestTimestampAfter;
         });
       });