test: commented test

Author: Tony133

test/start.test.js

@@ -566,76 +566,77 @@ test('should start the server listening on 0.0.0.0 when running in kubernetes',
   t.assert.ok('server closed')
 })
 
-test('should start the server with watch options that the child process restart when directory changed', { skip: true }, async (t) => {
-  t.plan(3)
-  const tmpjs = path.resolve(baseFilename + '.js')
-
-  const port = getPort()
-
-  await writeFile(tmpjs, 'hello world')
-  const argv = ['-p', port, '-w', './examples/plugin.js']
-  const fastifyEmitter = await start.start(argv)
-
-  t.after(async () => {
-    if (fs.existsSync(tmpjs)) {
-      fs.unlinkSync(tmpjs)
-    }
-    await fastifyEmitter.stop()
-  })
-
-  await once(fastifyEmitter, 'ready')
-  t.assert.ok('should receive ready event')
-
-  const restartPromise = once(fastifyEmitter, 'restart')
-  await writeFile(tmpjs, 'hello fastify', { flag: 'a+' }) // chokidar watch can't catch change event in CI, but local test is all ok. you can remove annotation in local environment.
-  t.assert.ok('change tmpjs')
-
-  // this might happen more than once but does not matter in this context
-  await restartPromise
-  t.assert.ok('should receive restart event')
-})
-
-test('should start the server with watch and verbose-watch options that the child process restart when directory changed with console message about changes ', { skip: true }, async (t) => {
-  t.plan(4)
-
-  const spy = sinon.spy()
-  const watch = proxyquire('../lib/watch', {
-    './utils': {
-      logWatchVerbose: spy
-    }
-  })
-
-  const start = proxyquire('../start', {
-    './lib/watch': watch
-  })
-
-  const tmpjs = path.resolve(baseFilename + '.js')
-
-  const port = getPort()
-
-  await writeFile(tmpjs, 'hello world')
-  const argv = ['-p', port, '-w', '--verbose-watch', './examples/plugin.js']
-  const fastifyEmitter = await start.start(argv)
-
-  t.after(async () => {
-    if (fs.existsSync(tmpjs)) {
-      fs.unlinkSync(tmpjs)
-    }
-    await fastifyEmitter.stop()
-  })
-
-  await once(fastifyEmitter, 'ready')
-  t.assert.ok('should receive ready event')
-
-  const restartPromise = once(fastifyEmitter, 'restart')
-  await writeFile(tmpjs, 'hello fastify', { flag: 'a+' }) // chokidar watch can't catch change event in CI, but local test is all ok. you can remove annotation in local environment.
-  t.assert.ok('change tmpjs')
-
-  // this might happen more than once but does not matter in this context
-  await restartPromise
-  t.assert.ok('should receive restart event')
-  t.assert.ok(spy.args.length > 0, 'should print a console message on file update')
-})
+// Temporarily commented out during migration from TAP to node:test
+// test('should start the server with watch options that the child process restart when directory changed', { skip: ['win32', 'darwin'].includes(process.platform) || process.env.CI === 'true' }, async (t) => {
+//   t.plan(3)
+//   const tmpjs = path.resolve(baseFilename + '.js')
+
+//   const port = getPort()
+
+//   await writeFile(tmpjs, 'hello world')
+//   const argv = ['-p', port, '-w', './examples/plugin.js']
+//   const fastifyEmitter = await start.start(argv)
+
+//   t.after(async () => {
+//     if (fs.existsSync(tmpjs)) {
+//       fs.unlinkSync(tmpjs)
+//     }
+//     await fastifyEmitter.stop()
+//   })
+
+//   await once(fastifyEmitter, 'ready')
+//   t.assert.ok('should receive ready event')
+
+//   const restartPromise = once(fastifyEmitter, 'restart')
+//   await writeFile(tmpjs, 'hello fastify', { flag: 'a+' }) // chokidar watch can't catch change event in CI, but local test is all ok. you can remove annotation in local environment.
+//   t.assert.ok('change tmpjs')
+
+//   // this might happen more than once but does not matter in this context
+//   await restartPromise
+//   t.assert.ok('should receive restart event')
+// })
+
+// test('should start the server with watch and verbose-watch options that the child process restart when directory changed with console message about changes ', { skip: ['win32', 'darwin'].includes(process.platform) || process.env.CI === 'true' }, async (t) => {
+//   t.plan(4)
+
+//   const spy = sinon.spy()
+//   const watch = proxyquire('../lib/watch', {
+//     './utils': {
+//       logWatchVerbose: spy
+//     }
+//   })
+
+//   const start = proxyquire('../start', {
+//     './lib/watch': watch
+//   })
+
+//   const tmpjs = path.resolve(baseFilename + '.js')
+
+//   const port = getPort()
+
+//   await writeFile(tmpjs, 'hello world')
+//   const argv = ['-p', port, '-w', '--verbose-watch', './examples/plugin.js']
+//   const fastifyEmitter = await start.start(argv)
+
+//   t.after(async () => {
+//     if (fs.existsSync(tmpjs)) {
+//       fs.unlinkSync(tmpjs)
+//     }
+//     await fastifyEmitter.stop()
+//   })
+
+//   await once(fastifyEmitter, 'ready')
+//   t.assert.ok('should receive ready event')
+
+//   const restartPromise = once(fastifyEmitter, 'restart')
+//   await writeFile(tmpjs, 'hello fastify', { flag: 'a+' }) // chokidar watch can't catch change event in CI, but local test is all ok. you can remove annotation in local environment.
+//   t.assert.ok('change tmpjs')
+
+//   // this might happen more than once but does not matter in this context
+//   await restartPromise
+//   t.assert.ok('should receive restart event')
+//   t.assert.ok(spy.args.length > 0, 'should print a console message on file update')
+// })
 
 test('should reload the env on restart when watching', { skip: process.platform === 'win32' }, async (t) => {
   const testdir = await createTestDir({