0.8.0

index.js

@@ -14,24 +14,32 @@ class NamedRoutes {
 }
 
 const namedRoutesPlugin = function (fastify, opts, next) {
-  let instance = new NamedRoutes({
+  let plugin = new NamedRoutes({
     baseUrl: opts.baseUrl
   })
 
+  fastify.decorate('namedRoutes', plugin)
+
   fastify.addHook('onRoute', (routeOptions) => {
     if (routeOptions.config && routeOptions.config.routeName) {
       let { routeName } = routeOptions.config
 
-      if (instance.routesMap.has(routeName)) {
-        fastify.log.warn(`Named route ${routeName} already present, skipping...`)
+      if (plugin.routesMap.has(routeName)) {
+        // let existingRoute = plugin.routesMap.get(routeName)
+        // onRoute hook is called twice for normal route prefix or when prefixTrailingSlash = 'both',
+        // so we should complain ONLY if this is not the case
+        // if (!existingRoute.config || !existingRoute.config.url) {
+        fastify.log.warn(`Named route '${routeName}' already present, skipping...`)
+        // }
       } else {
-        fastify.log.info(`Adding named route => ${routeName}`)
-        instance.routesMap.set(routeName, routeOptions)
+        fastify.log.info(`Adding named route '${routeName}'`)
+        console.debug(routeOptions)
+        plugin.routesMap.set(routeName, routeOptions)
+        console.debug(plugin.routesMap)
       }
     }
   })
 
-  fastify.decorate('namedRoutes', instance)
   next()
 }
 

package.json

@@ -1,11 +1,11 @@
 {
   "name": "@gendronb/fastify-named-routes",
-  "version": "0.6.0",
+  "version": "0.8.0",
   "description": "Named routes support for Fastify",
   "main": "index.js",
   "scripts": {
     "test:lint": "standard",
-    "test:unit": "tap -J test/**/*.test.js",
+    "test:unit": "tap -J test/**/*.test.js --100",
     "test": "npm run test:lint && npm run test:unit"
   },
   "precommit": "test",

test/prefixed-routes.test.js.skip

@@ -0,0 +1,37 @@
+'use strict'
+
+const test = require('tap').test
+
+const plugin = require('..')
+
+const Fastify = require('fastify')
+
+test('does support prefixed routes', t => {
+  t.plan(1)
+
+  const fastify = Fastify({
+    logger: {
+      level: 'info'
+    }
+  })
+
+  fastify.addHook('onRoute', async function (routeOptions) {
+    // console.debug(!routeOptions.prefixTrailingSlash || routeOptions.prefixTrailingSlash == 'both')
+    console.debug('onRoute', routeOptions)
+  })
+
+  fastify.register(plugin)
+
+  const route = async function (fastify, opts) {
+    fastify.get('/', { config: { routeName: 'prefixed' } }, async () => 'Test')
+  }
+
+  fastify.register(route, { prefix: '/prefix' })
+
+  fastify.listen(0, function () {
+    console.debug(fastify.namedRoutes.get('prefixed'))
+    t.equal(fastify.namedRoutes.get('prefixed').path, '/prefix')
+    // t.equal('toto', 'titi')
+    fastify.server.unref()
+  })
+})