From ac81d69e61d1932fcf8ab84cf9d1e2a9c5d38040 Mon Sep 17 00:00:00 2001
From: jasquat <jasquat@users.noreply.github.com>
Date: Fri, 17 Jun 2022 11:05:14 -0400
Subject: [PATCH] attempt to send data to cypress.io w/ burnettk

---
 .github/workflows/tests.yml | 13 +++++++++++--
 .gitignore                  |  1 +
 cypress.config.js           |  1 +
 cypress/.eslintrc.json      |  5 +++++
 src/react_diagram_editor.js |  2 +-
 5 files changed, 19 insertions(+), 3 deletions(-)
 create mode 100644 cypress/.eslintrc.json

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index 0f7c368..0d12408 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -61,15 +61,24 @@ jobs:
         uses: cypress-io/github-action@v4
         with:
           working-directory: ./spiffworkflow-frontend
+          browser: chrome
+          record: true
+        env:
+          # pass the Dashboard record key as an environment variable
+          CYPRESS_RECORD_KEY: ${{ secrets.CYPRESS_RECORD_KEY }}
+          # pass GitHub token to allow accurately detecting a build vs a re-run build
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
       #  https://github.com/cypress-io/github-action#artifacts
-      - uses: actions/upload-artifact@v2
+      - name: upload_screenshots
+        uses: actions/upload-artifact@v2
         if: failure()
         with:
           name: cypress-screenshots
           path: ./spiffworkflow-frontend/cypress/screenshots
       # Test run video was always captured, so this action uses "always()" condition
-      - uses: actions/upload-artifact@v2
+      - name: upload_videos
+        uses: actions/upload-artifact@v2
         if: always()
         with:
           name: cypress-videos
diff --git a/.gitignore b/.gitignore
index 0b5b0e1..3ec1e0e 100644
--- a/.gitignore
+++ b/.gitignore
@@ -23,3 +23,4 @@ yarn-debug.log*
 yarn-error.log*
 
 cypress/videos
+cypress/screenshots
diff --git a/cypress.config.js b/cypress.config.js
index 843d5f4..ab9c647 100644
--- a/cypress.config.js
+++ b/cypress.config.js
@@ -1,6 +1,7 @@
 const { defineConfig } = require("cypress");
 
 module.exports = defineConfig({
+  projectId: "crax1q",
   e2e: {
     baseUrl: "http://localhost:7001",
     setupNodeEvents(on, config) {
diff --git a/cypress/.eslintrc.json b/cypress/.eslintrc.json
new file mode 100644
index 0000000..3739142
--- /dev/null
+++ b/cypress/.eslintrc.json
@@ -0,0 +1,5 @@
+{
+  "extends": [
+    "plugin:cypress/recommended"
+  ]
+}
diff --git a/src/react_diagram_editor.js b/src/react_diagram_editor.js
index f3198d8..33fefef 100644
--- a/src/react_diagram_editor.js
+++ b/src/react_diagram_editor.js
@@ -195,7 +195,7 @@ export default function ReactDiagramEditor(props) {
     function displayDiagram(diagramModelerToUse, diagramXMLToDisplay) {
       diagramModelerToUse.importXML(diagramXMLToDisplay);
     }
-  }, [props, diagramXML, diagramModelerState]);
+  }, [props, diagramXML, diagramModelerState, performingXmlUpdates]);
 
   function handleSave() {
     diagramModelerState.saveXML({ format: true })