chore(test): Added timeout to teardown() to wait for file system beca…

…use tests are failing
log4js-node · Jan 19, 2022 · bc5286d · bc5286d
1 parent 43a2199
commit bc5286d
Show file tree

Hide file tree

Showing 2 changed files with 16 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,14 @@ 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) {
+      // doesn't matter
+    }
   });
 
   batch.end();

diff --git a/test/tap/pause-test.js b/test/tap/pause-test.js
@@ -77,11 +77,16 @@ 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) {
+      // doesn't matter
+    }
   });
 
   batch.end();