Merge pull request #1374 from concord-consortium/188002783-investigat…

…e-flaky-table-tests

188002783 investigate flaky table tests

v3/cypress/e2e/adornments.spec.ts

@@ -5,10 +5,12 @@ import { ToolbarElements as toolbar } from "../support/elements/toolbar-elements
 
 context("Graph adornments", () => {
   beforeEach(function () {
+    cy.log('Starting test setup')
     const queryParams = "?sample=mammals&dashboard&mouseSensor"
     const url = `${Cypress.config("index")}${queryParams}`
     cy.visit(url)
     cy.wait(2500)
+    cy.log('Setup complete')
   })
 
   it("shows inspector palette when Display Values button is clicked", () => {

v3/cypress/e2e/attribute-types.spec.ts

@@ -3,11 +3,13 @@ import { CfmElements as cfm } from "../support/elements/cfm"
 
 context("attribute types", () => {
   beforeEach(() => {
+    cy.log('Starting test setup')
     const filename = "cypress/fixtures/attribute-types.codap"
     const url = `${Cypress.config("index")}`
     cy.visit(url)
     cy.wait(3000)
     cfm.openLocalDoc(filename)
+    cy.log('Setup complete')
   })
 
   describe("attribute types are rendered correctly", () => {

v3/cypress/e2e/axis.spec.ts

@@ -21,10 +21,12 @@ const arrayOfValues = [
 
 context("Test graph axes with various attribute types", () => {
   beforeEach(function () {
+    cy.log('Starting test setup')
     const queryParams = "?sample=mammals&dashboard&mouseSensor"
     const url = `${Cypress.config("index")}${queryParams}`
     cy.visit(url)
     cy.wait(2500)
+    cy.log('Setup complete')
   })
   it("will show default x axis label", () => {
     ah.verifyDefaultAxisLabel("bottom")

v3/cypress/e2e/bivariate-adornments.spec.ts

@@ -4,10 +4,12 @@ import { ToolbarElements as toolbar } from "../support/elements/toolbar-elements
 
 context("Graph adornments", () => {
   beforeEach(function () {
+    cy.log('Starting test setup')
     const queryParams = "?sample=mammals&dashboard&mouseSensor"
     const url = `${Cypress.config("index")}${queryParams}`
     cy.visit(url)
     cy.wait(2500)
+    cy.log('Setup complete')
   })
 
   it("adds a least squares line to graph when Least Squares Line checkbox is checked", () => {

v3/cypress/e2e/calculator.spec.ts

@@ -6,10 +6,12 @@ const calculatorName = "Calculator"
 
 context("Calculator", () => {
   beforeEach(function () {
+      cy.log('Starting test setup')
       const queryParams = "?sample=mammals&dashboard&mouseSensor"
       const url = `${Cypress.config("index")}${queryParams}`
       cy.visit(url)
       cy.wait(2500) // Ensuring the page and components are fully loaded.
+      cy.log('Setup complete')
   })
   it("populates default title", () => {
     c.getComponentTitle("calculator").should("contain", calculatorName)

v3/cypress/e2e/case-card.spec.ts

@@ -1,10 +1,12 @@
 
 beforeEach(() => {
   // cy.scrollTo() doesn't work as expected with `scroll-behavior: smooth`
+  cy.log('Starting test setup')
   const queryParams = "?sample=mammals&scrollBehavior=auto"
   const url = `${Cypress.config("index")}${queryParams}`
   cy.visit(url)
   cy.wait(2000)
+  cy.log('Setup complete')
 })
 
 context("case card", () => {

v3/cypress/e2e/cfm.spec.ts

@@ -4,10 +4,12 @@ import { TableTileElements as table } from "../support/elements/table-tile"
 
 context("CloudFileManager", () => {
   beforeEach(function () {
+    cy.log('Starting test setup')
     const queryParams = "?mouseSensor"
     const url = `${Cypress.config("index")}${queryParams}`
     cy.visit(url)
     cy.wait(2500)
+    cy.log('Setup complete')
   })
   it("Opens Mammals example document via CFM Open dialog", () => {
     // hamburger menu is hidden initially

v3/cypress/e2e/component.spec.ts

@@ -2,10 +2,12 @@ import { ComponentElements as c } from "../support/elements/component-elements"
 
 context("Component UI", () => {
   beforeEach(function () {
+    cy.log('Starting test setup')
     const queryParams = "?sample=mammals&dashboard&mouseSensor"
     const url = `${Cypress.config("index")}${queryParams}`
     cy.visit(url)
     cy.wait(2500)
+    cy.log('Setup complete')
   })
 
   it("moves components by dragging", () => {

v3/cypress/e2e/graph-legend.spec.ts

@@ -18,10 +18,12 @@ const arrayOfValues = [
 
 context("Test legend with various attribute types", () => {
   beforeEach(function () {
+    cy.log('Starting test setup')
     const queryParams = "?sample=mammals&dashboard&mouseSensor"
     const url = `${Cypress.config("index")}${queryParams}`
     cy.visit(url)
     cy.wait(2500)
+    cy.log('Setup complete')
   })
   it("will not draw legend if plot area is empty", () => {
     glh.dragAttributeToPlot(arrayOfAttributes[7]) // Habitat => plot area

v3/cypress/e2e/graph.spec.ts

@@ -15,11 +15,13 @@ const plots = graphRules.plots
 // (In local, this works fine and the tests can be run successfully)
 context.skip("Test graph plot transitions", () => {
   beforeEach(function () {
+    cy.log('Starting test setup')
     const queryParams = "?mouseSensor"
     const url = `${Cypress.config("index")}${queryParams}`
     cy.visit(url)
     cfm.openLocalDoc("cypress/fixtures/3TableGroups.codap")
     cy.wait(2500)
+    cy.log('Setup complete')
   })
 
   plots.forEach(test => {
@@ -148,14 +150,20 @@ context("Graph UI", () => {
       table.getGridCell(2, 2).should("contain", "African Elephant")
       cy.log("double-clicking the cell")
       // double-click to initiate editing cell
-      table.getGridCell(3, 4).click().dblclick()
-      cy.get("[data-testid=cell-text-editor]").type("700{enter}")
+      table.getGridCell(3, 4).dblclick()
+      // Wait for the input to appear and then type
+      table.getGridCell(3, 4).within(() => {
+        cy.get('input').should('be.visible').type("700{enter}", {force: true})
+      })
 
       table.getGridCell(2, 2).should("contain", "African Elephant")
       cy.log("double-clicking the cell")
       // double-click to initiate editing cell
-      table.getGridCell(3, 5).click().dblclick()
-      cy.get("[data-testid=cell-text-editor]").type("300{enter}")
+      table.getGridCell(3, 5).dblclick()
+      // Wait for the input to appear and then type
+      table.getGridCell(3, 5).within(() => {
+        cy.get('input').should('be.visible').type("300{enter}", {force: true})
+      })
 
       // get the rescale button
       c.getComponentTitle("graph").should("have.text", collectionName).click()

v3/cypress/e2e/hierarchical-table.spec.ts

@@ -4,10 +4,12 @@ const values = hierarchical.attributes
 
 context("hierarchical collections", () => {
   beforeEach(function () {
+    cy.log('Starting test setup')
     const queryParams = "?sample=mammals&dashboard&mouseSensor"
     const url = `${Cypress.config("index")}${queryParams}`
     cy.visit(url)
     cy.wait(2500)
+    cy.log('Setup complete')
   })
   hierarchical.tests.forEach((h) => {
     // FIXME: enable skipped tests

v3/cypress/e2e/map.spec.ts

@@ -17,9 +17,11 @@ const arrayOfAttributes = ["Category", "Educ_Tertiary_Perc", "Inversions"]
 
 context("Map UI", () => {
   beforeEach(function () {
+    cy.log('Starting test setup')
     const url = `${Cypress.config("index")}?mouseSensor&noComponentAnimation`
     cy.visit(url)
     cy.wait(3000)
+    cy.log('Setup complete')
   })
 
   it("verify map title", () => {

v3/cypress/e2e/plugin.spec.ts

@@ -7,8 +7,10 @@ import { WebViewTileElements as webView } from "../support/elements/web-view-til
 
 context("codap plugins", () => {
   beforeEach(function () {
+    cy.log('Starting test setup')
     const url = `${Cypress.config("index")}?sample=mammals&dashboard`
     cy.visit(url)
+    cy.log('Setup complete')
   })
   const openAPITester = () => {
     const url='https://concord-consortium.github.io/codap-data-interactives/DataInteractiveAPITester/index.html?lang=en'

v3/cypress/e2e/slider.spec.ts

@@ -10,10 +10,12 @@ const newSliderValue = "0.6"
 
 context("Slider UI", () => {
   beforeEach(function () {
+    cy.log('Starting test setup')
     const queryParams = "?sample=mammals&dashboard&mouseSensor"
     const url = `${Cypress.config("index")}${queryParams}`
     cy.visit(url)
     cy.wait(2500)
+    cy.log('Setup complete')
   })
   it("populates default title, variable name and value", () => {
     c.getComponentTitle("slider").should("contain", sliderName)