Fix: Clean up temporary test directories after each test

src/__tests__/settings.js

@@ -14,6 +14,12 @@
    limitations under the License.
  */
 
+// Store temporary directories created during tests in global scope
+// to ensure cleanup even when jest.resetModules() clears the module cache
+if (!global.__testTempDirectories__) {
+  global.__testTempDirectories__ = [];
+}
+
 /**
  * Creates a test-isolated settings module with temporary directory overrides.
  *
@@ -30,6 +36,7 @@ const testSettings = async () => {
   const settings = require('../settings');
   settings.paths.appDir = await fs.promises.mkdtemp(path.join(os.tmpdir(), 'electronim-test-'));
   settings.paths.settingsPath = path.join(settings.paths.appDir, 'settings.json');
+  global.__testTempDirectories__.push(settings.paths.appDir);
   return settings;
 };
 

src/__tests__/setup-jest.js

@@ -15,10 +15,17 @@
  */
 afterEach(async () => {
   jest.useRealTimers();
-  const os = require('node:os');
   const fs = require('node:fs');
-  const settings = require('../settings');
-  if (os.tmpdir && settings?.paths?.appDir && settings.paths.appDir.startsWith(os.tmpdir())) {
-    await fs.promises.rm(settings.paths.appDir, {recursive: true, force: true});
+
+  // Clean up all tracked temporary directories
+  if (global.__testTempDirectories__) {
+    while (global.__testTempDirectories__.length > 0) {
+      const dir = global.__testTempDirectories__.pop();
+      try {
+        await fs.promises.rm(dir, {recursive: true, force: true});
+      } catch {
+        // Ignore errors if directory was already cleaned up
+      }
+    }
   }
 });