chore(test): Fixed teardown() because tests are failing

log4js-node · Jan 19, 2022 · 057d8f8 · 057d8f8
1 parent 43a2199
commit 057d8f8
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 5 deletions.
diff --git a/test/tap/multi-file-appender-test.js b/test/tap/multi-file-appender-test.js
@@ -236,8 +236,15 @@ test("multiFile appender", batch => {
     });
   });
 
-  batch.tearDown(() => {
-    fs.rmdirSync("logs");
+  batch.tearDown(async () => {
+    try {
+      const files = fs.readdirSync("logs");
+      await removeFiles(files.map(filename => `logs/${filename}`));
+      fs.rmdirSync("logs");
+    } catch (e) {
+      throw e;
+      // doesn't matter
+    }
   });
 
   batch.end();

diff --git a/test/tap/pause-test.js b/test/tap/pause-test.js
@@ -77,11 +77,17 @@ tap.test("Drain event test", batch => {
         logger.info("This is a test for emitting drain event in date file logger");
     }
     t.end();
-
   });
 
-  batch.tearDown(() => {
-    fs.rmdirSync("logs");
+  batch.tearDown(async () => {
+    try {
+      const files = fs.readdirSync("logs");
+      await removeFiles(files.map(filename => `logs/${filename}`));
+      fs.rmdirSync("logs");
+    } catch (e) {
+      throw e;
+      // doesn't matter
+    }
   });
 
   batch.end();