Fix tests in Node.js v16 (#4597)

fastify · Feb 26, 2023 · 09cac4a · 09cac4a
1 parent 562332a
commit 09cac4a
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 10 deletions.
diff --git a/.github/workflows/package-manager-ci.yml b/.github/workflows/package-manager-ci.yml
@@ -15,7 +15,7 @@ jobs:
     strategy:
       matrix:
         # Maintenance and active LTS
-        node-version: [14, 16]
+        node-version: [14, 16, 18]
         os: [ubuntu-latest]
 
     steps:
@@ -43,7 +43,7 @@ jobs:
     strategy:
       matrix:
         # Maintenance and active LTS
-        node-version: [14, 16]
+        node-version: [14, 16, 18]
         os: [ubuntu-latest]
 
     steps:

diff --git a/test/logger.test.js b/test/logger.test.js
@@ -28,15 +28,17 @@ before(async function () {
   [localhost, localhostForURL] = await helper.getLoopbackHost()
 })
 
-teardown(() => {
-  files.forEach((file) => {
-    try {
-      fs.unlinkSync(file)
-    } catch (e) {
-      console.log(e)
-    }
+if (process.env.CI) {
+  teardown(() => {
+    files.forEach((file) => {
+      try {
+        fs.unlinkSync(file)
+      } catch (e) {
+        console.log(e)
+      }
+    })
   })
-})
+}
 
 test('defaults to info level', t => {
   let fastify = null