Fix Node.js in CITGM (v18.x) (#1828)

* Fix Node.js in CITGM (v18.x)

Signed-off-by: Matteo Collina <hello@matteocollina.com>

* fixup

Signed-off-by: Matteo Collina <hello@matteocollina.com>

---------

Signed-off-by: Matteo Collina <hello@matteocollina.com>

test/broken-pipe.test.js

@@ -11,6 +11,16 @@ if (process.platform === 'win32') {
   process.exit(0)
 }
 
+if (process.env.CITGM) {
+  // This looks like a some form of limitations of the CITGM test runner
+  // or the HW/SW we run it on. This file can hang on Node.js v18.x.
+  // The failure does not reproduce locally or on our CI.
+  // Skipping it is the only way to keep pino in CITGM.
+  // https://github.com/nodejs/citgm/pull/1002#issuecomment-1751942988
+  t.skip('Skipping on Node.js core CITGM because it hangs on v18.x')
+  process.exit(0)
+}
+
 function test (file) {
   file = join('fixtures', 'broken-pipe', file)
   t.test(file, { parallel: true }, async ({ equal }) => {

test/syncfalse.test.js

@@ -7,6 +7,9 @@ const { test } = require('tap')
 const { fork } = require('child_process')
 const writer = require('flush-write-stream')
 const { once, getPathToNull } = require('./helper')
+const { promisify } = require('util')
+
+const sleep = promisify(setTimeout)
 
 test('asynchronous logging', async ({ equal, teardown }) => {
   const now = Date.now
@@ -48,6 +51,8 @@ test('asynchronous logging', async ({ equal, teardown }) => {
     cb()
   }))
   await once(child, 'close')
+  // Wait for the last write to be flushed
+  await sleep(100)
   equal(actual, expected)
   equal(actual2.trim(), expected2)