add v8 cypress test fixture

Opentrons · Dec 6, 2023 · 6195e82 · 6195e82
1 parent 1aa2df6
commit 6195e82
Show file tree

Hide file tree

Showing 5 changed files with 4,071 additions and 23 deletions.
diff --git a/protocol-designer/cypress/integration/migrations.spec.js b/protocol-designer/cypress/integration/migrations.spec.js
@@ -53,15 +53,13 @@ describe('Protocol fixtures migrate and match snapshots', () => {
       migrationModal: 'v8',
       unusedPipettes: false,
     },
-    //  TODO(jr, 11/30/23): write a test fixture here for v8 migrated to v8 with deck config when the ff is removed
-    // {
-    //   title: 'doItAllV8 flex robot -> reimported',
-    //   importFixture: '../../fixtures/protocol/8/doItAllV8.json',
-    //   expectedExportFixture:
-    //     '../../fixtures/protocol/8/doItAllV8.json',
-    //   migrationModal: 'noBehaviorChange',
-    //   unusedPipettes: false,
-    // },
+    {
+      title: 'doItAllV8 flex robot -> reimported, should not migrate',
+      importFixture: '../../fixtures/protocol/8/doItAllV8.json',
+      expectedExportFixture: '../../fixtures/protocol/8/doItAllV8.json',
+      migrationModal: null,
+      unusedPipettes: false,
+    },
   ]
 
   testCases.forEach(
@@ -159,7 +157,7 @@ describe('Protocol fixtures migrate and match snapshots', () => {
                 ).forEach(stepForm => {
                   if (stepForm.stepType === 'moveLiquid') {
                     stepForm.dropTip_location = 'trash drop tip location'
-                    if (stepForm.blowout_location.includes('trashBin')) {
+                    if (stepForm.blowout_location?.includes('trashBin')) {
                       stepForm.blowout_location = 'trash blowout location'
                     }
                   }