Wrap patterns in non-matching group before output (#79)

Author: blakeembrey

index.js

@@ -133,7 +133,7 @@ function tokensToFunction (tokens) {
   // Compile all the patterns before compilation.
   for (var i = 0; i < tokens.length; i++) {
     if (typeof tokens[i] === 'object') {
-      matches[i] = new RegExp('^' + tokens[i].pattern + '$')
+      matches[i] = new RegExp('^(?:' + tokens[i].pattern + ')$')
     }
   }
 
@@ -337,7 +337,7 @@ function tokensToRegExp (tokens, options) {
       route += escapeString(token)
     } else {
       var prefix = escapeString(token.prefix)
-      var capture = token.pattern
+      var capture = '(?:' + token.pattern + ')'
 
       if (token.repeat) {
         capture += '(?:' + prefix + capture + ')*'

test.ts

@@ -859,6 +859,35 @@ var TESTS: Test[] = [
       [{ route: 'that' }, '/that']
     ]
   ],
+  [
+    '/:path(abc|xyz)*',
+    null,
+    [
+      {
+        name: 'path',
+        prefix: '/',
+        delimiter: '/',
+        optional: true,
+        repeat: true,
+        pattern: 'abc|xyz'
+      }
+    ],
+    [
+      ['/abc', ['/abc', 'abc']],
+      ['/abc/abc', ['/abc/abc', 'abc/abc']],
+      ['/xyz/xyz', ['/xyz/xyz', 'xyz/xyz']],
+      ['/abc/xyz', ['/abc/xyz', 'abc/xyz']],
+      ['/abc/xyz/abc/xyz', ['/abc/xyz/abc/xyz', 'abc/xyz/abc/xyz']],
+      ['/xyzxyz', null]
+    ],
+    [
+      [{ path: 'abc' }, '/abc'],
+      [{ path: ['abc', 'xyz'] }, '/abc/xyz'],
+      [{ path: ['xyz', 'abc', 'xyz'] }, '/xyz/abc/xyz'],
+      [{ path: 'abc123' }, null],
+      [{ path: 'abcxyz' }, null]
+    ]
+  ],
 
   /**
    * Prefixed slashes could be omitted.