refactor: prefix unused params with underscores

lib/routes.js

@@ -28,7 +28,7 @@ function fastifySwagger (fastify, opts, done) {
   }
 
   if (typeof staticCSP === 'string' || typeof opts.transformStaticCSP === 'function') {
-    fastify.addHook('onSend', function (request, reply, payload, done) {
+    fastify.addHook('onSend', function (_request, reply, _payload, done) {
       // set static csp when it is passed
       if (typeof staticCSP === 'string') {
         reply.header('content-security-policy', staticCSP.trim())
@@ -62,7 +62,7 @@ function fastifySwagger (fastify, opts, done) {
           method: 'GET',
           schema: { hide: true },
           ...hooks,
-          handler: (req, reply) => {
+          handler: (_req, reply) => {
             reply
               .header('content-type', 'text/css; charset=UTF-8')
               .send(cssFile.content)
@@ -78,7 +78,7 @@ function fastifySwagger (fastify, opts, done) {
           method: 'GET',
           schema: { hide: true },
           ...hooks,
-          handler: (req, reply) => {
+          handler: (_req, reply) => {
             reply
               .header('content-type', 'application/javascript; charset=utf-8')
               .send(jsFile.content)
@@ -94,7 +94,7 @@ function fastifySwagger (fastify, opts, done) {
           method: 'GET',
           schema: { hide: true },
           ...hooks,
-          handler: (req, reply) => {
+          handler: (_req, reply) => {
             reply
               .header('content-type', favicon.type)
               .send(favicon.content)
@@ -135,7 +135,7 @@ function fastifySwagger (fastify, opts, done) {
     method: 'GET',
     schema: { hide: true },
     ...hooks,
-    handler: (req, reply) => {
+    handler: (_req, reply) => {
       reply
         .header('content-type', 'application/javascript; charset=utf-8')
         .send(swaggerInitializerContent)
@@ -158,7 +158,7 @@ function fastifySwagger (fastify, opts, done) {
         : function (req, reply) {
           reply.send(transformSpecification(fastify.swagger(), req, reply))
         }
-      : function (req, reply) {
+      : function (_req, reply) {
         reply.send(fastify.swagger())
       }
   })
@@ -180,7 +180,7 @@ function fastifySwagger (fastify, opts, done) {
             .type('application/x-yaml')
             .send(yaml.stringify(transformSpecification(fastify.swagger(), req, reply)))
         }
-      : function (req, reply) {
+      : function (_req, reply) {
         reply
           .type('application/x-yaml')
           .send(fastify.swagger({ yaml: true }))

test/hooks.test.js

@@ -9,7 +9,7 @@ const fastifySwaggerUi = require('../index')
 const { swaggerOption, schemaBody } = require('../examples/options')
 
 const authOptions = {
-  validate (username, password, req, reply, done) {
+  validate (username, password, _req, _reply, done) {
     if (username === 'admin' && password === 'admin') {
       done()
     } else {
@@ -116,7 +116,7 @@ test('catch all added schema', async t => {
   await fastify.register(fastifySwagger, {
     openapi: {},
     refResolver: {
-      buildLocalReference: (json, baseUri, fragment, i) => {
+      buildLocalReference: (json, _baseUri, _fragment, i) => {
         return json.$id || `def-${i}`
       }
     }

test/integration.test.js

@@ -33,7 +33,7 @@ test('fastify will response swagger csp', async (t) => {
   })
 
   // route for testing CSP headers
-  fastify.get('/', (req, reply) => {
+  fastify.get('/', (_req, reply) => {
     reply.send({
       foo: 'bar'
     })

test/route.test.js

@@ -197,7 +197,7 @@ test('/v1/foobar should display index html', async (t) => {
   t.plan(4)
   const fastify = Fastify()
 
-  fastify.register(async function (fastify, options) {
+  fastify.register(async function (fastify) {
     await fastify.register(fastifySwagger, swaggerOption)
     await fastify.register(fastifySwaggerUi, { routePrefix: '/foobar', noRedirect: true })
 
@@ -223,7 +223,7 @@ test('/v1/foobar/ should display index html', async (t) => {
   t.plan(4)
   const fastify = Fastify()
 
-  fastify.register(async function (fastify, options) {
+  fastify.register(async function (fastify) {
     await fastify.register(fastifySwagger, swaggerOption)
     await fastify.register(fastifySwaggerUi, { routePrefix: '/foobar' })
 
@@ -476,7 +476,7 @@ test('/documentation/:myfile should return 404 in dynamic mode', async (t) => {
 test('/documentation/:myfile should run custom NotFoundHandler in dynamic mode', async (t) => {
   t.plan(1)
   const fastify = Fastify()
-  const notFoundHandler = function (req, reply) {
+  const notFoundHandler = function (_req, reply) {
     reply.code(410).send()
   }
   fastify.setNotFoundHandler(notFoundHandler)

test/static.test.js

@@ -224,7 +224,7 @@ test('/documentation/non-existing-file calls custom NotFoundHandler', async (t)
   const fastify = Fastify()
   await fastify.register(fastifySwagger, config)
   await fastify.register(fastifySwaggerUi)
-  fastify.setNotFoundHandler((request, reply) => {
+  fastify.setNotFoundHandler((_request, reply) => {
     reply.code(410).send()
   })
 

types/http2-types.test-d.ts

@@ -20,15 +20,15 @@ app.put('/some-route/:id', {
     summary: 'qwerty',
     security: [{ apiKey: [] }]
   }
-}, (req, reply) => {})
+}, () => {})
 
 app.get('/public/route', {
   schema: {
     description: 'returns 200 OK',
     summary: 'qwerty',
     security: []
   }
-}, (_req, _reply) => {})
+}, () => {})
 
 app
   .register(fastifySwaggerUi, {

types/types.test-d.ts

@@ -21,8 +21,8 @@ const initOAuth: FastifySwaggerInitOAuthOptions = {
   scopes: ['openid', 'profile', 'email', 'offline_access'],
 }
 const uiHooks: FastifySwaggerUiHooksOptions = {
-  onRequest: (request, reply, done) => { done() },
-  preHandler: (request, reply, done) => { done() },
+  onRequest: (_request, _reply, done) => { done() },
+  preHandler: (_request, _reply, done) => { done() },
 }
 
 app.register(fastifySwaggerUi)
@@ -43,7 +43,7 @@ app.get('/deprecated', {
     deprecated: true,
     hide: true
   }
-}, (req, reply) => {})
+}, () => {})
 
 app.put('/some-route/:id', {
   schema: {
@@ -58,7 +58,7 @@ app.put('/some-route/:id', {
       description: 'Find more info here'
     },
   }
-}, (req, reply) => {})
+}, () => {})
 
 app.put('/image.png', {
   schema: {
@@ -73,7 +73,7 @@ app.put('/image.png', {
       }
     }
   }
-}, async (req, reply) => {
+}, async (_req, reply) => {
   reply
     .type('image/png')
     .send(Buffer.from('iVBORw0KGgoAAAANSUhEUgAAAAUAAAAFCAIAAAACDbGyAAAAAXNSR0IArs4c6QAAAARnQU1BAACxjwv8YQUAAAAJcEhZcwAAEnQAABJ0Ad5mH3gAAAAgSURBVBhXY/iPCkB8BgYkEiSIBICiCCEoB0SBwf///wGHRzXLSklJLQAAAABJRU5ErkJggg==', 'base64'))
@@ -89,7 +89,7 @@ app.get('/public/route', {
   links: {
     200: { 'some-route': { operationId: 'opeId' } }
   }
-}, (req, reply) => {})
+}, () => {})
 
 app
   .register(fastifySwaggerUi, {