refactor: remove unused done function (#874)

Author: inyourtime

lib/mode/dynamic.js

@@ -31,7 +31,7 @@ module.exports = function (fastify, opts, done) {
     string: null
   }
 
-  const swagger = resolveSwaggerFunction(opts, cache, routes, Ref, done)
+  const swagger = resolveSwaggerFunction(opts, cache, routes, Ref)
   fastify.decorate(opts.decorator, swagger)
 
   done()

lib/spec/openapi/index.js

@@ -4,7 +4,7 @@ const yaml = require('yaml')
 const { shouldRouteHide } = require('../../util/should-route-hide')
 const { prepareDefaultOptions, prepareOpenapiObject, prepareOpenapiMethod, prepareOpenapiSchemas, normalizeUrl, resolveServerUrls } = require('./utils')
 
-module.exports = function (opts, cache, routes, Ref, done) {
+module.exports = function (opts, cache, routes, Ref) {
   let ref
 
   const defOpts = prepareDefaultOptions(opts)
@@ -17,7 +17,7 @@ module.exports = function (opts, cache, routes, Ref, done) {
     }
 
     // Base Openapi info
-    const openapiObject = prepareOpenapiObject(defOpts, done)
+    const openapiObject = prepareOpenapiObject(defOpts)
 
     ref = Ref()
     openapiObject.components.schemas = prepareOpenapiSchemas({

lib/spec/swagger/index.js

@@ -4,7 +4,7 @@ const yaml = require('yaml')
 const { shouldRouteHide } = require('../../util/should-route-hide')
 const { prepareDefaultOptions, prepareSwaggerObject, prepareSwaggerMethod, normalizeUrl, prepareSwaggerDefinitions } = require('./utils')
 
-module.exports = function (opts, cache, routes, Ref, done) {
+module.exports = function (opts, cache, routes, Ref) {
   let ref
 
   const defOpts = prepareDefaultOptions(opts)
@@ -16,7 +16,7 @@ module.exports = function (opts, cache, routes, Ref, done) {
       if (cache.object) return cache.object
     }
 
-    const swaggerObject = prepareSwaggerObject(defOpts, done)
+    const swaggerObject = prepareSwaggerObject(defOpts)
 
     ref = Ref()
     swaggerObject.definitions = prepareSwaggerDefinitions({

lib/util/resolve-swagger-function.js

@@ -1,10 +1,10 @@
 'use strict'
 
-function resolveSwaggerFunction (opts, cache, routes, Ref, done) {
+function resolveSwaggerFunction (opts, cache, routes, Ref) {
   if (opts.openapi === undefined || opts.openapi === null) {
-    return require('../spec/swagger')(opts, cache, routes, Ref, done)
+    return require('../spec/swagger')(opts, cache, routes, Ref)
   } else {
-    return require('../spec/openapi')(opts, cache, routes, Ref, done)
+    return require('../spec/openapi')(opts, cache, routes, Ref)
   }
 }