From ca449f4f08c36c26b0e2c25ff831bba41ecc8682 Mon Sep 17 00:00:00 2001
From: burnettk <burnettk@users.noreply.github.com>
Date: Wed, 28 Dec 2022 22:55:46 -0500
Subject: [PATCH] delete videos on success and address race condition in
 cypress test

---
 spiffworkflow-frontend/cypress.config.js      | 31 ++++++++++++++++++-
 .../cypress/support/commands.js               | 21 ++++++++++---
 .../src/routes/ProcessInstanceLogList.tsx     |  2 +-
 3 files changed, 48 insertions(+), 6 deletions(-)

diff --git a/spiffworkflow-frontend/cypress.config.js b/spiffworkflow-frontend/cypress.config.js
index 580bf1ffd..3d6c7410e 100644
--- a/spiffworkflow-frontend/cypress.config.js
+++ b/spiffworkflow-frontend/cypress.config.js
@@ -1,12 +1,41 @@
 /* eslint-disable */
 const { defineConfig } = require('cypress');
+const { rm } = require('fs/promises')
+
+// yes use video compression in CI, where we will set the env var so we upload to cypress dashboard
+const useVideoCompression = !!process.env.CYPRESS_RECORD_KEY
+
+// https://github.com/cypress-io/cypress/issues/2522
+const deleteVideosOnSuccess = (on) => {
+  const filesToDelete = []
+  on('after:spec', (_spec, results) => {
+    if (results.stats.failures === 0 && results.video) {
+      filesToDelete.push(results.video)
+    }
+  })
+  on('after:run', async () => {
+    if (filesToDelete.length) {
+      console.log(
+        'after:run hook: Deleting %d video(s) from successful specs',
+        filesToDelete.length
+      )
+      await Promise.all(filesToDelete.map((videoFile) => rm(videoFile)))
+    }
+  })
+}
 
 module.exports = defineConfig({
   projectId: 'crax1q',
+
+  // since it's slow
+  videoCompression: useVideoCompression,
+
+  videoUploadOnPasses: false,
   chromeWebSecurity: false,
   e2e: {
     baseUrl: 'http://localhost:7001',
-    setupNodeEvents(_on, config) {
+    setupNodeEvents(on, config) {
+      deleteVideosOnSuccess(on)
       require('@cypress/grep/src/plugin')(config);
       return config;
     },
diff --git a/spiffworkflow-frontend/cypress/support/commands.js b/spiffworkflow-frontend/cypress/support/commands.js
index 16b276b6e..124d1acc8 100644
--- a/spiffworkflow-frontend/cypress/support/commands.js
+++ b/spiffworkflow-frontend/cypress/support/commands.js
@@ -121,10 +121,23 @@ Cypress.Commands.add('basicPaginationTest', () => {
 
   // NOTE: this is a em dash instead of en dash
   cy.contains(/\b1–2 of \d+/);
-  cy.get('.cds--pagination__button--forward').click();
-  cy.contains(/\b3–4 of \d+/);
-  cy.get('.cds--pagination__button--backward').click();
-  cy.contains(/\b1–2 of \d+/);
+
+  // ok, trying to ensure that we have everything loaded before we leave this
+  // function and try to sign out. Just showing results 1-2 of blah is not good enough,
+  // since the ajax request may not have finished yet.
+  // to be sure it's finished, grab the log id from page 1. remember it.
+  // then use the magical contains command that waits for the element to exist AND
+  // for that element to contain the text we're looking for.
+  cy.getBySel('process-instance-log-id')
+    .first()
+    .then(($element) => {
+      const oldId = $element.text().trim();
+      cy.get('.cds--pagination__button--forward').click();
+      cy.contains(/\b3–4 of \d+/);
+      cy.get('.cds--pagination__button--backward').click();
+      cy.contains(/\b1–2 of \d+/);
+      cy.contains('[data-qa=process-instance-log-id]', oldId);
+    });
 });
 
 Cypress.Commands.add('assertAtLeastOneItemInPaginatedResults', () => {
diff --git a/spiffworkflow-frontend/src/routes/ProcessInstanceLogList.tsx b/spiffworkflow-frontend/src/routes/ProcessInstanceLogList.tsx
index e3ba8b932..89bc869cc 100644
--- a/spiffworkflow-frontend/src/routes/ProcessInstanceLogList.tsx
+++ b/spiffworkflow-frontend/src/routes/ProcessInstanceLogList.tsx
@@ -45,7 +45,7 @@ export default function ProcessInstanceLogList() {
       const rowToUse = row as any;
       return (
         <tr key={rowToUse.id}>
-          <td>{rowToUse.id}</td>
+          <td data-qa="process-instance-log-id">{rowToUse.id}</td>
           <td>{rowToUse.message}</td>
           <td>{rowToUse.bpmn_task_name}</td>
           {isDetailedView && (