fastify 3.9.2 → 3.12.0

package/test/close.test.js

@@ -27,7 +27,7 @@ test('close callback', t => {
 test('inside register', t => {
   t.plan(5)
   const fastify = Fastify()
-  fastify.register(function (f, opts, next) {
+  fastify.register(function (f, opts, done) {
     f.addHook('onClose', onClose)
     function onClose (instance, done) {
       t.ok(instance.prototype === fastify.prototype)
@@ -35,7 +35,7 @@ test('inside register', t => {
       done()
     }
 
-    next()
+    done()
   })
 
   fastify.listen(0, err => {
@@ -53,13 +53,13 @@ test('close order', t => {
   const fastify = Fastify()
   const order = [1, 2, 3]
 
-  fastify.register(function (f, opts, next) {
+  fastify.register(function (f, opts, done) {
     f.addHook('onClose', (instance, done) => {
       t.is(order.shift(), 1)
       done()
     })
 
-    next()
+    done()
   })
 
   fastify.addHook('onClose', (instance, done) => {
@@ -82,12 +82,12 @@ test('close order - async', async t => {
   const fastify = Fastify()
   const order = [1, 2, 3]
 
-  fastify.register(function (f, opts, next) {
+  fastify.register(function (f, opts, done) {
     f.addHook('onClose', async instance => {
       t.is(order.shift(), 1)
     })
 
-    next()
+    done()
   })
 
   fastify.addHook('onClose', () => {
@@ -119,7 +119,7 @@ test('onClose should keep the context', t => {
   const fastify = Fastify()
   fastify.register(plugin)
 
-  function plugin (instance, opts, next) {
+  function plugin (instance, opts, done) {
     instance.decorate('test', true)
     instance.addHook('onClose', onClose)
     t.ok(instance.prototype === fastify.prototype)
@@ -130,7 +130,7 @@ test('onClose should keep the context', t => {
       done()
     }
 
-    next()
+    done()
   }
 
   fastify.close((err) => {

package/test/context-config.test.js

@@ -67,3 +67,55 @@ test('config', t => {
     t.deepEquals(JSON.parse(response.payload), { url: '/no-config', method: 'GET' })
   })
 })
+
+test('config with exposeHeadRoutes', t => {
+  t.plan(9)
+  const fastify = Fastify({ exposeHeadRoutes: true })
+
+  fastify.get('/get', {
+    schema: schema.schema,
+    config: Object.assign({}, schema.config)
+  }, handler)
+
+  fastify.route({
+    method: 'GET',
+    url: '/route',
+    schema: schema.schema,
+    handler: handler,
+    config: Object.assign({}, schema.config)
+  })
+
+  fastify.route({
+    method: 'GET',
+    url: '/no-config',
+    schema: schema.schema,
+    handler: handler
+  })
+
+  fastify.inject({
+    method: 'GET',
+    url: '/get'
+  }, (err, response) => {
+    t.error(err)
+    t.strictEqual(response.statusCode, 200)
+    t.deepEquals(JSON.parse(response.payload), Object.assign({ url: '/get', method: 'GET' }, schema.config))
+  })
+
+  fastify.inject({
+    method: 'GET',
+    url: '/route'
+  }, (err, response) => {
+    t.error(err)
+    t.strictEqual(response.statusCode, 200)
+    t.deepEquals(JSON.parse(response.payload), Object.assign({ url: '/route', method: 'GET' }, schema.config))
+  })
+
+  fastify.inject({
+    method: 'GET',
+    url: '/no-config'
+  }, (err, response) => {
+    t.error(err)
+    t.strictEqual(response.statusCode, 200)
+    t.deepEquals(JSON.parse(response.payload), { url: '/no-config', method: 'GET' })
+  })
+})

package/test/custom-parser.test.js

@@ -298,18 +298,18 @@ test('contentTypeParser should support encapsulation', t => {
   t.plan(6)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     instance.addContentTypeParser('application/jsoff', () => {})
     t.ok(instance.hasContentTypeParser('application/jsoff'))
 
-    instance.register((instance, opts, next) => {
+    instance.register((instance, opts, done) => {
       instance.addContentTypeParser('application/ffosj', () => {})
       t.ok(instance.hasContentTypeParser('application/jsoff'))
       t.ok(instance.hasContentTypeParser('application/ffosj'))
-      next()
+      done()
     })
 
-    next()
+    done()
   })
 
   fastify.ready(err => {
@@ -323,7 +323,7 @@ test('contentTypeParser should support encapsulation, second try', t => {
   t.plan(4)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     instance.post('/', (req, reply) => {
       reply.send(req.body)
     })
@@ -334,7 +334,7 @@ test('contentTypeParser should support encapsulation, second try', t => {
       })
     })
 
-    next()
+    done()
   })
 
   fastify.listen(0, err => {
@@ -662,7 +662,7 @@ test('Can override the default json parser in a plugin', t => {
   t.plan(5)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     instance.addContentTypeParser('application/json', function (req, payload, done) {
       t.ok('called')
       jsonParser(payload, function (err, body) {
@@ -674,7 +674,7 @@ test('Can override the default json parser in a plugin', t => {
       reply.send(req.body)
     })
 
-    next()
+    done()
   })
 
   fastify.listen(0, err => {

package/test/decorator.test.js

@@ -17,14 +17,14 @@ test('server methods should exist', t => {
   t.ok(fastify.hasDecorator)
 })
 
-test('server methods should be incapsulated via .register', t => {
+test('server methods should be encapsulated via .register', t => {
   t.plan(2)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     instance.decorate('test', () => {})
     t.ok(instance.test)
-    next()
+    done()
   })
 
   fastify.ready(() => {
@@ -36,10 +36,10 @@ test('hasServerMethod should check if the given method already exist', t => {
   t.plan(2)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     instance.decorate('test', () => {})
     t.ok(instance.hasDecorator('test'))
-    next()
+    done()
   })
 
   fastify.ready(() => {
@@ -51,7 +51,7 @@ test('decorate should throw if a declared dependency is not present', t => {
   t.plan(3)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     try {
       instance.decorate('test', () => {}, ['dependency'])
       t.fail()
@@ -59,7 +59,7 @@ test('decorate should throw if a declared dependency is not present', t => {
       t.is(e.code, 'FST_ERR_DEC_MISSING_DEPENDENCY')
       t.is(e.message, 'The decorator is missing dependency \'dependency\'.')
     }
-    next()
+    done()
   })
 
   fastify.ready(() => t.pass())
@@ -70,8 +70,8 @@ test('should pass error for missing request decorator', t => {
   t.plan(2)
   const fastify = Fastify()
 
-  const plugin = fp(function (instance, opts, next) {
-    next()
+  const plugin = fp(function (instance, opts, done) {
+    done()
   }, {
     decorators: {
       request: ['foo']
@@ -89,7 +89,7 @@ test('decorateReply inside register', t => {
   t.plan(12)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     instance.decorateReply('test', 'test')
     t.ok(instance[symbols.kReply].prototype.test)
 
@@ -98,7 +98,7 @@ test('decorateReply inside register', t => {
       reply.send({ hello: 'world' })
     })
 
-    next()
+    done()
   })
 
   fastify.get('/no', (req, reply) => {
@@ -136,7 +136,7 @@ test('decorateReply as plugin (inside .after)', t => {
   t.plan(11)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     instance.register(fp((i, o, n) => {
       instance.decorateReply('test', 'test')
       n()
@@ -146,7 +146,7 @@ test('decorateReply as plugin (inside .after)', t => {
         reply.send({ hello: 'world' })
       })
     })
-    next()
+    done()
   })
 
   fastify.get('/no', (req, reply) => {
@@ -184,7 +184,7 @@ test('decorateReply as plugin (outside .after)', t => {
   t.plan(11)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     instance.register(fp((i, o, n) => {
       instance.decorateReply('test', 'test')
       n()
@@ -194,7 +194,7 @@ test('decorateReply as plugin (outside .after)', t => {
       t.ok(reply.test)
       reply.send({ hello: 'world' })
     })
-    next()
+    done()
   })
 
   fastify.get('/no', (req, reply) => {
@@ -232,7 +232,7 @@ test('decorateRequest inside register', t => {
   t.plan(12)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     instance.decorateRequest('test', 'test')
     t.ok(instance[symbols.kRequest].prototype.test)
 
@@ -241,7 +241,7 @@ test('decorateRequest inside register', t => {
       reply.send({ hello: 'world' })
     })
 
-    next()
+    done()
   })
 
   fastify.get('/no', (req, reply) => {
@@ -279,7 +279,7 @@ test('decorateRequest as plugin (inside .after)', t => {
   t.plan(11)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     instance.register(fp((i, o, n) => {
       instance.decorateRequest('test', 'test')
       n()
@@ -289,7 +289,7 @@ test('decorateRequest as plugin (inside .after)', t => {
         reply.send({ hello: 'world' })
       })
     })
-    next()
+    done()
   })
 
   fastify.get('/no', (req, reply) => {
@@ -327,7 +327,7 @@ test('decorateRequest as plugin (outside .after)', t => {
   t.plan(11)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     instance.register(fp((i, o, n) => {
       instance.decorateRequest('test', 'test')
       n()
@@ -337,7 +337,7 @@ test('decorateRequest as plugin (outside .after)', t => {
       t.ok(req.test)
       reply.send({ hello: 'world' })
     })
-    next()
+    done()
   })
 
   fastify.get('/no', (req, reply) => {
@@ -413,10 +413,10 @@ test('hasRequestDecorator', t => {
 
     t.notOk(fastify.hasRequestDecorator(requestDecoratorName))
 
-    fastify.register(function (fastify2, opts, next) {
+    fastify.register(function (fastify2, opts, done) {
       fastify2.decorateRequest(requestDecoratorName, 42)
       t.ok(fastify2.hasRequestDecorator(requestDecoratorName))
-      next()
+      done()
     })
 
     t.notOk(fastify.hasRequestDecorator(requestDecoratorName))
@@ -432,9 +432,9 @@ test('hasRequestDecorator', t => {
 
     fastify.decorateRequest(requestDecoratorName, 42)
 
-    fastify.register(function (fastify2, opts, next) {
+    fastify.register(function (fastify2, opts, done) {
       t.ok(fastify2.hasRequestDecorator(requestDecoratorName))
-      next()
+      done()
     })
 
     fastify.ready(function () {
@@ -469,10 +469,10 @@ test('hasReplyDecorator', t => {
 
     t.notOk(fastify.hasReplyDecorator(replyDecoratorName))
 
-    fastify.register(function (fastify2, opts, next) {
+    fastify.register(function (fastify2, opts, done) {
       fastify2.decorateReply(replyDecoratorName, 42)
       t.ok(fastify2.hasReplyDecorator(replyDecoratorName))
-      next()
+      done()
     })
 
     t.notOk(fastify.hasReplyDecorator(replyDecoratorName))
@@ -488,9 +488,9 @@ test('hasReplyDecorator', t => {
 
     fastify.decorateReply(replyDecoratorName, 42)
 
-    fastify.register(function (fastify2, opts, next) {
+    fastify.register(function (fastify2, opts, done) {
       t.ok(fastify2.hasReplyDecorator(replyDecoratorName))
-      next()
+      done()
     })
 
     fastify.ready(function () {
@@ -505,7 +505,7 @@ test('should register properties via getter/setter objects', t => {
   t.plan(3)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     instance.decorate('test', {
       getter () {
         return 'a getter'
@@ -513,7 +513,7 @@ test('should register properties via getter/setter objects', t => {
     })
     t.ok(instance.test)
     t.is(instance.test, 'a getter')
-    next()
+    done()
   })
 
   fastify.ready(() => {
@@ -525,7 +525,7 @@ test('decorateRequest should work with getter/setter', t => {
   t.plan(5)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     instance.decorateRequest('test', {
       getter () {
         return 'a getter'
@@ -536,7 +536,7 @@ test('decorateRequest should work with getter/setter', t => {
       res.send({ test: req.test })
     })
 
-    next()
+    done()
   })
 
   fastify.get('/not-decorated', (req, res) => {
@@ -561,7 +561,7 @@ test('decorateReply should work with getter/setter', t => {
   t.plan(5)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     instance.decorateReply('test', {
       getter () {
         return 'a getter'
@@ -572,7 +572,7 @@ test('decorateReply should work with getter/setter', t => {
       res.send({ test: res.test })
     })
 
-    next()
+    done()
   })
 
   fastify.get('/not-decorated', (req, res) => {
@@ -597,10 +597,10 @@ test('should register empty values', t => {
   t.plan(2)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     instance.decorate('test', null)
     t.true(instance.hasOwnProperty('test'))
-    next()
+    done()
   })
 
   fastify.ready(() => {
@@ -617,7 +617,7 @@ test('nested plugins can override things', t => {
   fastify.decorateRequest('test', rootFunc)
   fastify.decorateReply('test', rootFunc)
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     const func = () => {}
     instance.decorate('test', func)
     instance.decorateRequest('test', func)
@@ -626,7 +626,7 @@ test('nested plugins can override things', t => {
     t.equal(instance.test, func)
     t.equal(instance[symbols.kRequest].prototype.test, func)
     t.equal(instance[symbols.kReply].prototype.test, func)
-    next()
+    done()
   })
 
   fastify.ready(() => {
@@ -640,24 +640,24 @@ test('a decorator should addSchema to all the encapsulated tree', t => {
   t.plan(1)
   const fastify = Fastify()
 
-  const decorator = function (instance, opts, next) {
+  const decorator = function (instance, opts, done) {
     instance.decorate('decoratorAddSchema', function (whereAddTheSchema) {
       instance.addSchema({
         $id: 'schema',
         type: 'string'
       })
     })
-    next()
+    done()
   }
 
   fastify.register(fp(decorator))
 
-  fastify.register(function (instance, opts, next) {
-    instance.register((subInstance, opts, next) => {
+  fastify.register(function (instance, opts, done) {
+    instance.register((subInstance, opts, done) => {
       subInstance.decoratorAddSchema()
-      next()
+      done()
     })
-    next()
+    done()
   })
 
   fastify.ready(t.error)
@@ -671,10 +671,10 @@ test('after can access to a decorated instance and previous plugin decoration',
 
   const fastify = Fastify()
 
-  fastify.register(fp(function (instance, options, next) {
+  fastify.register(fp(function (instance, options, done) {
     instance.decorate('test', TEST_VALUE)
 
-    next()
+    done()
   })).after(function (err, instance, done) {
     t.error(err)
     t.equal(instance.test, TEST_VALUE)
@@ -683,13 +683,13 @@ test('after can access to a decorated instance and previous plugin decoration',
     done()
   })
 
-  fastify.register(fp(function (instance, options, next) {
+  fastify.register(fp(function (instance, options, done) {
     t.equal(instance.test, TEST_VALUE)
     t.equal(instance.test2, OTHER_TEST_VALUE)
 
     instance.decorate('test3', NEW_TEST_VALUE)
 
-    next()
+    done()
   })).after(function (err, instance, done) {
     t.error(err)
     t.equal(instance.test, TEST_VALUE)

package/test/default-route.test.js

@@ -0,0 +1,43 @@
+'use strict'
+
+const t = require('tap')
+const test = t.test
+const Fastify = require('..')
+
+test('should fail if defaultRoute is not a function', t => {
+  t.plan(1)
+
+  const fastify = Fastify()
+  const defaultRoute = {}
+
+  fastify.get('/', () => {})
+
+  try {
+    fastify.setDefaultRoute(defaultRoute)
+  } catch (error) {
+    t.equal(error.code, 'FST_ERR_DEFAULT_ROUTE_INVALID_TYPE')
+  }
+})
+
+test('correctly sets, returns, and calls defaultRoute', t => {
+  t.plan(3)
+
+  const fastify = Fastify()
+  const defaultRoute = (req, res) => {
+    res.end('hello from defaultRoute')
+  }
+
+  fastify.setDefaultRoute(defaultRoute)
+  const returnedDefaultRoute = fastify.getDefaultRoute()
+  t.equal(returnedDefaultRoute, defaultRoute)
+
+  fastify.get('/', () => {})
+
+  fastify.inject({
+    method: 'GET',
+    url: '/random'
+  }, (err, res) => {
+    t.error(err)
+    t.equal(res.body, 'hello from defaultRoute')
+  })
+})

package/test/fastify-instance.test.js

@@ -79,7 +79,7 @@ test('errorHandler in plugin should be separate from the external one', async t
   t.plan(4)
   const fastify = Fastify()
 
-  fastify.register((instance, opts, next) => {
+  fastify.register((instance, opts, done) => {
     const inPluginErrHandler = (_, __, reply) => {
       reply.send({ plugin: 'error-object' })
     }
@@ -89,7 +89,7 @@ test('errorHandler in plugin should be separate from the external one', async t
     t.notSame(instance.errorHandler, fastify.errorHandler)
     t.equal(instance.errorHandler.name, 'bound inPluginErrHandler')
 
-    next()
+    done()
   })
 
   await fastify.ready()

package/test/fluent-schema.test.js

@@ -3,9 +3,9 @@
 const t = require('tap')
 const test = t.test
 const Fastify = require('..')
-const S = require('fluent-schema')
+const S = require('fluent-json-schema')
 
-test('use fluent-schema object', t => {
+test('use fluent-json-schema object', t => {
   t.plan(15)
   const fastify = Fastify()
 
@@ -90,7 +90,7 @@ test('use fluent-schema object', t => {
   })
 })
 
-test('use complex fluent-schema object', t => {
+test('use complex fluent-json-schema object', t => {
   t.plan(1)
   const fastify = Fastify()
 

package/test/handler-context.test.js

@@ -8,9 +8,9 @@ test('handlers receive correct `this` context', (t) => {
   t.plan(4)
 
   // simulate plugin that uses fastify-plugin
-  const plugin = function (instance, opts, next) {
+  const plugin = function (instance, opts, done) {
     instance.decorate('foo', 'foo')
-    next()
+    done()
   }
   plugin[Symbol.for('skip-override')] = true
 

package/test/hooks-async.test.js

@@ -530,7 +530,7 @@ test('preHandler respond with a stream', t => {
   })
 
   // we are calling `reply.send` inside the `preHandler` hook with a stream,
-  // this triggers the `onSend` hook event if `preHanlder` has not yet finished
+  // this triggers the `onSend` hook event if `preHandler` has not yet finished
   const order = [1, 2]
 
   fastify.addHook('preHandler', async (req, reply) => {
@@ -655,8 +655,8 @@ test('preSerializationEnd should handle errors if the serialize method throws',
     t.plan(2)
     const fastify = Fastify()
 
-    fastify.addHook('preSerialization', (request, reply, payload, next) => {
-      next(null, payload)
+    fastify.addHook('preSerialization', (request, reply, payload, done) => {
+      done(null, payload)
     })
 
     fastify.post('/', {