fix: sync hooks (#54)

index.js

@@ -194,7 +194,7 @@ function restoreClientErrorListeners (server, oldListeners) {
 
 async function executeHooks (hooks, app, opts) {
   for (const hook of hooks) {
-    await hook(app, opts).catch((error) => app.log.error(error))
+    await hook(app, opts)?.catch((error) => app.log.error(error))
   }
 }
 

test/hooks.test.js

@@ -170,3 +170,30 @@ test('should throw if onRestartHook is not a function', async (t) => {
     app.addOnRestartHook('not a function')
   }, 'onRestartHook must be a function')
 })
+
+test('should not throw if onRestartHook is a sync function', async (t) => {
+  t.plan(3)
+
+  async function createApplication (fastify, opts) {
+    return fastify(opts)
+  }
+
+  const app = await restartable(createApplication, {
+    keepAliveTimeout: 1
+  })
+
+  t.teardown(async () => {
+    await app.close()
+  })
+
+  const expectedRestartOptions = { foo: 'bar' }
+
+  app.addOnRestartHook((app, restartOptions) => {
+    t.equal(app.restarted, true)
+    t.same(restartOptions, expectedRestartOptions)
+  })
+
+  await app.restart(expectedRestartOptions)
+
+  t.equal(app.restarted, true)
+})