fix #151 Wildcard route should not be blocked by Parametric with diff…

node.js

@@ -122,7 +122,12 @@ Node.prototype.findChild = function (path, method) {
     }
   }
 
-  child = this.children[':'] || this.children['*']
+  child = this.children[':']
+  if (child !== undefined && (child.numberOfChildren > 0 || child.handlers[method] !== null)) {
+    return child
+  }
+
+  child = this.children['*']
   if (child !== undefined && (child.numberOfChildren > 0 || child.handlers[method] !== null)) {
     return child
   }
@@ -138,7 +143,12 @@ Node.prototype.findVersionChild = function (version, path, method) {
     }
   }
 
-  child = this.children[':'] || this.children['*']
+  child = this.children[':']
+  if (child !== undefined && (child.numberOfChildren > 0 || child.getVersionHandler(version, method) !== null)) {
+    return child
+  }
+
+  child = this.children['*']
   if (child !== undefined && (child.numberOfChildren > 0 || child.getVersionHandler(version, method) !== null)) {
     return child
   }

test/issue-151.test.js

@@ -0,0 +1,55 @@
+'use strict'
+
+const t = require('tap')
+const test = t.test
+const FindMyWay = require('../')
+
+test('Wildcard route should not be blocked by Parametric with different method / 1', t => {
+  t.plan(1)
+  const findMyWay = FindMyWay({
+    defaultRoute: (req, res) => {
+      t.fail('Should not be defaultRoute')
+    }
+  })
+
+  findMyWay.on('OPTIONS', '/*', (req, res, params) => {
+    t.fail('Should not be here')
+  })
+
+  findMyWay.on('OPTIONS', '/obj/*', (req, res, params) => {
+    t.strictEqual(req.method, 'OPTIONS')
+  })
+
+  findMyWay.on('GET', '/obj/:id', (req, res, params) => {
+    t.fail('Should not be GET')
+  })
+
+  findMyWay.lookup({ method: 'OPTIONS', url: '/obj/params', headers: {} }, null)
+})
+
+test('Wildcard route should not be blocked by Parametric with different method / 2', t => {
+  t.plan(1)
+  const findMyWay = FindMyWay({
+    defaultRoute: (req, res) => {
+      t.fail('Should not be defaultRoute')
+    }
+  })
+
+  findMyWay.on('OPTIONS', '/*', { version: '1.2.3' }, (req, res, params) => {
+    t.fail('Should not be here')
+  })
+
+  findMyWay.on('OPTIONS', '/obj/*', { version: '1.2.3' }, (req, res, params) => {
+    t.strictEqual(req.method, 'OPTIONS')
+  })
+
+  findMyWay.on('GET', '/obj/:id', { version: '1.2.3' }, (req, res, params) => {
+    t.fail('Should not be GET')
+  })
+
+  findMyWay.lookup({
+    method: 'OPTIONS',
+    url: '/obj/params',
+    headers: { 'accept-version': '1.2.3' }
+  }, null)
+})