update flush function type

pino.d.ts

@@ -123,7 +123,7 @@ export interface LoggerExtras<Options = LoggerOptions> extends EventEmitter {
     /**
      * Flushes the content of the buffer when using pino.destination({ sync: false }).
      */
-    flush(): () => void;
+    flush(): void;
 }
 
 

test/syncfalse.test.js

@@ -112,7 +112,7 @@ test('sync false with child', async ({ equal, teardown }) => {
   })
 })
 
-test('flush does nothing with sync true (default)', async () => {
+test('flush does nothing with sync true (default)', async ({ equal }) => {
   const instance = require('..')()
-  instance.flush()
+  equal(instance.flush(), undefined)
 })

test/transport/syncfalse.test.js

@@ -10,15 +10,17 @@ const { watchFileCreated, file } = require('../helper')
 const { pid } = process
 const hostname = os.hostname()
 
-test('thread-stream async flush', async ({ same }) => {
+test('thread-stream async flush', async ({ equal, same }) => {
   const destination = file()
   const transport = pino.transport({
     target: join(__dirname, '..', 'fixtures', 'to-file-transport.js'),
     options: { destination }
   })
   const instance = pino(transport)
   instance.info('hello')
-  instance.flush()
+
+  equal(instance.flush(), undefined)
+
   await watchFileCreated(destination)
   const result = JSON.parse(await readFile(destination))
   delete result.time

test/types/pino.test-d.ts

@@ -111,6 +111,7 @@ pino({ base: null });
 // @ts-expect-error
 if ("pino" in log) console.log(`pino version: ${log.pino}`);
 
+expectType<void>(log.flush());
 log.child({ a: "property" }).info("hello child!");
 log.level = "error";
 log.info("nope");