Fix surrounded optional parameter

index.js

@@ -95,8 +95,9 @@ function insertParameters(route, obj) {
     str = str.replace(route.keys[key], value + '$1');
   });
 
+  // massage optional parameter
   return str
-    .replace(/\(\/\)$/g, '')
+    .replace(/\(\/\)(\/|$)/g, '$1')
     .replace(/[)(]/g, '');
 }
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "docuri",
-  "version": "4.0.0",
+  "version": "4.0.1",
   "description": "Rich document ids for CouchDB",
   "main": "index.js",
   "scripts": {

test/docuri-test.js

@@ -11,41 +11,3 @@ test('simple route', function(t) {
   t.end();
 });
 
-test('named parameter', function(t) {
-  docuri.route('page/:id', 'page');
-
-  t.deepEqual(docuri.page('page/mypage'), { id: 'mypage' }, 'parsed page has "id" set to "mypage"');
-  t.equal(docuri.page({ id: 'mypage' }), 'page/mypage', 'stringified page results in "page/mypage"');
-
-  t.end();
-});
-
-test('optional parameter', function(t) {
-  docuri.route('page(/:id)', 'page');
-
-  t.deepEqual(docuri.page('page'), {}, 'parsed page returns empty object');
-  t.deepEqual(docuri.page('page/mypage'), { id: 'mypage' }, 'parsed page has "id" set to "mypage"');
-  t.equal(docuri.page(), 'page', 'stringified empty page results in "page"');
-  t.equal(docuri.page({ id: 'mypage' }), 'page/mypage', 'stringified page results in "page/mypage"');
-
-  t.end();
-});
-
-test('two named parameters', function(t) {
-  docuri.route('page/:page_id/content/:id', 'content');
-
-  t.deepEqual(docuri.content('page/mypage/content/mycontent'), { page_id: 'mypage', id: 'mycontent' }, 'parsed content has "page_id" set to "mypage" and "id" set to "mycontent"');
-  t.equal(docuri.content({ page_id: 'mypage', id: 'mycontent' }), 'page/mypage/content/mycontent', 'stringified content results in "page/mypage/content/mycontent"');
-
-  t.end();
-});
-
-test('splat parameter', function(t) {
-  docuri.route('page/*path', 'page');
-
-  t.deepEqual(docuri.page('page/mypage/otherpage'), { path: ['mypage', 'otherpage'] }, 'parsed page has "parts" set to ["mypage","otherpage"]');
-  t.equal(docuri.page({ path: ['mypage', 'otherpage'] }), 'page/mypage/otherpage', 'stringified page results in "page/mypage/otherpage"');
-
-  t.end();
-});
-

test/named-parameter-test.js

@@ -0,0 +1,20 @@
+var test = require('tap').test;
+var docuri = require('..');
+
+test('named parameter', function(t) {
+  docuri.route('page/:id', 'page');
+
+  t.deepEqual(docuri.page('page/mypage'), { id: 'mypage' }, 'parsed page has "id" set to "mypage"');
+  t.equal(docuri.page({ id: 'mypage' }), 'page/mypage', 'stringified page results in "page/mypage"');
+
+  t.end();
+});
+
+test('two named parameters', function(t) {
+  docuri.route('page/:page_id/content/:id', 'content');
+
+  t.deepEqual(docuri.content('page/mypage/content/mycontent'), { page_id: 'mypage', id: 'mycontent' }, 'parsed content has "page_id" set to "mypage" and "id" set to "mycontent"');
+  t.equal(docuri.content({ page_id: 'mypage', id: 'mycontent' }), 'page/mypage/content/mycontent', 'stringified content results in "page/mypage/content/mycontent"');
+
+  t.end();
+});

test/optional-parameter-test.js

@@ -0,0 +1,24 @@
+var test = require('tap').test;
+var docuri = require('..');
+
+test('optional parameter', function(t) {
+  docuri.route('page(/:id)', 'page');
+
+  t.deepEqual(docuri.page('page'), {}, 'parsed page returns empty object');
+  t.deepEqual(docuri.page('page/mypage'), { id: 'mypage' }, 'parsed page has "id" set to "mypage"');
+  t.equal(docuri.page(), 'page', 'stringified empty page results in "page"');
+  t.equal(docuri.page({ id: 'mypage' }), 'page/mypage', 'stringified page results in "page/mypage"');
+
+  t.end();
+});
+
+test('surrounded optional parameter', function(t) {
+  docuri.route('page(/:id)/suffix', 'page');
+
+  t.deepEqual(docuri.page('page/suffix'), {}, 'parsed page returns empty object');
+  t.deepEqual(docuri.page('page/mypage/suffix'), { id: 'mypage' }, 'parsed page has "id" set to "mypage"');
+  t.equal(docuri.page(), 'page/suffix', 'stringified empty page results in "page"');
+  t.equal(docuri.page({ id: 'mypage' }), 'page/mypage/suffix', 'stringified page results in "page/mypage"');
+
+  t.end();
+});

test/splat-parameter-test.js

@@ -0,0 +1,12 @@
+var test = require('tap').test;
+var docuri = require('..');
+
+test('splat parameter', function(t) {
+  docuri.route('page/*path', 'page');
+
+  t.deepEqual(docuri.page('page/mypage/otherpage'), { path: ['mypage', 'otherpage'] }, 'parsed page has "parts" set to ["mypage","otherpage"]');
+  t.equal(docuri.page({ path: ['mypage', 'otherpage'] }), 'page/mypage/otherpage', 'stringified page results in "page/mypage/otherpage"');
+
+  t.end();
+});
+