fix: avoid overriding the prefix option (#224)

Author: inyourtime

index.js

@@ -10,19 +10,19 @@ async function fastifySwaggerUi (fastify, opts) {
 
   // if no logo is provided, read default static logo
   let logoContent = opts.logo
-  if (logoContent == null) {
+  if (logoContent === undefined) {
     const bufferLogoContent = await fsPromises.readFile(path.join(__dirname, './static/logo.svg'))
     logoContent = { type: 'image/svg+xml', content: bufferLogoContent }
   }
 
   await fastify.register(require('./lib/routes'), {
+    ...opts,
     prefix: opts.routePrefix || '/documentation',
     uiConfig: opts.uiConfig || {},
     initOAuth: opts.initOAuth || {},
     hooks: opts.uiHooks,
     theme: opts.theme || {},
-    logo: logoContent,
-    ...opts
+    logo: logoContent
   })
 }
 

test/route.test.js

@@ -623,7 +623,7 @@ test('/api/v1/docs should display index html with correct asset urls', async (t)
   t.plan(13)
   const fastify = Fastify()
   await fastify.register(fastifySwagger, swaggerOption)
-  await fastify.register(fastifySwaggerUi, { prefix: '/api/v1/docs', theme: { js: [{ filename: 'theme-js.js' }] } })
+  await fastify.register(fastifySwaggerUi, { routePrefix: '/api/v1/docs', theme: { js: [{ filename: 'theme-js.js' }] } })
 
   const res = await fastify.inject({
     method: 'GET',
@@ -750,3 +750,21 @@ test('/docs/ should display index html with correct asset urls when documentatio
   t.assert.strictEqual(res.payload.includes('href="./index.css"'), false)
   t.assert.strictEqual(res.payload.includes('src="./theme/theme-js.js"'), false)
 })
+
+test('should ignore prefix when register plugin', async (t) => {
+  t.plan(4)
+  const fastify = Fastify()
+  await fastify.register(fastifySwagger, swaggerOption)
+  await fastify.register(fastifySwaggerUi, { routePrefix: '/v1/documentation', prefix: '/' })
+
+  fastify.get('/', () => {})
+
+  const res = await fastify.inject({
+    method: 'GET',
+    url: '/v1/documentation'
+  })
+  t.assert.deepStrictEqual(res.statusCode, 200)
+  t.assert.deepStrictEqual(res.headers.location, undefined)
+  t.assert.deepStrictEqual(typeof res.payload, 'string')
+  t.assert.deepStrictEqual('text/html; charset=utf-8', res.headers['content-type'])
+})