fix(core): workspace remove generator should handle no root jest conf…

…ig (#23328)
nrwl · May 13, 2024 · 4f31608 · 4f31608
1 parent 7561e71
commit 4f31608
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 0 deletions.
diff --git a/packages/workspace/src/generators/remove/lib/update-jest-config.spec.ts b/packages/workspace/src/generators/remove/lib/update-jest-config.spec.ts
@@ -91,4 +91,24 @@ describe('updateRootJestConfig', () => {
 
     expect(rootJestConfig).toMatchSnapshot();
   });
+
+  it('should handle not having a root jest config file', async () => {
+    // ARRANGE
+    tree.delete('jest.config.ts');
+
+    await libraryGenerator(tree, {
+      name: 'test',
+      bundler: 'vite',
+      unitTestRunner: 'vitest',
+    });
+
+    // ACT
+    expect(() =>
+      updateJestConfig(
+        tree,
+        { projectName: 'test', skipFormat: false, forceRemove: false },
+        readProjectConfiguration(tree, 'test')
+      )
+    ).not.toThrow();
+  });
 });
diff --git a/packages/workspace/src/generators/remove/lib/update-jest-config.ts b/packages/workspace/src/generators/remove/lib/update-jest-config.ts
@@ -69,6 +69,7 @@ export function updateJestConfig(
   const rootConfigPath = findRootJestConfig(tree);
 
   if (
+    !rootConfigPath ||
     !tree.exists(rootConfigPath) ||
     !tree.exists(join(projectConfig.root, 'jest.config.ts')) ||
     isUsingUtilityFunction(tree) ||