From da0ef5a98f0cbf26e35e81025f52dde7b8754be0 Mon Sep 17 00:00:00 2001
From: Patrik Oldsberg <poldsberg@gmail.com>
Date: Mon, 2 Oct 2023 19:01:10 +0200
Subject: [PATCH] backend-test-utils: skip mock directory cleanup in CI

Signed-off-by: Patrik Oldsberg <poldsberg@gmail.com>
---
 .../src/filesystem/MockDirectory.test.ts      | 22 -------------------
 .../src/filesystem/MockDirectory.ts           | 10 +++++++--
 2 files changed, 8 insertions(+), 24 deletions(-)

diff --git a/packages/backend-test-utils/src/filesystem/MockDirectory.test.ts b/packages/backend-test-utils/src/filesystem/MockDirectory.test.ts
index 54d32f47d7..45a206d7df 100644
--- a/packages/backend-test-utils/src/filesystem/MockDirectory.test.ts
+++ b/packages/backend-test-utils/src/filesystem/MockDirectory.test.ts
@@ -269,28 +269,6 @@ describe('createMockDirectory', () => {
     );
   });
 
-  describe('cleanup', () => {
-    let cleanupMockDir: MockDirectory;
-
-    describe('inner', () => {
-      cleanupMockDir = createMockDirectory();
-
-      it('should populate a directory', () => {
-        cleanupMockDir.setContent({
-          'a.txt': 'a',
-        });
-
-        expect(cleanupMockDir.content()).toEqual({
-          'a.txt': 'a',
-        });
-      });
-    });
-
-    it('should clean up after itself automatically', () => {
-      expect(cleanupMockDir.content()).toBeUndefined();
-    });
-  });
-
   describe('tmpdir mock', () => {
     let tmpDirMock: MockDirectory;
 
diff --git a/packages/backend-test-utils/src/filesystem/MockDirectory.ts b/packages/backend-test-utils/src/filesystem/MockDirectory.ts
index 43df843946..1d48d157df 100644
--- a/packages/backend-test-utils/src/filesystem/MockDirectory.ts
+++ b/packages/backend-test-utils/src/filesystem/MockDirectory.ts
@@ -351,14 +351,20 @@ export function createMockDirectory(
     os.tmpdir = () => mocker.path;
   }
 
-  process.on('beforeExit', mocker.remove);
+  // In CI we expect there to be no need to clean up temporary directories
+  const needsCleanup = !process.env.CI;
+  if (needsCleanup) {
+    process.on('beforeExit', mocker.remove);
+  }
 
   try {
     afterAll(() => {
       if (origTmpdir) {
         os.tmpdir = origTmpdir;
       }
-      mocker.remove();
+      if (needsCleanup) {
+        mocker.remove();
+      }
     });
   } catch {
     /* ignore */