Merge pull request #12 from dartess/master

added trim leading "?" in init string, parameters without values and tests

Author: WebReflection

build/url-search-params.amd.js

@@ -34,6 +34,9 @@ function decode(str) {
 function URLSearchParams(query) {
   this[secret] = Object.create(null);
   if (!query) return;
+  if (query.charAt(0) === '?') {
+    query = query.slice(1);
+  }
   for (var
     index, value,
     pairs = (query || '').split('&'),
@@ -47,6 +50,11 @@ function URLSearchParams(query) {
         decode(value.slice(0, index)),
         decode(value.slice(index + 1))
       );
+    } else if (value.length){
+      this.append(
+        decode(value),
+        ''
+      );
     }
   }
 }

build/url-search-params.js

@@ -34,6 +34,9 @@ function decode(str) {
 function URLSearchParams(query) {
   this[secret] = Object.create(null);
   if (!query) return;
+  if (query.charAt(0) === '?') {
+    query = query.slice(1);
+  }
   for (var
     index, value,
     pairs = (query || '').split('&'),
@@ -47,6 +50,11 @@ function URLSearchParams(query) {
         decode(value.slice(0, index)),
         decode(value.slice(index + 1))
       );
+    } else if (value.length){
+      this.append(
+        decode(value),
+        ''
+      );
     }
   }
 }

build/url-search-params.max.amd.js

@@ -33,6 +33,9 @@ function decode(str) {
 function URLSearchParams(query) {
   this[secret] = Object.create(null);
   if (!query) return;
+  if (query.charAt(0) === '?') {
+    query = query.slice(1);
+  }
   for (var
     index, value,
     pairs = (query || '').split('&'),
@@ -46,6 +49,11 @@ function URLSearchParams(query) {
         decode(value.slice(0, index)),
         decode(value.slice(index + 1))
       );
+    } else if (value.length){
+      this.append(
+        decode(value),
+        ''
+      );
     }
   }
 }

build/url-search-params.max.js

@@ -11,6 +11,9 @@ function decode(str) {
 function URLSearchParams(query) {
   this[secret] = Object.create(null);
   if (!query) return;
+  if (query.charAt(0) === '?') {
+    query = query.slice(1);
+  }
   for (var
     index, value,
     pairs = (query || '').split('&'),
@@ -24,6 +27,11 @@ function URLSearchParams(query) {
         decode(value.slice(0, index)),
         decode(value.slice(index + 1))
       );
+    } else if (value.length){
+      this.append(
+        decode(value),
+        ''
+      );
     }
   }
 }

build/url-search-params.node.js

@@ -9,9 +9,31 @@ wru.test([
       wru.assert(/^function|object$/.test(typeof URLSearchParams));
     }
   }, {
-    name: 'basics',
+    name: 'basics (including parameters without values)',
     test: function () {
-      var usp = new URLSearchParams('a=1&b=2');
+      var usp = new URLSearchParams('a=1&b=2&c');
+      wru.assert('has keys', usp.has('a') && usp.has('b') && usp.has('c'));
+      wru.assert('a returns right value', usp.get('a') === '1');
+      wru.assert('b returns right value', usp.get('b') === '2');
+      wru.assert('c returns right value', usp.get('c') === '');
+      wru.assert('a getAll returns right value', usp.getAll('a').join(',') === '1');
+      wru.assert('b getAll returns right value', usp.getAll('b').join(',') === '2');
+      wru.assert('c getAll returns right value', usp.getAll('c').join(',') === '');
+      usp.append('a', '3');
+      wru.assert('append adds values', usp.getAll('a').join(',') === '1,3');
+      wru.assert('append preserves get', usp.get('a') === '1');
+      wru.assert('append does not affect others', usp.getAll('b').join(',') === '2' && usp.getAll('c').join(',') === '');
+      usp.set('a', '4');
+      wru.assert('set overwrites known values', usp.getAll('a').join(',') === '4');
+      usp['delete']('a');
+      wru.assert('usp can delete', usp.has('a') === false);
+      wru.assert('usp can return null', usp.get('a') === null);
+      wru.assert('usp to string works as expected', usp.toString() === 'b=2&c=');
+    }
+  }, {
+    name: 'basics with leading "?"',
+    test: function () {
+      var usp = new URLSearchParams('?a=1&b=2');
       wru.assert('has keys', usp.has('a') && usp.has('b'));
       wru.assert('a returns right value', usp.get('a') === '1');
       wru.assert('b returns right value', usp.get('b') === '2');