test(MongoMemoryReplSet): better cleanup created temporary directories

typegoose · Jun 9, 2023 · bd18403 · bd18403
1 parent 4d22ce5
commit bd18403
Showing 1 changed file with 12 additions and 21 deletions.
diff --git a/packages/mongodb-memory-server-core/src/__tests__/MongoMemoryReplSet.test.ts b/packages/mongodb-memory-server-core/src/__tests__/MongoMemoryReplSet.test.ts
@@ -551,9 +551,7 @@ describe('MongoMemoryReplSet', () => {
       const replSet = await MongoMemoryReplSet.create({ replSet: { count: 1 } });
 
       const cleanupSpy = jest.spyOn(replSet, 'cleanup');
-      const cleanupInstance0Spy = jest
-        .spyOn(replSet.servers[0], 'cleanup')
-        .mockResolvedValue(void 0);
+      const cleanupInstance0Spy = jest.spyOn(replSet.servers[0], 'cleanup');
 
       await replSet.stop();
 
@@ -568,9 +566,7 @@ describe('MongoMemoryReplSet', () => {
       const replSet = await MongoMemoryReplSet.create({ replSet: { count: 1 } });
 
       const cleanupSpy = jest.spyOn(replSet, 'cleanup');
-      const cleanupInstance0Spy = jest
-        .spyOn(replSet.servers[0], 'cleanup')
-        .mockResolvedValue(void 0);
+      const cleanupInstance0Spy = jest.spyOn(replSet.servers[0], 'cleanup');
 
       await replSet.stop({ doCleanup: false });
 
@@ -608,9 +604,7 @@ describe('MongoMemoryReplSet', () => {
       const replSet = await MongoMemoryReplSet.create({ replSet: { count: 1 } });
 
       const cleanupSpy = jest.spyOn(replSet, 'cleanup');
-      const cleanupInstance0Spy = jest
-        .spyOn(replSet.servers[0], 'cleanup')
-        .mockResolvedValue(void 0);
+      const cleanupInstance0Spy = jest.spyOn(replSet.servers[0], 'cleanup');
 
       await replSet.stop({ doCleanup: false, force: false });
 
@@ -635,9 +629,7 @@ describe('MongoMemoryReplSet', () => {
       const replSet = await MongoMemoryReplSet.create({ replSet: { count: 1 } });
 
       const cleanupSpy = jest.spyOn(replSet, 'cleanup');
-      const cleanupInstance0Spy = jest
-        .spyOn(replSet.servers[0], 'cleanup')
-        .mockResolvedValue(void 0);
+      const cleanupInstance0Spy = jest.spyOn(replSet.servers[0], 'cleanup');
 
       await replSet.stop({ doCleanup: false });
       await replSet.cleanup();
@@ -653,9 +645,7 @@ describe('MongoMemoryReplSet', () => {
       const replSet = await MongoMemoryReplSet.create({ replSet: { count: 1 } });
 
       const cleanupSpy = jest.spyOn(replSet, 'cleanup');
-      const cleanupInstance0Spy = jest
-        .spyOn(replSet.servers[0], 'cleanup')
-        .mockResolvedValue(void 0);
+      const cleanupInstance0Spy = jest.spyOn(replSet.servers[0], 'cleanup');
 
       await replSet.stop({ doCleanup: false });
       await replSet.cleanup({ doCleanup: true, force: false });
@@ -670,9 +660,7 @@ describe('MongoMemoryReplSet', () => {
 
       await replSet.start();
 
-      const cleanupInstance1Spy = jest
-        .spyOn(replSet.servers[0], 'cleanup')
-        .mockResolvedValue(void 0);
+      const cleanupInstance1Spy = jest.spyOn(replSet.servers[0], 'cleanup');
 
       await replSet.stop({ doCleanup: false });
       await replSet.cleanup({ doCleanup: true, force: true });
@@ -702,9 +690,7 @@ describe('MongoMemoryReplSet', () => {
       const replSet = await MongoMemoryReplSet.create({ replSet: { count: 1 } });
 
       const cleanupSpy = jest.spyOn(replSet, 'cleanup');
-      const cleanupInstance0Spy = jest
-        .spyOn(replSet.servers[0], 'cleanup')
-        .mockResolvedValue(void 0);
+      const cleanupInstance0Spy = jest.spyOn(replSet.servers[0], 'cleanup');
 
       await replSet.stop({ doCleanup: false });
       await replSet.cleanup({ doCleanup: true, force: true });
@@ -754,5 +740,10 @@ describe('MongoMemoryReplSet', () => {
     utils.assertion(!utils.isNullOrUndefined(instanceInfo));
     expect(instanceInfo.instance).toBeDefined();
     expect(instanceInfo?.launchTimeout).toStrictEqual(2000);
+
+    await utils.removeDir(
+      // @ts-expect-error "_instanceInfo" is protected
+      replSet.servers[0]._instanceInfo.tmpDir!
+    ); // manual cleanup
   });
 });