fastify 5.3.2 → 5.4.0

package/test/logger/response.test.js

@@ -2,19 +2,18 @@
 
 const stream = require('node:stream')
 
-const t = require('tap')
+const t = require('node:test')
 const split = require('split2')
 const pino = require('pino')
 
 const Fastify = require('../../fastify')
+const { partialDeepStrictEqual } = require('../toolkit')
 const { on } = stream
 
-t.test('response serialization', (t) => {
-  t.setTimeout(60000)
-
+t.test('response serialization', { timeout: 60000 }, async (t) => {
   t.plan(4)
 
-  t.test('Should use serializers from plugin and route', async (t) => {
+  await t.test('Should use serializers from plugin and route', async (t) => {
     const lines = [
       { msg: 'incoming request' },
       { test: 'XHello', test2: 'ZHello' },
@@ -28,7 +27,7 @@ t.test('response serialization', (t) => {
     const fastify = Fastify({
       loggerInstance
     })
-    t.teardown(fastify.close.bind(fastify))
+    t.after(() => fastify.close())
 
     fastify.register(context1, {
       logSerializers: { test: value => 'X' + value }
@@ -51,16 +50,16 @@ t.test('response serialization', (t) => {
     {
       const response = await fastify.inject({ method: 'GET', url: '/' })
       const body = await response.json()
-      t.same(body, { hello: 'world' })
+      t.assert.deepStrictEqual(body, { hello: 'world' })
     }
 
     for await (const [line] of on(stream, 'data')) {
-      t.match(line, lines.shift())
+      t.assert.ok(partialDeepStrictEqual(line, lines.shift()))
       if (lines.length === 0) break
     }
   })
 
-  t.test('Should use serializers from instance fastify and route', async (t) => {
+  await t.test('Should use serializers from instance fastify and route', async (t) => {
     const lines = [
       { msg: 'incoming request' },
       { test: 'XHello', test2: 'ZHello' },
@@ -80,7 +79,7 @@ t.test('response serialization', (t) => {
     const fastify = Fastify({
       loggerInstance
     })
-    t.teardown(fastify.close.bind(fastify))
+    t.after(() => fastify.close())
 
     fastify.get('/', {
       logSerializers: {
@@ -96,16 +95,16 @@ t.test('response serialization', (t) => {
     {
       const response = await fastify.inject({ method: 'GET', url: '/' })
       const body = await response.json()
-      t.same(body, { hello: 'world' })
+      t.assert.deepStrictEqual(body, { hello: 'world' })
     }
 
     for await (const [line] of on(stream, 'data')) {
-      t.match(line, lines.shift())
+      t.assert.ok(partialDeepStrictEqual(line, lines.shift()))
       if (lines.length === 0) break
     }
   })
 
-  t.test('Should use serializers inherit from contexts', async (t) => {
+  await t.test('Should use serializers inherit from contexts', async (t) => {
     const lines = [
       { msg: 'incoming request' },
       { test: 'XHello', test2: 'YHello', test3: 'ZHello' },
@@ -123,7 +122,7 @@ t.test('response serialization', (t) => {
     }, stream)
 
     const fastify = Fastify({ loggerInstance })
-    t.teardown(fastify.close.bind(fastify))
+    t.after(() => fastify.close())
 
     fastify.register(context1, { logSerializers: { test2: value => 'Y' + value } })
 
@@ -144,16 +143,16 @@ t.test('response serialization', (t) => {
     {
       const response = await fastify.inject({ method: 'GET', url: '/' })
       const body = await response.json()
-      t.same(body, { hello: 'world' })
+      t.assert.deepStrictEqual(body, { hello: 'world' })
     }
 
     for await (const [line] of on(stream, 'data')) {
-      t.match(line, lines.shift())
+      t.assert.ok(partialDeepStrictEqual(line, lines.shift()))
       if (lines.length === 0) break
     }
   })
 
-  t.test('should serialize request and response', async (t) => {
+  await t.test('should serialize request and response', async (t) => {
     const lines = [
       { req: { method: 'GET', url: '/500' }, msg: 'incoming request' },
       { req: { method: 'GET', url: '/500' }, msg: '500 error' },
@@ -163,7 +162,7 @@ t.test('response serialization', (t) => {
 
     const stream = split(JSON.parse)
     const fastify = Fastify({ logger: { level: 'info', stream } })
-    t.teardown(fastify.close.bind(fastify))
+    t.after(() => fastify.close())
 
     fastify.get('/500', (req, reply) => {
       reply.code(500).send(Error('500 error'))
@@ -173,11 +172,11 @@ t.test('response serialization', (t) => {
 
     {
       const response = await fastify.inject({ method: 'GET', url: '/500' })
-      t.equal(response.statusCode, 500)
+      t.assert.strictEqual(response.statusCode, 500)
     }
 
     for await (const [line] of on(stream, 'data')) {
-      t.match(line, lines.shift())
+      t.assert.ok(partialDeepStrictEqual(line, lines.shift()))
       if (lines.length === 0) break
     }
   })

package/test/nullable-validation.test.js

@@ -1,7 +1,6 @@
 'use strict'
 
 const { test } = require('node:test')
-const sget = require('simple-get').concat
 const Fastify = require('..')
 
 test('nullable string', (t, done) => {
@@ -52,8 +51,8 @@ test('nullable string', (t, done) => {
   })
 })
 
-test('object or null body', (t, done) => {
-  t.plan(5)
+test('object or null body', async (t) => {
+  t.plan(4)
 
   const fastify = Fastify()
 
@@ -88,24 +87,20 @@ test('object or null body', (t, done) => {
     }
   })
 
-  fastify.listen({ port: 0 }, (err) => {
-    t.assert.ifError(err)
-    t.after(() => { fastify.close() })
-
-    sget({
-      method: 'POST',
-      url: 'http://localhost:' + fastify.server.address().port
-    }, (err, response, body) => {
-      t.assert.ifError(err)
-      t.assert.strictEqual(response.statusCode, 200)
-      t.assert.deepStrictEqual(JSON.parse(body), { isUndefinedBody: true })
-      done()
-    })
+  const fastifyServer = await fastify.listen({ port: 0 })
+  t.after(() => { fastify.close() })
+
+  const result = await fetch(fastifyServer, {
+    method: 'POST'
   })
+
+  t.assert.ok(result.ok)
+  t.assert.strictEqual(result.status, 200)
+  t.assert.deepStrictEqual(await result.json(), { isUndefinedBody: true })
 })
 
-test('nullable body', (t, done) => {
-  t.plan(5)
+test('nullable body', async (t) => {
+  t.plan(4)
 
   const fastify = Fastify()
 
@@ -141,24 +136,20 @@ test('nullable body', (t, done) => {
     }
   })
 
-  fastify.listen({ port: 0 }, (err) => {
-    t.assert.ifError(err)
-    t.after(() => fastify.close())
-
-    sget({
-      method: 'POST',
-      url: 'http://localhost:' + fastify.server.address().port
-    }, (err, response, body) => {
-      t.assert.ifError(err)
-      t.assert.strictEqual(response.statusCode, 200)
-      t.assert.deepStrictEqual(JSON.parse(body), { isUndefinedBody: true })
-      done()
-    })
+  const fastifyServer = await fastify.listen({ port: 0 })
+  t.after(() => fastify.close())
+
+  const result = await fetch(fastifyServer, {
+    method: 'POST'
   })
+
+  t.assert.ok(result.ok)
+  t.assert.strictEqual(result.status, 200)
+  t.assert.deepStrictEqual(await result.json(), { isUndefinedBody: true })
 })
 
-test('Nullable body with 204', (t, done) => {
-  t.plan(5)
+test('Nullable body with 204', async (t) => {
+  t.plan(4)
 
   const fastify = Fastify()
 
@@ -183,18 +174,14 @@ test('Nullable body with 204', (t, done) => {
     }
   })
 
-  fastify.listen({ port: 0 }, (err) => {
-    t.assert.ifError(err)
-    t.after(() => fastify.close())
-
-    sget({
-      method: 'POST',
-      url: 'http://localhost:' + fastify.server.address().port
-    }, (err, response, body) => {
-      t.assert.ifError(err)
-      t.assert.strictEqual(response.statusCode, 204)
-      t.assert.strictEqual(body.length, 0)
-      done()
-    })
+  const fastifyServer = await fastify.listen({ port: 0 })
+  t.after(() => fastify.close())
+
+  const result = await fetch(fastifyServer, {
+    method: 'POST'
   })
+
+  t.assert.ok(result.ok)
+  t.assert.strictEqual(result.status, 204)
+  t.assert.strictEqual((await result.text()).length, 0)
 })

package/test/options.error-handler.test.js

@@ -1,5 +1,5 @@
 'use strict'
 
-const t = require('tap')
+const t = require('node:test')
 require('./helper').payloadMethod('options', t, true)
 require('./input-validation').payloadMethod('options', t)

package/test/options.test.js

@@ -1,5 +1,5 @@
 'use strict'
 
-const t = require('tap')
+const t = require('node:test')
 require('./helper').payloadMethod('options', t)
 require('./input-validation').payloadMethod('options', t)

package/test/output-validation.test.js

@@ -1,8 +1,6 @@
 'use strict'
 
-const t = require('tap')
-const test = t.test
-const sget = require('simple-get').concat
+const { test } = require('node:test')
 const fastify = require('..')()
 
 const opts = {
@@ -34,9 +32,9 @@ test('shorthand - output string', t => {
     fastify.get('/string', opts, function (req, reply) {
       reply.code(200).send({ hello: 'world' })
     })
-    t.pass()
+    t.assert.ok(true)
   } catch (e) {
-    t.fail()
+    t.assert.fail()
   }
 })
 
@@ -46,9 +44,9 @@ test('shorthand - output number', t => {
     fastify.get('/number', opts, function (req, reply) {
       reply.code(201).send({ hello: 55 })
     })
-    t.pass()
+    t.assert.ok(true)
   } catch (e) {
-    t.fail()
+    t.assert.fail()
   }
 })
 
@@ -59,9 +57,9 @@ test('wrong object for schema - output', t => {
       // will send { }
       reply.code(201).send({ hello: 'world' })
     })
-    t.pass()
+    t.assert.ok(true)
   } catch (e) {
-    t.fail()
+    t.assert.fail()
   }
 })
 
@@ -72,9 +70,9 @@ test('empty response', t => {
     fastify.get('/empty', opts, function (req, reply) {
       reply.code(204).send()
     })
-    t.pass()
+    t.assert.ok(true)
   } catch (e) {
-    t.fail()
+    t.assert.fail()
   }
 })
 
@@ -84,80 +82,59 @@ test('unlisted response code', t => {
     fastify.get('/400', opts, function (req, reply) {
       reply.code(400).send({ hello: 'DOOM' })
     })
-    t.pass()
+    t.assert.ok(true)
   } catch (e) {
-    t.fail()
+    t.assert.fail()
   }
 })
 
-fastify.listen({ port: 0 }, err => {
-  t.error(err)
-  t.teardown(() => { fastify.close() })
+test('start server and run tests', async (t) => {
+  const fastifyServer = await fastify.listen({ port: 0 })
+  t.after(() => fastify.close())
 
-  test('shorthand - string get ok', t => {
-    t.plan(4)
-    sget({
-      method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/string'
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 200)
-      t.equal(response.headers['content-length'], '' + body.length)
-      t.same(JSON.parse(body), { hello: 'world' })
-    })
+  await test('shorthand - string get ok', async (t) => {
+    const result = await fetch(fastifyServer + '/string')
+    t.assert.ok(result.ok)
+    t.assert.strictEqual(result.status, 200)
+    const body = await result.text()
+    t.assert.strictEqual(result.headers.get('content-length'), '' + body.length)
+    t.assert.deepStrictEqual(JSON.parse(body), { hello: 'world' })
   })
 
-  test('shorthand - number get ok', t => {
-    t.plan(4)
-    sget({
-      method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/number'
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 201)
-      t.equal(response.headers['content-length'], '' + body.length)
-      t.same(JSON.parse(body), { hello: 55 })
-    })
+  await test('shorthand - number get ok', async (t) => {
+    const result = await fetch(fastifyServer + '/number')
+    t.assert.ok(result.ok)
+    t.assert.strictEqual(result.status, 201)
+    const body = await result.text()
+    t.assert.strictEqual(result.headers.get('content-length'), '' + body.length)
+    t.assert.deepStrictEqual(JSON.parse(body), { hello: 55 })
   })
 
-  test('shorthand - wrong-object-for-schema', t => {
-    t.plan(4)
-    sget({
-      method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/wrong-object-for-schema'
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 500)
-      t.equal(response.headers['content-length'], '' + body.length)
-      t.same(JSON.parse(body), {
-        statusCode: 500,
-        error: 'Internal Server Error',
-        message: 'The value "world" cannot be converted to a number.'
-      })
+  await test('shorthand - wrong-object-for-schema', async (t) => {
+    const result = await fetch(fastifyServer + '/wrong-object-for-schema')
+    t.assert.ok(!result.ok)
+    t.assert.strictEqual(result.status, 500)
+    const body = await result.text()
+    t.assert.strictEqual(result.headers.get('content-length'), '' + body.length)
+    t.assert.deepStrictEqual(JSON.parse(body), {
+      statusCode: 500,
+      error: 'Internal Server Error',
+      message: 'The value "world" cannot be converted to a number.'
     })
   })
 
-  test('shorthand - empty', t => {
-    t.plan(2)
-    sget({
-      method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/empty'
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 204)
-    })
+  await test('shorthand - empty', async (t) => {
+    const result = await fetch(fastifyServer + '/empty')
+    t.assert.ok(result.ok)
+    t.assert.strictEqual(result.status, 204)
   })
 
-  test('shorthand - 400', t => {
-    t.plan(4)
-    sget({
-      method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/400'
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 400)
-      t.equal(response.headers['content-length'], '' + body.length)
-      t.same(JSON.parse(body), { hello: 'DOOM' })
-    })
+  await test('shorthand - 400', async (t) => {
+    const result = await fetch(fastifyServer + '/400')
+    t.assert.ok(!result.ok)
+    t.assert.strictEqual(result.status, 400)
+    const body = await result.text()
+    t.assert.strictEqual(result.headers.get('content-length'), '' + body.length)
+    t.assert.deepStrictEqual(JSON.parse(body), { hello: 'DOOM' })
   })
 })

package/test/patch.error-handler.test.js

@@ -1,5 +1,5 @@
 'use strict'
 
-const t = require('tap')
+const t = require('node:test')
 require('./helper').payloadMethod('patch', t, true)
 require('./input-validation').payloadMethod('patch', t)

package/test/patch.test.js

@@ -1,5 +1,5 @@
 'use strict'
 
-const t = require('tap')
+const t = require('node:test')
 require('./helper').payloadMethod('patch', t)
 require('./input-validation').payloadMethod('patch', t)