fix(vite): Only attempt to amend test object if one exists

Test object is assigned using optional chaining but previously did not check that any value had actually been assigned which would cause the `getStart` call to fail resulting in a failed migration.
nrwl · Jul 3, 2024 · bc6ca70 · bc6ca70
1 parent 28939dd
commit bc6ca70
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/packages/vite/src/migrations/update-17-3-0/lib/fix-coverage-and-reporters.ts b/packages/vite/src/migrations/update-17-3-0/lib/fix-coverage-and-reporters.ts
@@ -102,11 +102,14 @@ export function fixCoverageAndRerporters(
       configNode,
       `PropertyAssignment:has(Identifier[name="test"])`
     )?.[0];
-    changes.push({
-      type: ChangeType.Insert,
-      index: testObject.getStart() + `test: {`.length + 1,
-      text: `reporters: ['default'],`,
-    });
+
+    if (testObject) {
+      changes.push({
+        type: ChangeType.Insert,
+        index: testObject.getStart() + `test: {`.length + 1,
+        text: `reporters: ['default'],`,
+      });
+    }
   }
 
   if (changes.length > 0) {