move the wait outside

Signed-off-by: Matteo Collina <[email protected]>

Author: mcollina

lib/plugin.js

@@ -1,6 +1,5 @@
 import { readFile } from 'node:fs/promises'
 import { basename } from 'node:path'
-import { setTimeout as sleep } from 'node:timers/promises'
 
 const HTTP_METHODS = ['GET', 'POST', 'PUT', 'DELETE', 'PATCH', 'OPTIONS', 'TRACE']
 
@@ -29,10 +28,6 @@ export async function plugin (server, opts) {
     done(null, body)
   })
 
-  server.addHook('onClose', async () => {
-    await sleep(1000)
-  })
-
   for (const method of HTTP_METHODS) {
     server.route({
       method,

test/plugin.test.js

@@ -5,6 +5,7 @@ import { resolve, join } from 'node:path'
 import { test } from 'node:test'
 import { stackable } from '../lib/index.js'
 import formAutoContet from 'form-auto-content'
+import { setTimeout as sleep } from 'node:timers/promises'
 
 async function startStackable (t, docroot = join(import.meta.dirname, './fixtures/hello'), opts = {}) {
   const config = {
@@ -21,9 +22,11 @@ async function startStackable (t, docroot = join(import.meta.dirname, './fixture
     }
   }
 
-  const server = await buildServer(config, stackable)
+  let server = await buildServer(config, stackable)
   t.after(async () => {
     await server.close()
+    server = null
+    await sleep(1000) // wait for the server to close
   })
 
   return server