fix: fix a bug where an entry failed if the patches dir was missing (#56

)
bazel-contrib · Sep 25, 2023 · 9ad8490 · 9ad8490
1 parent 262c9e1
commit 9ad8490
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/src/domain/create-entry.spec.ts b/src/domain/create-entry.spec.ts
@@ -61,10 +61,15 @@ beforeEach(() => {
       .map((f) => path.basename(f));
   }) as any);
 
-  mocked(fs.existsSync).mockImplementation(((path: string) => {
-    if (path in mockedFileReads) {
+  mocked(fs.existsSync).mockImplementation(((p: string) => {
+    if (p in mockedFileReads) {
       return true;
     }
+    for (const f of Object.keys(mockedFileReads)) {
+      if (path.dirname(f) == p) {
+        return true;
+      }
+    }
     return (jest.requireActual("node:fs") as any).existsSync(path);
   }) as any);
 

diff --git a/src/domain/create-entry.ts b/src/domain/create-entry.ts
@@ -154,6 +154,9 @@ export class CreateEntryService {
     moduleRoot: string
   ): void {
     const patchesPath = rulesetRepo.patchesPath(moduleRoot);
+    if (!fs.existsSync(patchesPath)) {
+      return;
+    }
     const patches = fs
       .readdirSync(patchesPath)
       .filter((f) => f.endsWith(".patch"));