added the the test cases of a program accept empty string as key or value and updated the function

Author: Ebrahim-Moqbel

Sprint-2/implement/querystring.js

@@ -3,16 +3,22 @@ function parseQueryString(queryString) {
   if (queryString.length === 0) {
     return queryParams;
   }
-  const keyValuePairs = queryString.replace(/\+/g," ").split("&").filter((pair)=> pair !=='');
+  const keyValuePairs = queryString
+    .replace(/\+/g, " ")
+    .split("&")
+    .filter((pair) => pair !== "");
 
   for (const pair of keyValuePairs) {
     const indexFirstEqual = pair.indexOf("=");
-    const [key, value]=[
-      decodeURIComponent(pair.slice(0,indexFirstEqual)),
-      decodeURIComponent(pair.slice(indexFirstEqual+1))
-    ]
-    queryParams[key]=value;
-
+    let key, value;
+    if (indexFirstEqual === -1) {
+      key = decodeURIComponent(pair);
+      value = "";
+    } else {
+      ((key = decodeURIComponent(pair.slice(0, indexFirstEqual))),
+        (value = decodeURIComponent(pair.slice(indexFirstEqual + 1))));
+    }
+    queryParams[key] = value;
   }
 
   return queryParams;

Sprint-2/implement/querystring.test.js

@@ -3,7 +3,7 @@
 // Below are some test cases the implementation doesn't handle well.
 // Fix the implementation for these tests, and try to think of as many other edge cases as possible - write tests and fix those too.
 
-const parseQueryString = require("./querystring.js")
+const parseQueryString = require("./querystring.js");
 
 test("should parse values containing '='", () => {
   expect(parseQueryString("equation=a=b-2")).toEqual({
@@ -20,6 +20,8 @@ test("should ignore empty key-value pairs", () => {
 
 test("should accept empty string as key or as value", () => {
   expect(parseQueryString("=value")).toEqual({ "": "value" });
+  expect(parseQueryString("key")).toEqual({ key: "" });
+  expect(parseQueryString("key=")).toEqual({ key: "" });
   expect(parseQueryString("=")).toEqual({ "": "" });
 });