fix(6562): update explorer to work with iox (#6584)

* test: add flags for cloud explorer test

* test: add flags and testID for explorer queries graphs test

* chore: run prettier

Author: appletreeisyellow

cypress/e2e/cloud/explorer.test.ts

@@ -14,18 +14,23 @@ function getTimeMachineText() {
     .invoke('text')
 }
 
-describe.skip('DataExplorer', () => {
+describe('DataExplorer', () => {
   beforeEach(() => {
-    cy.flush()
-    cy.signin().then(() => {
-      cy.get('@org').then(({id}: Organization) => {
-        cy.createMapVariable(id)
-        cy.fixture('routes').then(({orgs, explorer}) => {
-          cy.visit(`${orgs}/${id}${explorer}`)
-          cy.getByTestID('tree-nav').should('be.visible')
-        })
+    cy.flush().then(() =>
+      cy.signin().then(() => {
+        cy.setFeatureFlags({
+          showOldDataExplorerInNewIOx: true,
+        }).then(() =>
+          cy.get('@org').then(({id}: Organization) => {
+            cy.createMapVariable(id)
+            cy.fixture('routes').then(({orgs, explorer}) => {
+              cy.visit(`${orgs}/${id}${explorer}`)
+              cy.getByTestID('tree-nav').should('be.visible')
+            })
+          })
+        )
       })
-    })
+    )
   })
 
   describe('Script Editor', () => {

cypress/e2e/shared/explorer.queries.graphs.test.ts

@@ -1,26 +1,33 @@
 import {Organization} from '../../../src/types'
 
-describe.skip('writing queries and making graphs using Data Explorer', () => {
+describe('writing queries and making graphs using Data Explorer', () => {
   let route: string
 
   beforeEach(() => {
-    cy.flush()
-    cy.signin()
-    // Double check that the new schemaComposition flag does not interfere.
-    cy.setFeatureFlags({
-      schemaComposition: true,
-    })
-    // cy.wait($time) is necessary to consistently ensure sufficient time for the feature flag override.
-    // The flag reset happens via redux, (it's not a network request), so we can't cy.wait($intercepted_route).
-    cy.wait(1200)
-    cy.get('@org').then(({id}: Organization) => {
-      cy.createMapVariable(id)
-      cy.fixture('routes').then(({orgs, explorer}) => {
-        route = `${orgs}/${id}${explorer}`
-        cy.visit(route)
-        cy.getByTestID('tree-nav').should('be.visible')
-      })
-    })
+    cy.flush().then(() =>
+      cy.signin().then(() =>
+        cy
+          .setFeatureFlags({
+            showOldDataExplorerInNewIOx: true,
+            showTasksInNewIOx: true,
+            showVariablesInNewIOx: true,
+            schemaComposition: true, // Double check that the new schemaComposition flag does not interfere.
+          })
+          .then(() => {
+            // cy.wait($time) is necessary to consistently ensure sufficient time for the feature flag override.
+            // The flag reset happens via redux, (it's not a network request), so we can't cy.wait($intercepted_route).
+            cy.wait(1200)
+            cy.get('@org').then(({id}: Organization) => {
+              cy.createMapVariable(id)
+              cy.fixture('routes').then(({orgs, explorer}) => {
+                route = `${orgs}/${id}${explorer}`
+                cy.visit(route)
+                cy.getByTestID('tree-nav').should('be.visible')
+              })
+            })
+          })
+      )
+    )
   })
 
   describe('numeric input in graphs', () => {
@@ -169,13 +176,6 @@ describe.skip('writing queries and making graphs using Data Explorer', () => {
     })
 
     it('shows the empty state when the query returns no results', () => {
-      cy.isIoxOrg().then(isIox => {
-        // iox uses `${orgId}_${bucketId}` for a namespace_id
-        // And gives a namespace_id failure if no data is written yet.
-        // https://github.com/influxdata/monitor-ci/issues/402#issuecomment-1362368473
-        cy.skipOn(isIox)
-      })
-
       cy.getByTestID('time-machine--bottom').within(() => {
         cy.getByTestID('flux-editor').should('be.visible')
           .monacoType(`from(bucket: "defbuck")

src/timeMachine/components/TagSelector.tsx

@@ -260,7 +260,7 @@ class TagSelector extends PureComponent<Props> {
 
     if (valuesStatus === RemoteDataState.Done && !values.length) {
       return (
-        <BuilderCard.Empty>
+        <BuilderCard.Empty testID="empty-tag-keys">
           No values found <small>in the current time range</small>
         </BuilderCard.Empty>
       )