test: prefix unused params with underscores (#437)

fastify · Jan 11, 2025 · 26d60ea · 26d60ea
1 parent 464b04a
commit 26d60ea
Show file tree

Hide file tree

Showing 20 changed files with 29 additions and 29 deletions.
diff --git a/test/issues/369/routes/.autohooks.js b/test/issues/369/routes/.autohooks.js
@@ -1,7 +1,7 @@
 'use strict'
 
-module.exports = async function (app, opts) {
-  app.addHook('onRequest', async (req, reply) => {
+module.exports = async function (app) {
+  app.addHook('onRequest', async (req) => {
     req.hooked = req.hooked || []
     req.hooked.push('root')
   })

diff --git a/test/issues/369/routes/child/.autohooks.js b/test/issues/369/routes/child/.autohooks.js
@@ -1,7 +1,7 @@
 'use strict'
 
-module.exports = async function (app, opts) {
-  app.addHook('onRequest', async (req, reply) => {
+module.exports = async function (app) {
+  app.addHook('onRequest', async (req) => {
     req.hooked = req.hooked || []
     req.hooked.push('child')
   })

diff --git a/test/issues/369/routes/child/routes.js b/test/issues/369/routes/child/routes.js
@@ -1,6 +1,6 @@
 'use strict'
 
-module.exports = async function (app, opts) {
+module.exports = async function (app) {
   app.get('/', async function (req, reply) {
     reply.status(200).send({ hooked: req.hooked })
   })

diff --git a/test/issues/369/routes/promisified/.autohooks.js b/test/issues/369/routes/promisified/.autohooks.js
@@ -1,8 +1,8 @@
 'use strict'
 
 module.exports = new Promise((resolve) => {
-  resolve(async function (app, opts) {
-    app.addHook('onRequest', async (req, reply) => {
+  resolve(async function (app) {
+    app.addHook('onRequest', async (req) => {
       req.hooked = req.hooked || []
       req.hooked.push('promisified')
     })

diff --git a/test/issues/369/routes/promisified/routes.js b/test/issues/369/routes/promisified/routes.js
@@ -1,6 +1,6 @@
 'use strict'
 
-module.exports = async function (app, opts) {
+module.exports = async function (app) {
   app.get('/', async function (req, reply) {
     reply.status(200).send({ hooked: req.hooked })
   })

diff --git a/test/issues/369/routes/routes.js b/test/issues/369/routes/routes.js
@@ -1,6 +1,6 @@
 'use strict'
 
-module.exports = async function (app, opts) {
+module.exports = async function (app) {
   app.get('/', async function (req, reply) {
     reply.status(200).send({ hooked: req.hooked })
   })

diff --git a/test/issues/374/routes/entity/.autohooks.js b/test/issues/374/routes/entity/.autohooks.js
@@ -1,7 +1,7 @@
 'use strict'
 
-module.exports = async function (app, opts) {
-  app.addHook('onRequest', async (req, reply) => {
+module.exports = async function (app) {
+  app.addHook('onRequest', async (req) => {
     req.hooked = req.hooked || []
     req.hooked.push('root')
   })

diff --git a/test/issues/374/routes/entity/_entity/index.js b/test/issues/374/routes/entity/_entity/index.js
@@ -1,6 +1,6 @@
 'use strict'
 
-module.exports = async function (app, opts) {
+module.exports = async function (app) {
   app.get('/', async function (req, reply) {
     reply.status(200).send({ path: req.url, hooked: req.hooked, params: req.params })
   })

diff --git a/test/issues/374/routes/entity/index.js b/test/issues/374/routes/entity/index.js
@@ -1,6 +1,6 @@
 'use strict'
 
-module.exports = async function (app, opts) {
+module.exports = async function (app) {
   app.get('/', async function (req, reply) {
     reply.status(200).send({ path: req.url, hooked: req.hooked })
   })

diff --git a/test/issues/374/routes/index.js b/test/issues/374/routes/index.js
@@ -1,6 +1,6 @@
 'use strict'
 
-module.exports = async function (app, opts) {
+module.exports = async function (app) {
   app.get('/', async function (req, reply) {
     reply.status(200).send({ path: req.url })
   })

diff --git a/test/issues/376/routes/entity/.autohooks.js b/test/issues/376/routes/entity/.autohooks.js
@@ -1,7 +1,7 @@
 'use strict'
 
-module.exports = async function (app, opts) {
-  app.addHook('onRequest', async (req, reply) => {
+module.exports = async function (app) {
+  app.addHook('onRequest', async (req) => {
     req.hooked = req.hooked || []
     req.hooked.push('root')
   })

diff --git a/test/issues/376/routes/entity/index.js b/test/issues/376/routes/entity/index.js
@@ -1,6 +1,6 @@
 'use strict'
 
-module.exports = async function (app, opts) {
+module.exports = async function (app) {
   app.get('/', async function (req, reply) {
     reply.status(200).send({ path: req.url, hooked: req.hooked })
   })

diff --git a/test/issues/376/routes/index.js b/test/issues/376/routes/index.js
@@ -1,6 +1,6 @@
 'use strict'
 
-module.exports = async function (app, opts) {
+module.exports = async function (app) {
   app.get('/', async function (req, reply) {
     reply.status(200).send({ path: req.url })
   })

diff --git a/test/issues/388/routes/foo.tsx b/test/issues/388/routes/foo.tsx
@@ -1,7 +1,7 @@
 'use strict'
 
-module.exports = async function (app, opts) {
-  app.get('/', async function (req, reply) {
+module.exports = async function (app) {
+  app.get('/', async function (_req, reply) {
     reply.status(200).send({ tsx: 'ok' })
   })
 }
diff --git a/test/typescript-jest/basic/app.ts b/test/typescript-jest/basic/app.ts
@@ -2,7 +2,7 @@ import { FastifyPluginCallback } from 'fastify'
 import { join } from 'node:path'
 const fastifyAutoLoad = require('../../../index')
 
-const app: FastifyPluginCallback = function (fastify, opts, next): void {
+const app: FastifyPluginCallback = function (fastify, _opts, next): void {
   fastify.register(fastifyAutoLoad, {
     dir: join(__dirname, 'foo'),
   })

diff --git a/test/typescript-jest/basic/foo/typescript.ts b/test/typescript-jest/basic/foo/typescript.ts
@@ -1,7 +1,7 @@
 import { FastifyPluginCallback } from 'fastify'
 
-const plugin: FastifyPluginCallback = function (fastify, opts, next): void {
-  fastify.get('/typescript', (request, reply): void => {
+const plugin: FastifyPluginCallback = function (fastify, _opts, next): void {
+  fastify.get('/typescript', (_request, reply): void => {
     reply.send({ script: 'type' })
   })
 

diff --git a/test/typescript-jest/integration/integration.test.ts b/test/typescript-jest/integration/integration.test.ts
@@ -4,7 +4,7 @@ describe('integration test', function () {
   test.concurrent.each(['ts-node', 'ts-node-dev'])(
     'integration with %s',
     async function (instance) {
-      await new Promise(function (resolve, reject) {
+      await new Promise(function (resolve) {
         const child = exec(`${instance} "${process.cwd()}/test/typescript-jest/integration/instance.ts"`)
         let stderr = ''
         child.stderr?.on('data', function (b) {

diff --git a/test/typescript/basic/app.ts b/test/typescript/basic/app.ts
@@ -2,7 +2,7 @@ import { FastifyPluginCallback } from 'fastify'
 import { join } from 'node:path'
 import fastifyAutoLoad from '../../../'
 
-const app: FastifyPluginCallback = function (fastify, opts, next): void {
+const app: FastifyPluginCallback = function (fastify, _opts, next): void {
   fastify.register(fastifyAutoLoad, {
     dir: join(__dirname, 'foo'),
   })

diff --git a/test/typescript/basic/foo/javascript.js b/test/typescript/basic/foo/javascript.js
@@ -1,7 +1,7 @@
 'use strict'
 
-module.exports = function (fastify, opts, next) {
-  fastify.get('/javascript', (request, reply) => {
+module.exports = function (fastify, _opts, next) {
+  fastify.get('/javascript', (_request, reply) => {
     reply.send({ script: 'java' })
   })
 

diff --git a/test/typescript/basic/foo/typescript.ts b/test/typescript/basic/foo/typescript.ts
@@ -1,7 +1,7 @@
 import { FastifyPluginCallback } from 'fastify'
 
-const plugin: FastifyPluginCallback = function (fastify, opts, next): void {
-  fastify.get('/typescript', (request, reply): void => {
+const plugin: FastifyPluginCallback = function (fastify, _opts, next): void {
+  fastify.get('/typescript', (_request, reply): void => {
     reply.send({ script: 'type' })
   })