querystring function and testing it

Author: Nataliia74

Sprint-2/implement/querystring.js

@@ -6,11 +6,21 @@ function parseQueryString(queryString) {
   const keyValuePairs = queryString.split("&");
 
   for (const pair of keyValuePairs) {
-    const [key, value] = pair.split("=");
-    queryParams[key] = value;
+    if (!pair) {
+      continue;
+    } else if (pair.includes("=")) {
+      let i = pair.indexOf("=");
+      let key = pair.slice(0, i).trim();
+      let value = pair.slice(i + 1).trim();
+      queryParams[key] = value;
+    } else {
+      queryParams[pair.trim()] = "";
+    }
   }
 
   return queryParams;
 }
+const result = parseQueryString("more=1=1+0&equation = x = y + 1&goga");
+console.log(result);
 
 module.exports = parseQueryString;

Sprint-2/implement/querystring.test.js

@@ -3,10 +3,54 @@
 // Below is one test case for an edge case the implementation doesn't handle well.
 // Fix the implementation for this test, 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("parses querystring values containing =", () => {
-  expect(parseQueryString("equation=x=y+1")).toEqual({
-    "equation": "x=y+1",
+describe("edge cases for parseQueryString function", () => {
+  test("parses querystring values containing =", () => {
+    expect(parseQueryString("equation=x=y+1")).toEqual({
+      equation: "x=y+1",
+    });
+  });
+
+  test("parses querystring values containing '=' and additional whitespaces around '=' sign", () => {
+    expect(parseQueryString("equation = x=y+1")).toEqual({
+      equation: "x=y+1",
+    });
+  });
+
+  test("parses empty querystring", () => {
+    expect(parseQueryString("")).toEqual({});
+  });
+
+  test("parses querystring with duplicate keys", () => {
+    expect(parseQueryString("equation=x&equation=y+1")).toEqual({
+      equation: "x",
+      equation: "y+1",
+    });
+  });
+
+  test("parses querystring with leading or trailing &s", () => {
+    expect(parseQueryString("&equation=x&")).toEqual({
+      equation: "x",
+    });
+  });
+
+  test("parses querystring with missing key", () => {
+    expect(parseQueryString("=x")).toEqual({
+      "": "x",
+    });
+  });
+
+  test("parses querystring with missing value", () => {
+    expect(parseQueryString("equation=")).toEqual({
+      equation: "",
+    });
+  });
+
+  test("parses querystring with missing '='", () => {
+    expect(parseQueryString("equation&gh5")).toEqual({
+      equation: "",
+      gh5: "",
+    });
   });
 });