You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
'use strict'constt=require('tap')consttest=t.testconstFindMyWay=require('../')test('Parametric regex match with similar routes',(t)=>{t.plan(1)constfindMyWay=FindMyWay()findMyWay.on('GET','/:a(a)',()=>{})findMyWay.on('GET','/:b(b)/static',()=>{})t.same(findMyWay.find('GET','/b/static',{}).params,{'b': 'b'})})
The text was updated successfully, but these errors were encountered:
@subhero24 I was wrong here #280 (comment). We can resolve it because there would always be only one matched route. The problem if we add multiple parametric route support is to check that /:a(a)/static and /:b(b)/static can't be registered at the same time.
Using two routes which start with a regex param, the second route which should match
/b/static/
doesn't.Here is a complete test:
The text was updated successfully, but these errors were encountered: