Fix error on trying to parse empty string. Return empty object instead

pahan35 · pahan35 · commit c45898c911e7 · 2018-08-30T01:45:57.000+03:00
diff --git a/index.js b/index.js
@@ -3,15 +3,17 @@ function parseConnectionParams(paramstring, parseValues = false) {
         const params = {};
         const numberRegex = /^[-+]?(\d+\.)?\d+(E[-+]?\d+)?$/i;
 
-        paramstring.split('&').map(s => {
-            let [key, val] = s.split('=');
-            const lowerCasedValue = val.toLowerCase();
-            if (lowerCasedValue === 'true') val = true;
-            else if (lowerCasedValue === 'false') val = false;
-            else if (numberRegex.test(val)) val = parseFloat(val);
+        if (paramstring) {
+            paramstring.split('&').map(s => {
+                let [key, val] = s.split('=');
+                const lowerCasedValue = val.toLowerCase();
+                if (lowerCasedValue === 'true') val = true;
+                else if (lowerCasedValue === 'false') val = false;
+                else if (numberRegex.test(val)) val = parseFloat(val);
 
-            params[key] = val;
-        });
+                params[key] = val;
+            });
+        }
 
         return params;
     } else {
diff --git a/test/index.test.js b/test/index.test.js
@@ -9,6 +9,11 @@ describe('Test parseConnectionParams method', function () {
         assert.equal(result, testedParamString)
     });
 
+    it('should return empty object', function () {
+        const result = parseConnectionParams('', true);
+        assert.deepEqual(result, {})
+    });
+
     describe('Test parsed values', function () {
         const result = parseConnectionParams(testedParamString, true);
 
