chore: update semantic release

Allow inserting falsy values, excluding undefined

.travis.yml

@@ -1,4 +1,3 @@
-sudo: false
 language: node_js
 cache:
   directories:
@@ -6,15 +5,13 @@ cache:
 notifications:
   email: false
 node_js:
-  - 4
-  - iojs-v3
-  - iojs-v2
-  - iojs-v1
-  - '0.12'
-  - '0.10'
-before_install:
-  - npm i -g npm@^2.0.0
+  - '7'
+  - '6'
+  - '4'
 before_script:
   - npm prune
 after_success:
   - npm run semantic-release
+branches:
+  except:
+    - /^v\d+\.\d+\.\d+$/

index.js

@@ -89,7 +89,7 @@ function insertParameters(route, obj) {
 
   Object.keys(route.keys).forEach(function(key) {
     var k = key.substr(1);
-    var value = obj[k] || '';
+    var value = (obj[k] !== undefined) ? obj[k] : '';
 
     if (Array.isArray(value)) {
       value = value.map(encodeURIComponent).join('/');

package.json

@@ -21,7 +21,8 @@
   "homepage": "https://github.com/jo/docuri",
   "devDependencies": {
     "shuffle-array": "^0.1.0",
-    "tap": "^0.4.8",
-    "semantic-release": "^4.3.5"
-  }
+    "tap": "^5.1.1",
+    "semantic-release": "^6.3.2"
+  },
+  "version": "0.0.0-development"
 }

test/named-parameter-test.js

@@ -10,6 +10,18 @@ test('named parameter', function(t) {
   t.end();
 });
 
+test('insert falsy parameter values', function(t) {
+  var page = docuri.route('page/:id');
+
+  t.deepEqual(page({}), 'page/', 'stringifies without id');
+  t.deepEqual(page({id: undefined}), 'page/', 'stringifies without id');
+  t.deepEqual(page({id: 0}), 'page/0', 'inserts "0"');
+  t.deepEqual(page({id: false}), 'page/false', 'inserts "false"');
+  t.deepEqual(page({id: null}), 'page/null', 'inserts "null"');
+
+  t.end();
+});
+
 test('two named parameters', function(t) {
   var content = docuri.route('page/:page_id/content/:id');
 
@@ -19,6 +31,14 @@ test('two named parameters', function(t) {
   t.end();
 });
 
+test('named parameters replaced all at once', function(t) {
+  var page = docuri.route('page/:id/:bar');
+
+  t.equal(page({ id: ':bar', bar: 'foo' }), 'page/:bar/foo', 'url built correctly, no conflicts', { todo: true });
+
+  t.end();
+});
+
 test('named parameter followed by optional parameter', function(t) {
   var page = docuri.route('page/:id(/:optional)');
 
@@ -43,3 +63,11 @@ test('named parameter with slash in the content', function(t) {
 
   t.end();
 });
+
+test('named parameter with parens in the content', function(t) {
+  var page = docuri.route('page/:id');
+
+  t.deepEqual(page(page({ id: 'value(slash)' })), { id: 'value(slash)' }, 'parsed page has parens  set back correctly', { todo: true });
+
+  t.end();
+});